Don't consume touch events if no controller is attached.

Issue:#6109
PiperOrigin-RevId: 255933121
This commit is contained in:
tonihei 2019-07-01 13:03:07 +01:00 committed by Oliver Woodman
parent 5365272b60
commit 71e0f2e81c

View file

@ -1050,6 +1050,9 @@ public class PlayerView extends FrameLayout implements AdsLoader.AdViewProvider
@Override
public boolean onTouchEvent(MotionEvent event) {
if (!useController || player == null) {
return false;
}
switch (event.getAction()) {
case MotionEvent.ACTION_DOWN:
isTouching = true;
@ -1150,9 +1153,6 @@ public class PlayerView extends FrameLayout implements AdsLoader.AdViewProvider
// Internal methods.
private boolean toggleControllerVisibility() {
if (!useController || player == null) {
return false;
}
if (!controller.isVisible()) {
maybeShowController(true);
} else if (controllerHideOnTouch) {
@ -1472,6 +1472,9 @@ public class PlayerView extends FrameLayout implements AdsLoader.AdViewProvider
@Override
public boolean onSingleTapUp(MotionEvent e) {
if (!useController || player == null) {
return false;
}
return toggleControllerVisibility();
}
}