Add Tron Logging to Keyboard Shortcuts Helper

Bug: 28161718
Change-Id: I5dcbb4c9f26d133e695595168ac2cd0ad1fa68d7

diff --git a/packages/SystemUI/src/com/android/systemui/statusbar/KeyboardShortcuts.java b/packages/SystemUI/src/com/android/systemui/statusbar/KeyboardShortcuts.java
index 79e06c6..0e1441e 100644 (file)
--- a/packages/SystemUI/src/com/android/systemui/statusbar/KeyboardShortcuts.java
+++ b/packages/SystemUI/src/com/android/systemui/statusbar/KeyboardShortcuts.java
@@ -54,6 +54,8 @@ import android.widget.RelativeLayout;
 import android.widget.TextView;
 
 import com.android.internal.app.AssistUtils;
+import com.android.internal.logging.MetricsLogger;
+import com.android.internal.logging.MetricsProto;
 import com.android.systemui.R;
 import com.android.systemui.recents.Recents;
 
@@ -126,6 +128,8 @@ public final class KeyboardShortcuts {
     }
 
     public static void show(Context context, int deviceId) {
+        MetricsLogger.visible(context,
+                MetricsProto.MetricsEvent.KEYBOARD_SHORTCUTS_HELPER);
         synchronized (sLock) {
             if (sInstance != null && !sInstance.mContext.equals(context)) {
                 dismiss();
@@ -146,6 +150,8 @@ public final class KeyboardShortcuts {
     }
 
     public static void dismiss() {
+        MetricsLogger.hidden(sInstance.mContext,
+                MetricsProto.MetricsEvent.KEYBOARD_SHORTCUTS_HELPER);
         synchronized (sLock) {
             if (sInstance != null) {
                 sInstance.dismissKeyboardShortcuts();

diff --git a/proto/src/metrics_constants.proto b/proto/src/metrics_constants.proto
index 1c12565..cda3892 100644 (file)
--- a/proto/src/metrics_constants.proto
+++ b/proto/src/metrics_constants.proto
@@ -2239,6 +2239,10 @@ message MetricsEvent {
     // CATEGORY: SETTINGS
     STORAGE_MANAGER_SETTINGS = 458;
 
+    // ------- Begin N Keyboard Shortcuts Helper -----
+    // Keyboard Shortcuts Helper is opened/closed.
+    KEYBOARD_SHORTCUTS_HELPER = 459;
+
     // Add new aosp constants above this line.
     // END OF AOSP CONSTANTS
   }