From 8f3e66f2fe88550f0dd38d5e225062cc68edc518 Mon Sep 17 00:00:00 2001 From: Julia Reynolds Date: Thu, 12 May 2016 10:33:47 -0400 Subject: [PATCH] Logging for QS settings launch. Change-Id: Iff8bbb448454c8a82196d292a285827846b28357 Fixes: 28678032 --- .../systemui/statusbar/phone/QuickStatusBarHeader.java | 5 +++++ proto/src/metrics_constants.proto | 3 +++ 2 files changed, 8 insertions(+) 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 d6deba06d4dae..8bb1f24983fcd 100644 --- a/packages/SystemUI/src/com/android/systemui/statusbar/phone/QuickStatusBarHeader.java +++ b/packages/SystemUI/src/com/android/systemui/statusbar/phone/QuickStatusBarHeader.java @@ -30,6 +30,9 @@ import android.view.ViewGroup; import android.widget.ImageView; import android.widget.TextView; import android.widget.Toast; + +import com.android.internal.logging.MetricsLogger; +import com.android.internal.logging.MetricsProto; import com.android.keyguard.KeyguardStatusView; import com.android.systemui.FontSizeUtils; import com.android.systemui.R; @@ -340,6 +343,8 @@ public class QuickStatusBarHeader extends BaseStatusBarHeader implements @Override public void onClick(View v) { if (v == mSettingsButton) { + MetricsLogger.action(mContext, + MetricsProto.MetricsEvent.ACTION_QS_EXPANDED_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 b7677c65451dc..46da957ef90fd 100644 --- a/proto/src/metrics_constants.proto +++ b/proto/src/metrics_constants.proto @@ -2203,6 +2203,9 @@ message MetricsEvent { // The dialog shown by 3P intent to change current webview implementation. WEBVIEW_IMPLEMENTATION = 405; + // Settings launched from expanded quick settings. + ACTION_QS_EXPANDED_SETTINGS_LAUNCH = 406; + // Add new aosp constants above this line. // END OF AOSP CONSTANTS }