diff options
| author | Vladislav Kaznacheev <kaznacheev@google.com> | 2017-05-02 09:39:06 -0700 |
|---|---|---|
| committer | Vladislav Kaznacheev <kaznacheev@google.com> | 2017-05-02 09:39:06 -0700 |
| commit | fd48ed6669f490369fdade9a356028838d290fcd (patch) | |
| tree | 2ca518207edff4d290d0477cf9cda9a738b910ad /core/java/android/widget/VideoView.java | |
| parent | af12b17674ba2122da6512358b52800829734591 (diff) | |
Fix VideoView touch and trackball handlers to call super.
Currently VideoView.onTouchEvent and VideoView.onTrackballEvent do not
call super. This completely disables several useful View features
such as long click, context menu or tooltips.
Bug: 37755662
Test: manual
Change-Id: I0c82c32da36e4ca7cacc78027daed2fa298a6b58
Diffstat (limited to 'core/java/android/widget/VideoView.java')
| -rw-r--r-- | core/java/android/widget/VideoView.java | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/core/java/android/widget/VideoView.java b/core/java/android/widget/VideoView.java index 7b2efeac75fc..58a2b0f00877 100644 --- a/core/java/android/widget/VideoView.java +++ b/core/java/android/widget/VideoView.java @@ -708,18 +708,20 @@ public class VideoView extends SurfaceView @Override public boolean onTouchEvent(MotionEvent ev) { - if (isInPlaybackState() && mMediaController != null) { + if (ev.getAction() == MotionEvent.ACTION_DOWN + && isInPlaybackState() && mMediaController != null) { toggleMediaControlsVisiblity(); } - return false; + return super.onTouchEvent(ev); } @Override public boolean onTrackballEvent(MotionEvent ev) { - if (isInPlaybackState() && mMediaController != null) { + if (ev.getAction() == MotionEvent.ACTION_DOWN + && isInPlaybackState() && mMediaController != null) { toggleMediaControlsVisiblity(); } - return false; + return super.onTrackballEvent(ev); } @Override |