Add histograms to the rendez-vous process (ProcessSingleton).

chrome/browser/chrome_browser_main.cc

Index: chrome/browser/chrome_browser_main.cc
diff --git a/chrome/browser/chrome_browser_main.cc b/chrome/browser/chrome_browser_main.cc
index c23936ba25af2a4ed92a0086711e7e14a9b440eb..148940205e343b8b41e30393a43b2548c938c7ab 100644
--- a/chrome/browser/chrome_browser_main.cc
+++ b/chrome/browser/chrome_browser_main.cc
@@ -1646,6 +1646,9 @@ int ChromeBrowserMainParts::PreMainMessageLoopRunImpl() {
   // 20 seconds to respond. Note that this needs to be done before we attempt
   // to read the profile.
   notify_result_ = process_singleton_->NotifyOtherProcessOrCreate();
+  UMA_HISTOGRAM_ENUMERATION("Chrome.ProcessSingleton.NotifyResult",
+                            notify_result_,
+                            ProcessSingleton::NUM_NOTIFY_RESULTS);
   switch (notify_result_) {
     case ProcessSingleton::PROCESS_NONE:
       // No process already running, fall through to starting a new one.
@@ -1679,8 +1682,9 @@ int ChromeBrowserMainParts::PreMainMessageLoopRunImpl() {
                     "now to avoid profile corruption.";
       return chrome::RESULT_CODE_PROFILE_IN_USE;
 
-    default:
+    case ProcessSingleton::NUM_NOTIFY_RESULTS:

[bcwhite, 2016/10/07 16:17:44]

Why this change?

[gab, 2016/10/07 16:34:14]

Because it's better to enumerate all cases than to have a default switch as it
allows clang to warn when a switch doesn't handle a new enum value.

[bcwhite, 2016/10/07 16:59:45]

Acknowledged.

       NOTREACHED();
+      break;
   }
 #endif  // !defined(OS_ANDROID)