[Mac] UMA Metrics for the Default Touch Bar

chrome/browser/ui/cocoa/browser_window_touch_bar.mm

 // Copyright 2017 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #import "chrome/browser/ui/cocoa/browser_window_touch_bar.h"
 
 #include <memory>
 
 #include "base/mac/mac_util.h"
 #import "base/mac/scoped_nsobject.h"
 #import "base/mac/sdk_forward_declarations.h"
+#include "base/metrics/histogram_macros.h"
 #include "base/strings/sys_string_conversions.h"
 #include "chrome/app/chrome_command_ids.h"
 #include "chrome/app/vector_icons/vector_icons.h"
 #include "chrome/browser/command_updater.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/search_engines/template_url_service_factory.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/browser_command_controller.h"
 #import "chrome/browser/ui/cocoa/browser_window_controller.h"
 #include "chrome/common/chrome_features.h"
 #include "chrome/common/pref_names.h"
 #include "chrome/grit/generated_resources.h"
 #include "components/omnibox/browser/vector_icons.h"
 #include "components/prefs/pref_member.h"
 #include "components/search_engines/util.h"
 #include "components/toolbar/vector_icons.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/gfx/color_palette.h"
 #include "ui/gfx/color_utils.h"
 #include "ui/gfx/image/image.h"
 #include "ui/gfx/image/image_skia_util_mac.h"
 #include "ui/gfx/paint_vector_icon.h"
 #include "ui/vector_icons/vector_icons.h"
 
 namespace {
 
+// The touch bar actions that are being recorded in a histogram. These values
+// should not be re-ordered or removed.
+enum TouchBarAction {
+  BACK = 0,
+  FORWARD,
+  STOP,
+  RELOAD,
+  HOME,
+  SEARCH,
+  STAR,
+  NEW_TAB,
+  TOUCH_BAR_ACTION_COUNT
+};
+
 // The touch bar's identifier.
 const NSTouchBarCustomizationIdentifier kBrowserWindowTouchBarId =
     @"BrowserWindowTouchBarId";
 
 // Touch bar items identifiers.
 const NSTouchBarItemIdentifier kBackForwardTouchId = @"BackForwardTouchId";
 const NSTouchBarItemIdentifier kReloadOrStopTouchId = @"ReloadOrStopTouchId";
 const NSTouchBarItemIdentifier kHomeTouchId = @"HomeTouchId";
 const NSTouchBarItemIdentifier kSearchTouchId = @"SearchTouchId";
 const NSTouchBarItemIdentifier kStarTouchId = @"StarTouchId";
@@ skipping 28 matching lines @@
                                int command,
                                SkColor color = kTouchBarDefaultIconColor) {
   NSButton* button =
       [NSButton buttonWithImage:CreateNSImageFromIcon(icon, color)
                          target:owner
                          action:@selector(executeCommand:)];
   button.tag = command;
   return button;
 }
 
+// Logs the sample's UMA metrics into the DefaultTouchBar.Metrics histogram.
+void LogTouchBarUMA(int command) {
+  TouchBarAction action;
+  if (command == IDC_BACK)

[Robert Sesek, 2017/03/13 21:05:25]

Would this be better as a switch?

[spqchan, 2017/03/14 00:04:33]

Done.

+    action = TouchBarAction::BACK;
+  else if (command == IDC_FORWARD)
+    action = TouchBarAction::FORWARD;
+  else if (command == IDC_STOP)
+    action = TouchBarAction::STOP;
+  else if (command == IDC_RELOAD)
+    action = TouchBarAction::RELOAD;
+  else if (command == IDC_HOME)
+    action = TouchBarAction::HOME;
+  else if (command == IDC_FOCUS_LOCATION)
+    action = TouchBarAction::SEARCH;
+  else if (command == IDC_BOOKMARK_PAGE)
+    action = TouchBarAction::STAR;
+  else if (command == IDC_NEW_TAB)
+    action = TouchBarAction::NEW_TAB;
+  else
+    action = TouchBarAction::TOUCH_BAR_ACTION_COUNT;
+
+  DCHECK_NE(action, TouchBarAction::TOUCH_BAR_ACTION_COUNT);
+  UMA_HISTOGRAM_ENUMERATION("TouchBar.Default.Metrics", action,
+                            TOUCH_BAR_ACTION_COUNT);
+}
+
 // A class registered for C++ notifications. This is used to detect changes in
 // the home button preferences and update the Touch Bar.
 class HomePrefNotificationBridge {
  public:
   explicit HomePrefNotificationBridge(BrowserWindowController* bwc)
       : bwc_(bwc) {}
 
   ~HomePrefNotificationBridge() {}
 
   void UpdateTouchBar() { [bwc_ invalidateTouchBar]; }
@@ skipping 165 matching lines @@
   searchButton.imageHugsTitle = YES;
   searchButton.tag = IDC_FOCUS_LOCATION;
   int width = showHomeButton_.GetValue() ? kSearchBtnWidthWithHomeBtn
                                          : kSearchBtnWidthWithoutHomeBtn;
   [searchButton.widthAnchor constraintEqualToConstant:width].active = YES;
   return searchButton;
 }
 
 - (void)backOrForward:(id)sender {
   NSSegmentedControl* control = sender;
-  if ([control selectedSegment] == kBackSegmentIndex)
-    commandUpdater_->ExecuteCommand(IDC_BACK);
-  else
-    commandUpdater_->ExecuteCommand(IDC_FORWARD);
+  int command =
+      [control selectedSegment] == kBackSegmentIndex ? IDC_BACK : IDC_FORWARD;
+  LogTouchBarUMA(command);
+  commandUpdater_->ExecuteCommand(command);
 }
 
 - (void)executeCommand:(id)sender {
-  commandUpdater_->ExecuteCommand([sender tag]);
+  int command = [sender tag];
+  LogTouchBarUMA(command);
+  commandUpdater_->ExecuteCommand(command);
 }
 
 @end