From b26d578ff39b6592dd7d747561afea0409a05db6 Mon Sep 17 00:00:00 2001 From: insun Date: Wed, 5 Aug 2020 12:57:32 +0100 Subject: [PATCH] Fix bug that clicking didn't show up controls at the beginning StyledLayoutManager#isFullyVisibility() was wrong and so hiding/showing logic didn't work properly. PiperOrigin-RevId: 324996446 --- .../exoplayer2/ui/StyledPlayerControlViewLayoutManager.java | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/library/ui/src/main/java/com/google/android/exoplayer2/ui/StyledPlayerControlViewLayoutManager.java b/library/ui/src/main/java/com/google/android/exoplayer2/ui/StyledPlayerControlViewLayoutManager.java index 907b055def..a5e6e40df6 100644 --- a/library/ui/src/main/java/com/google/android/exoplayer2/ui/StyledPlayerControlViewLayoutManager.java +++ b/library/ui/src/main/java/com/google/android/exoplayer2/ui/StyledPlayerControlViewLayoutManager.java @@ -53,7 +53,7 @@ import java.util.ArrayList; private final Runnable hideControllerRunnable; private final OnLayoutChangeListener onLayoutChangeListener; - private int uxState = UX_STATE_ALL_VISIBLE; + private int uxState; private boolean initiallyHidden; private boolean isMinimalMode; private boolean needToShowBars; @@ -88,6 +88,7 @@ import java.util.ArrayList; hideControllerRunnable = this::hideController; onLayoutChangeListener = this::onLayoutChange; animationEnabled = true; + uxState = UX_STATE_ALL_VISIBLE; } public void show() { @@ -432,7 +433,7 @@ import java.util.ArrayList; if (styledPlayerControlView == null) { return false; } - return uxState == UX_STATE_ALL_VISIBLE; + return uxState == UX_STATE_ALL_VISIBLE && styledPlayerControlView.isVisible(); } private void setUxState(int uxState) {