summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Josh Tsuji <tsuji@google.com> 2020-06-10 17:51:34 -0400
committer Josh Tsuji <tsuji@google.com> 2020-06-10 18:00:32 -0400
commite78129e3f167f6b61f98b47c7beeaf07a5b124c6 (patch)
treedf467fe3dfd89421e2d3956f8881d3afc37fcc80
parentab6c4a7685600ac67798a931377814b2244b4006 (diff)
Don't crash if we've already been educated about Bubbles.
Let people apply their new knowledge. Test: manual Fixes: 158695768 Change-Id: I95db75f81ca95ebd1aa0d18539e04b42cb3f1dee
-rw-r--r--packages/SystemUI/src/com/android/systemui/bubbles/BubbleStackView.java9
1 files changed, 6 insertions, 3 deletions
diff --git a/packages/SystemUI/src/com/android/systemui/bubbles/BubbleStackView.java b/packages/SystemUI/src/com/android/systemui/bubbles/BubbleStackView.java
index 6bf20b88f64a..492d138e7ddb 100644
--- a/packages/SystemUI/src/com/android/systemui/bubbles/BubbleStackView.java
+++ b/packages/SystemUI/src/com/android/systemui/bubbles/BubbleStackView.java
@@ -1975,9 +1975,12 @@ public class BubbleStackView extends FrameLayout
@Override
public void subtractObscuredTouchableRegion(Region touchableRegion, View view) {
// If the notification shade is expanded, or the manage menu is open, or we are showing
- // manage bubbles user education, we shouldn't let the
- // ActivityView steal any touch events from any location.
- if (!mIsExpanded || mShowingManage || mManageEducationView.getVisibility() == VISIBLE) {
+ // manage bubbles user education, we shouldn't let the ActivityView steal any touch events
+ // from any location.
+ if (!mIsExpanded
+ || mShowingManage
+ || (mManageEducationView != null
+ && mManageEducationView.getVisibility() == VISIBLE)) {
touchableRegion.setEmpty();
}
}