Add histograms for session startup enum and number of URLs to open.

chrome/browser/prefs/pref_metrics_service.cc

Index: chrome/browser/prefs/pref_metrics_service.cc
diff --git a/chrome/browser/prefs/pref_metrics_service.cc b/chrome/browser/prefs/pref_metrics_service.cc
index ed1d5380e5849a2e828a2951ac73b4e25d494df8..c9154f3aa2183d9368b5f149071099355248765c 100644
--- a/chrome/browser/prefs/pref_metrics_service.cc
+++ b/chrome/browser/prefs/pref_metrics_service.cc
@@ -6,6 +6,7 @@
 
 #include "base/metrics/histogram.h"
 #include "base/prefs/pref_service.h"
+#include "chrome/browser/prefs/session_startup_pref.h"
 #include "chrome/browser/profiles/incognito_helpers.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/common/pref_names.h"
@@ -24,6 +25,17 @@ void PrefMetricsService::RecordLaunchPrefs() {
       profile_->GetPrefs()->GetBoolean(prefs::kShowHomeButton));
   UMA_HISTOGRAM_BOOLEAN("Settings.HomePageIsNewTabPage",
       profile_->GetPrefs()->GetBoolean(prefs::kHomePageIsNewTabPage));
+
+  int restore_on_startup = profile_->GetPrefs()->GetInteger(
+      prefs::kRestoreOnStartup);
+  UMA_HISTOGRAM_ENUMERATION("Settings.StartupPageLoadSettings",
+      restore_on_startup, SessionStartupPref::kPrefValueMax);
+  if (restore_on_startup == SessionStartupPref::kPrefValueURLs) {
+    int url_list_size = profile_->GetPrefs()->GetList(

[Mark P, 2013/07/30 17:56:57]

nit: const int or use the function directly in the UMA_HISTOGRAM call

[meacer, 2013/07/30 18:41:50]

Done.

+        prefs::kURLsToRestoreOnStartup)->GetSize();
+    UMA_HISTOGRAM_COUNTS_100("Settings.StartupPageLoadURLs",

[Mark P, 2013/07/30 17:56:57]

minor comment: this creates a histogram with 50 buckets.  Do you need that many?
 Perhaps you'd be fine with a histogram that maxes out at 50 and has only 20
buckets.

[meacer, 2013/07/30 18:41:50]

Done.

+                             url_list_size);
+  }
 }
 
 // static