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
diff --git a/java/com/android/newbubble/NewBubble.java b/java/com/android/newbubble/NewBubble.java
index 65f7a0a..09634b5 100644
--- a/java/com/android/newbubble/NewBubble.java
+++ b/java/com/android/newbubble/NewBubble.java
@@ -926,13 +926,7 @@
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