From bc7744b9951fe1b35031dbabe5cff349545b56b8 Mon Sep 17 00:00:00 2001 From: Joshua Tsuji Date: Wed, 4 Mar 2020 15:04:59 -0500 Subject: [PATCH] Don't allow the overflow bubble to be dragged out. Test: Try and fail to drag out the overflow bubble. Also quick-fling it towards where the dismiss target would be and observe that sysui doesn't crash. Fixes: 149870727 Change-Id: I23fdfa3a9b0e1b3722704bfc28d5ef42c3bf66c4 --- .../com/android/systemui/bubbles/BubbleStackView.java | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/packages/SystemUI/src/com/android/systemui/bubbles/BubbleStackView.java b/packages/SystemUI/src/com/android/systemui/bubbles/BubbleStackView.java index 10b90f28d893a..0b7dbd2bfa2e4 100644 --- a/packages/SystemUI/src/com/android/systemui/bubbles/BubbleStackView.java +++ b/packages/SystemUI/src/com/android/systemui/bubbles/BubbleStackView.java @@ -1386,6 +1386,11 @@ public class BubbleStackView extends FrameLayout { if (DEBUG_BUBBLE_STACK_VIEW) { Log.d(TAG, "onBubbleDragStart: bubble=" + bubble); } + + if (bubble.equals(mBubbleOverflow.getIconView())) { + return; + } + mExpandedAnimationController.prepareForBubbleDrag(bubble, mMagneticTarget); // We're dragging an individual bubble, so set the magnetized object to the magnetized @@ -1398,7 +1403,7 @@ public class BubbleStackView extends FrameLayout { /** Called with the coordinates to which an individual bubble has been dragged. */ public void onBubbleDragged(View bubble, float x, float y) { - if (!mIsExpanded || mIsExpansionAnimating) { + if (!mIsExpanded || mIsExpansionAnimating || bubble.equals(mBubbleOverflow.getIconView())) { return; } @@ -1413,7 +1418,7 @@ public class BubbleStackView extends FrameLayout { Log.d(TAG, "onBubbleDragFinish: bubble=" + bubble); } - if (!mIsExpanded || mIsExpansionAnimating) { + if (!mIsExpanded || mIsExpansionAnimating || bubble.equals(mBubbleOverflow.getIconView())) { return; }