Change PrefService.ReadError histogram to sample success case.

chrome/browser/prefs/chrome_pref_service_factory.cc

Index: chrome/browser/prefs/chrome_pref_service_factory.cc
diff --git a/chrome/browser/prefs/chrome_pref_service_factory.cc b/chrome/browser/prefs/chrome_pref_service_factory.cc
index 0f5c90b6386a3f00afbfaaa9427fa5d30189caab..699a0d13c08185b99bd714235791ccce7cf396c1 100644
--- a/chrome/browser/prefs/chrome_pref_service_factory.cc
+++ b/chrome/browser/prefs/chrome_pref_service_factory.cc
@@ -39,6 +39,11 @@ namespace {
 
 // Shows notifications which correspond to PersistentPrefStore's reading errors.
 void HandleReadError(PersistentPrefStore::PrefReadError error) {
+  // Sample the histogram also for the successful case in order to get a
+  // baseline on the success rate in addition to the error distribution.
+  UMA_HISTOGRAM_ENUMERATION("PrefService.ReadError", error,
+                            PersistentPrefStore::PREF_READ_ERROR_MAX_ENUM);
+
   if (error != PersistentPrefStore::PREF_READ_ERROR_NONE) {
 #if !defined(OS_CHROMEOS)
     // Failing to load prefs on startup is a bad thing(TM). See bug 38352 for
@@ -59,8 +64,6 @@ void HandleReadError(PersistentPrefStore::PrefReadError error) {
     // On ChromeOS error screen with message about broken local state
     // will be displayed.
 #endif
-    UMA_HISTOGRAM_ENUMERATION("PrefService.ReadError", error,
-                              PersistentPrefStore::PREF_READ_ERROR_MAX_ENUM);
   }
 }