Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "MediaController: workaround to disable seek" into mnc-dev

Browse Source
This commit is contained in:
Robert Shih
2015-04-28 00:30:11 +00:00
committed by Android (Google) Code Review
parent 1ce80dc0e2 39dbbd93e4
commit 43429656ff
1 changed files with 10 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
core/java/android/widget/MediaController.java
View File

@@ -326,6 +326,16 @@ public class MediaController extends FrameLayout {
if (mFfwdButton != null && !mPlayer.canSeekForward()) {
mFfwdButton.setEnabled(false);
}
// TODO What we really should do is add a canSeek to the MediaPlayerControl interface;
// this scheme can break the case when applications want to allow seek through the
// progress bar but disable forward/backward buttons.
//
// However, currently the flags SEEK_BACKWARD_AVAILABLE, SEEK_FORWARD_AVAILABLE,
// and SEEK_AVAILABLE are all (un)set together; as such the aforementioned issue
// shouldn't arise in existing applications.
if (mProgress != null && !mPlayer.canSeekBackward() && !mPlayer.canSeekForward()) {
mProgress.setEnabled(false);
}
} catch (IncompatibleClassChangeError ex) {
// We were given an old version of the interface, that doesn't have
// the canPause/canSeekXYZ methods. This is OK, it just means we