diff options
author | yueg <yueg@google.com> | 2018-01-17 15:32:18 -0800 |
---|---|---|
committer | Copybara-Service <copybara-piper@google.com> | 2018-01-17 16:37:04 -0800 |
commit | 10f6e8222a5ee1cc2da1b107c3a344b34c726501 (patch) | |
tree | e67c4075d728812876a08789a07c60f85e106b77 /java/com/android/newbubble | |
parent | b494ce8c015cf54a28a47f44791dc95ea6c25aad (diff) |
Move onUiShowing() call from onResume/onPause to onStart/onStop.
It makes sure bubble doesn't appear on split screen when in-call UI is not showing. Although bubble still shows in recents screen.
It also make sure the same onUiShowing() is not called multiple times.
Bug: 67605985,71746139
Test: InCallActivityTest
PiperOrigin-RevId: 182280886
Change-Id: I2a28d0258c722d530000729a89751c6d6b1ee6fd
Diffstat (limited to 'java/com/android/newbubble')
-rw-r--r-- | java/com/android/newbubble/NewBubble.java | 8 |
1 files changed, 1 insertions, 7 deletions
diff --git a/java/com/android/newbubble/NewBubble.java b/java/com/android/newbubble/NewBubble.java index 65f7a0aab..09634b572 100644 --- a/java/com/android/newbubble/NewBubble.java +++ b/java/com/android/newbubble/NewBubble.java @@ -926,13 +926,7 @@ public class NewBubble { root.setOnConfigurationChangedListener( (configuration) -> { if (expanded) { - // Collapse immediately without animation - if (collapseAnimatorSet != null) { - collapseAnimatorSet.removeAllListeners(); - collapseAnimatorSet.cancel(); - } - setDrawerVisibility(View.GONE); - expanded = false; + startCollapse(CollapseEnd.NOTHING, false /* shouldRecoverYPosition */); } // The values in the current MoveHandler may be stale, so replace it. Then ensure the // Window is in bounds |