Add metrics to the new ProfileChooser Tutorials

chrome/browser/profiles/profile_metrics.cc

 // Copyright (c) 2011 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.
 
 #include "chrome/browser/profiles/profile_metrics.h"
 
 #include "base/files/file_path.h"
 #include "base/logging.h"
 #include "base/metrics/histogram.h"
 #include "chrome/browser/browser_process.h"
@@ skipping 347 matching lines @@
       UMA_HISTOGRAM_ENUMERATION("Profile.DesktopMenu.GAIADefault", metric,
                                 NUM_PROFILE_DESKTOP_MENU_METRICS);
       break;
   }
 }
 
 void ProfileMetrics::LogProfileDelete(bool profile_was_signed_in) {
   UMA_HISTOGRAM_BOOLEAN("Profile.Delete", profile_was_signed_in);
 }
 
+void ProfileMetrics::LogProfileDesktopSignin(ProfileDesktopSignin metric) {
+  DCHECK(metric < NUM_PROFILE_DESKTOP_SIGNIN_METRICS);

[Alexei Svitkine (slow), 2014/08/14 17:59:46]

Nit: Can this be DCHECK_LT? Same below.

[Mike Lerman, 2014/08/14 18:15:30]

Didn't know that was an option. Done, thanks!

+  UMA_HISTOGRAM_ENUMERATION("Profile.NewAvatarMenu.Signin", metric,
+                            NUM_PROFILE_DESKTOP_SIGNIN_METRICS);
+}
+
+void ProfileMetrics::LogProfileNewAvatarMenuFRE(
+    ProfileNewAvatarMenuFRE metric) {
+  DCHECK(metric < NUM_PROFILE_AVATAR_MENU_UPGRADE_METRICS);
+  UMA_HISTOGRAM_ENUMERATION("Profile.NewAvatarMenu.FirstRun", metric,
+                            NUM_PROFILE_AVATAR_MENU_UPGRADE_METRICS);
+}
+
+void ProfileMetrics::LogProfileNewAvatarMenuNotYou(
+    ProfileNewAvatarMenuNotYou metric) {
+  DCHECK(metric < NUM_PROFILE_AVATAR_MENU_NOT_YOU_METRICS);
+  UMA_HISTOGRAM_ENUMERATION("Profile.NewAvatarMenu.NotYou", metric,
+                            NUM_PROFILE_AVATAR_MENU_NOT_YOU_METRICS);
+}
+
 #if defined(OS_ANDROID)
 void ProfileMetrics::LogProfileAndroidAccountManagementMenu(
     ProfileAndroidAccountManagementMenu metric,
     signin::GAIAServiceType gaia_service) {
   // The first parameter to the histogram needs to be literal, because of the
   // optimized implementation of |UMA_HISTOGRAM_ENUMERATION|. Do not attempt
   // to refactor.
   switch (gaia_service) {
     case signin::GAIA_SERVICE_TYPE_NONE:
       UMA_HISTOGRAM_ENUMERATION(
@@ skipping 51 matching lines @@
   UMA_HISTOGRAM_ENUMERATION("Profile.SyncSignIn",
                             GetProfileType(profile_path),
                             NUM_PROFILE_TYPE_METRICS);
 }
 
 void ProfileMetrics::LogProfileUpdate(const base::FilePath& profile_path) {
   UMA_HISTOGRAM_ENUMERATION("Profile.Update",
                             GetProfileType(profile_path),
                             NUM_PROFILE_TYPE_METRICS);
 }