Refactor preference syncing.

chrome/test/reliability/page_load_test.cc

-// Copyright (c) 2010 The Chromium Authors. All rights reserved.
+// Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 //
 // This file provides reliablity test which runs under UI test framework. The
 // test is intended to run within QEMU environment.
 //
 // Usage 1: reliability_test
 // Upon invocation, it visits a hard coded list of sample URLs. This is mainly
 // used by buildbot, to verify reliability_test itself runs ok.
 //
@@ skipping 620 matching lines @@
     FilePath path = user_data_dir().Append(chrome::kLocalStateFilename);
     return PrefServiceMockBuilder().WithUserFilePrefs(path).Create();
   }
 
   void GetStabilityMetrics(NavigationMetrics* metrics) {
     if (!metrics)
       return;
     scoped_ptr<PrefService> local_state(GetLocalState());
     if (!local_state.get())
       return;
-    local_state->RegisterBooleanPref(prefs::kStabilityExitedCleanly, false);
-    local_state->RegisterIntegerPref(prefs::kStabilityLaunchCount, -1);
-    local_state->RegisterIntegerPref(prefs::kStabilityPageLoadCount, -1);
-    local_state->RegisterIntegerPref(prefs::kStabilityCrashCount, 0);
-    local_state->RegisterIntegerPref(prefs::kStabilityRendererCrashCount, 0);
+    local_state->RegisterBooleanPref(prefs::kStabilityExitedCleanly,
+                                     false,
+                                     false /* don't sync pref */);
+    local_state->RegisterIntegerPref(prefs::kStabilityLaunchCount,
+                                     -1,
+                                     false /* don't sync pref */);
+    local_state->RegisterIntegerPref(prefs::kStabilityPageLoadCount,
+                                     -1,
+                                     false /* don't sync pref */);
+    local_state->RegisterIntegerPref(prefs::kStabilityCrashCount,
+                                     0,
+                                     false /* don't sync pref */);
+    local_state->RegisterIntegerPref(prefs::kStabilityRendererCrashCount,
+                                     0,
+                                     false /* don't sync pref */);
 
     metrics->browser_clean_exit =
         local_state->GetBoolean(prefs::kStabilityExitedCleanly);
     metrics->browser_launch_count =
         local_state->GetInteger(prefs::kStabilityLaunchCount);
     metrics->page_load_count =
         local_state->GetInteger(prefs::kStabilityPageLoadCount);
     metrics->browser_crash_count =
         local_state->GetInteger(prefs::kStabilityCrashCount);
     metrics->renderer_crash_count =
@@ skipping 163 matching lines @@
     }
   }
 
   if (parsed_command_line.HasSwitch(kStressOptSwitch)) {
     g_stress_opt = true;
   }
   if (parsed_command_line.HasSwitch(kStressDeoptSwitch)) {
     g_stress_deopt = true;
   }
 }