histograms: Retire MultiProfile.SigninUserUIPath

Fixed: 1105085
Change-Id: Id39e2ecdf4f2196c78c84b85bf6c0ab7ff051f0b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2299345Reviewed-by: Ahmed Fakhry <afakhry@chromium.org>
Reviewed-by: Brian White <bcwhite@chromium.org>
Commit-Queue: Brian White <bcwhite@chromium.org>
Auto-Submit: Roman Sorokin [CET] <rsorokin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#789243}

ash/multi_profile_uma.cc

@@ -8,12 +8,6 @@
 
 namespace ash {
 
-// static
-void MultiProfileUMA::RecordSigninUser(SigninUserAction action) {
-  UMA_HISTOGRAM_ENUMERATION("MultiProfile.SigninUserUIPath", action,
-                            NUM_SIGNIN_USER_ACTIONS);
-}
-
 // static
 void MultiProfileUMA::RecordSwitchActiveUser(SwitchActiveUserAction action) {
   UMA_HISTOGRAM_ENUMERATION("MultiProfile.SwitchActiveUserUIPath", action,

ash/multi_profile_uma.h

@@ -15,13 +15,6 @@ namespace ash {
 // browser frame that is recorded by the "Profile.OpenMethod" metric.
 class ASH_EXPORT MultiProfileUMA {
  public:
-  // Used for UMA metrics. Do not reorder.
-  enum SigninUserAction {
-    SIGNIN_USER_BY_TRAY = 0,
-    SIGNIN_USER_BY_BROWSER_FRAME,  // Deprecated.
-    NUM_SIGNIN_USER_ACTIONS
-  };
-
   // Used for UMA metrics. Do not reorder.
   enum SwitchActiveUserAction {
     SWITCH_ACTIVE_USER_BY_TRAY = 0,
@@ -29,9 +22,6 @@ class ASH_EXPORT MultiProfileUMA {
     NUM_SWITCH_ACTIVE_USER_ACTIONS
   };
 
-  // Record signing in a new user and what UI path was taken.
-  static void RecordSigninUser(SigninUserAction action);
-
   // Record switching the active user and what UI path was taken.
   static void RecordSwitchActiveUser(SwitchActiveUserAction action);
 

ash/system/unified/user_chooser_detailed_view_controller.cc

@@ -65,7 +65,6 @@ void UserChooserDetailedViewController::HandleUserSwitch(int user_index) {
 }
 
 void UserChooserDetailedViewController::HandleAddUserAction() {
-  MultiProfileUMA::RecordSigninUser(MultiProfileUMA::SIGNIN_USER_BY_TRAY);
   tray_controller_->CloseBubble();
   Shell::Get()->session_controller()->ShowMultiProfileLogin();
   // ShowMultiProfileLogin may delete us.

tools/metrics/histograms/histograms.xml

@@ -89082,6 +89082,9 @@ reviews. Googlers can read more about this at go/gwsq-gerrit.
 
 <histogram name="MultiProfile.SigninUserUIPath"
     enum="MultiProfileSigninUserAction" expires_after="2020-08-23">
+  <obsolete>
+    Retired in M86.
+  </obsolete>
   <owner>skuhne@chromium.org</owner>
   <summary>
     Count the number of times each UI path is taken for signing into a new