commit d9422831d4355f8627e050c38d098af6b52ecd1f
author Joshua Tsuji <tsuji@google.com> Tue Mar 05 13:32:37 2019 -0500
committer Josh Tsuji <tsuji@google.com> Tue Mar 05 22:34:19 2019 +0000
tree 3252667d5877340cb8497207c177ce7ceb821aec
parent 6583df45d023534f2a0223923afd0da09c0c3f8e

Make sure there are bubbles before animating due to IME changes.

Also, don't crash if we try to spring the first bubble when there are no bubbles.

Fixes: 127275756
Test: atest SystemUITests
Change-Id: I6259398d894039d42f93300d0d997d14372f07b6

packages/SystemUI/src/com/android/systemui/bubbles/BubbleController.java

diff --git a/packages/SystemUI/src/com/android/systemui/bubbles/BubbleController.java b/packages/SystemUI/src/com/android/systemui/bubbles/BubbleController.java
index 471619e..656d757 100644
--- a/packages/SystemUI/src/com/android/systemui/bubbles/BubbleController.java
+++ b/packages/SystemUI/src/com/android/systemui/bubbles/BubbleController.java
@@ -582,12 +582,9 @@
                 int displayRotation) throws RemoteException {}
 
         @Override
-        public void onImeVisibilityChanged(boolean imeVisible, int imeHeight)
-                throws RemoteException {
-            if (mStackView != null) {
-                mStackView.post(() -> {
-                    mStackView.onImeVisibilityChanged(imeVisible, imeHeight);
-                });
+        public void onImeVisibilityChanged(boolean imeVisible, int imeHeight) {
+            if (mStackView != null && mStackView.getBubbleCount() > 0) {
+                mStackView.post(() -> mStackView.onImeVisibilityChanged(imeVisible, imeHeight));
             }
         }