From 484fd36dfab91abccadf85595f2104323d41a253 Mon Sep 17 00:00:00 2001 From: Jason Monk Date: Wed, 13 Jul 2016 15:24:32 -0400 Subject: [PATCH] Fix QS gear metrics Change-Id: If1eeefa3831010d1567861dd71af6f29833496b3 Fixes: 30117924 --- .../android/systemui/statusbar/phone/QuickStatusBarHeader.java | 3 ++- proto/src/metrics_constants.proto | 3 +++ 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/packages/SystemUI/src/com/android/systemui/statusbar/phone/QuickStatusBarHeader.java b/packages/SystemUI/src/com/android/systemui/statusbar/phone/QuickStatusBarHeader.java index 85303f4229250..ba51619c14a20 100644 --- a/packages/SystemUI/src/com/android/systemui/statusbar/phone/QuickStatusBarHeader.java +++ b/packages/SystemUI/src/com/android/systemui/statusbar/phone/QuickStatusBarHeader.java @@ -315,7 +315,8 @@ public class QuickStatusBarHeader extends BaseStatusBarHeader implements public void onClick(View v) { if (v == mSettingsButton) { MetricsLogger.action(mContext, - MetricsProto.MetricsEvent.ACTION_QS_EXPANDED_SETTINGS_LAUNCH); + mExpanded ? MetricsProto.MetricsEvent.ACTION_QS_EXPANDED_SETTINGS_LAUNCH + : MetricsProto.MetricsEvent.ACTION_QS_COLLAPSED_SETTINGS_LAUNCH); if (mSettingsButton.isTunerClick()) { if (TunerService.isTunerEnabled(mContext)) { TunerService.showResetRequest(mContext, new Runnable() { diff --git a/proto/src/metrics_constants.proto b/proto/src/metrics_constants.proto index 64b30d917200c..130eea042ce69 100644 --- a/proto/src/metrics_constants.proto +++ b/proto/src/metrics_constants.proto @@ -2375,6 +2375,9 @@ message MetricsEvent { // SUBTYPE: false is off, true is on ACTION_TOGGLE_STORAGE_MANAGER = 489; + // Settings launched from collapsed quick settings. + ACTION_QS_COLLAPSED_SETTINGS_LAUNCH = 490; + // ---- End N-MR1 Constants, all N-MR1 constants go above this line ---- // Add new aosp constants above this line. // END OF AOSP CONSTANTS