From 10f6e8222a5ee1cc2da1b107c3a344b34c726501 Mon Sep 17 00:00:00 2001
From: yueg <yueg@google.com>
Date: Wed, 17 Jan 2018 15:32:18 -0800
Subject: 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
---
 java/com/android/newbubble/NewBubble.java | 8 +-------
 1 file changed, 1 insertion(+), 7 deletions(-)

(limited to 'java/com/android/newbubble')

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
-- 
cgit v1.2.3