summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Steven Wu <stwu@google.com> 2019-04-11 14:10:42 -0400
committer Steven Wu <stwu@google.com> 2019-04-12 21:23:32 -0400
commit1684f2d5ba35c131cbcfdcce5fc93d63622b0e9c (patch)
tree0446fa456e662f23745d1979f71c07d480275b5f
parent49e9f80efd86df589555193d90c631f183674908 (diff)
Add logging for bubble flyout action.
Bug: 123543171 Test: manual Change-Id: I7b0a21c0a0c23c211e4c72ec5b2dfc7ea60537d6
-rw-r--r--cmds/statsd/src/atoms.proto1
-rw-r--r--packages/SystemUI/src/com/android/systemui/bubbles/BubbleStackView.java4
2 files changed, 4 insertions, 1 deletions
diff --git a/cmds/statsd/src/atoms.proto b/cmds/statsd/src/atoms.proto
index 90ba7ce0c812..0d5585532db0 100644
--- a/cmds/statsd/src/atoms.proto
+++ b/cmds/statsd/src/atoms.proto
@@ -5575,6 +5575,7 @@ message BubbleUIChanged {
SWIPE_LEFT = 13;
SWIPE_RIGHT = 14;
STACK_EXPANDED = 15;
+ FLYOUT = 16;
}
optional Action action = 6;
diff --git a/packages/SystemUI/src/com/android/systemui/bubbles/BubbleStackView.java b/packages/SystemUI/src/com/android/systemui/bubbles/BubbleStackView.java
index 53e65e662fb8..76c89d09868f 100644
--- a/packages/SystemUI/src/com/android/systemui/bubbles/BubbleStackView.java
+++ b/packages/SystemUI/src/com/android/systemui/bubbles/BubbleStackView.java
@@ -889,6 +889,7 @@ public class BubbleStackView extends FrameLayout {
mFlyout.removeCallbacks(mHideFlyout);
mFlyout.postDelayed(mHideFlyout, FLYOUT_HIDE_AFTER);
});
+ logBubbleEvent(bubble, StatsLog.BUBBLE_UICHANGED__ACTION__FLYOUT);
}
}
@@ -1060,7 +1061,8 @@ public class BubbleStackView extends FrameLayout {
* @param action the user interaction enum.
*/
private void logBubbleEvent(@Nullable Bubble bubble, int action) {
- if (bubble == null) {
+ if (bubble == null || bubble.entry == null
+ || bubble.entry.notification == null) {
StatsLog.write(StatsLog.BUBBLE_UI_CHANGED,
null /* package name */,
null /* notification channel */,