Adding Histograms for new Desktop First Run.

chrome/browser/ui/webui/welcome_handler.h

Index: chrome/browser/ui/webui/welcome_handler.h
diff --git a/chrome/browser/ui/webui/welcome_handler.h b/chrome/browser/ui/webui/welcome_handler.h
index 55f186171a2638af757168643d38cdf1ea37bc75..d96f4f3946dd0bedc8bf201f223af73a2d304e4e 100644
--- a/chrome/browser/ui/webui/welcome_handler.h
+++ b/chrome/browser/ui/webui/welcome_handler.h
@@ -28,9 +28,13 @@ class WelcomeHandler : public content::WebUIMessageHandler,
 
  private:
   enum WelcomeResult {
-    DEFAULT = 0,
-    SIGNED_IN,  // User clicked the "Sign In" button and completed sign-in.
-    DECLINED    // User clicked the "No Thanks" button.
+    DEFAULT = 0,    // User navigated away from the page.

[rkaplow, 2016/10/21 15:06:05]

small nit - might make more sense for this to NAVIGATED_AWAY?

Up to you

[tmartino, 2016/10/21 18:50:57]

IMO DEFAULT makes the most sense. It's only really a proxy for "user navigated
away", and it represents a default in the sense that neither of the other two
events that cause a departure happened before destruct-time.

+    DECLINED = 1,   // User clicked the "No Thanks" button.
+    SIGNED_IN = 2,  // User clicked the "Sign In" button and completed sign-in.
+
+    // New results must be added before this line, and should correspond to
+    // values in tools/metrics/histograms/histograms.xml.
+    WELCOME_RESULT_MAX
   };
 
   void HandleActivateSignIn(const base::ListValue* args);