commit 9dc38ec22bee0ebc38ae8ec66209ed05e61b5262
author Lyn Han <lynhan@google.com> Wed May 06 10:50:16 2020 -0700
committer Lyn Han <lynhan@google.com> Wed May 06 10:50:16 2020 -0700
tree 8bec03b94158c6672429fdff584c4f8c4ee4024c
parent 1cf3ce8fdec202f818b8015e3b638ce46b85574c

Show bubble by default when stack expands

Fixes: 155486743
Test: manual
- Select overflow, collapse stack, expand stack -> previously selected
bubble is selected
- Select middle bubble, collapse stack, expand stack -> bubble now first
- Select first bubble, collapse stack, expand stack -> first bubble
still selected

Change-Id: Ie287274abd520219311288ba5d3492dae4b45b9e

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

diff --git a/packages/SystemUI/src/com/android/systemui/bubbles/BubbleStackView.java b/packages/SystemUI/src/com/android/systemui/bubbles/BubbleStackView.java
index c802b59..6cdbe2f 100644
--- a/packages/SystemUI/src/com/android/systemui/bubbles/BubbleStackView.java
+++ b/packages/SystemUI/src/com/android/systemui/bubbles/BubbleStackView.java
@@ -454,7 +454,6 @@
                     // that means overflow was previously expanded. Set the selected bubble
                     // internally without going through BubbleData (which would ignore it since it's
                     // already selected).
-                    mBubbleData.setShowingOverflow(true);
                     setSelectedBubble(clickedBubble);
                 }
             } else {
@@ -1342,7 +1341,10 @@
         }
         if (bubbleToSelect == null || bubbleToSelect.getKey() != BubbleOverflow.KEY) {
             mBubbleData.setShowingOverflow(false);
+        } else {
+            mBubbleData.setShowingOverflow(true);
         }
+
         final BubbleViewProvider previouslySelected = mExpandedBubble;
         mExpandedBubble = bubbleToSelect;
         updatePointerPosition();