Split PrefService into PrefService, PrefServiceSimple and PrefServiceSyncable.

chrome/browser/component_updater/recovery_component_installer.cc

Index: chrome/browser/component_updater/recovery_component_installer.cc
diff --git a/chrome/browser/component_updater/recovery_component_installer.cc b/chrome/browser/component_updater/recovery_component_installer.cc
index 1c2d61b3c02f5fc8a04ae83d3bdd506df0b7a68b..49084e299ba264ff78180b85dc94193daae5d7ea 100644
--- a/chrome/browser/component_updater/recovery_component_installer.cc
+++ b/chrome/browser/component_updater/recovery_component_installer.cc
@@ -60,7 +60,8 @@ class RecoveryComponentInstaller : public ComponentInstaller {
   PrefService* prefs_;
 };
 
-void RecoveryRegisterHelper(ComponentUpdateService* cus, PrefService* prefs) {
+void RecoveryRegisterHelper(ComponentUpdateService* cus,
+                            PrefServiceSimple* prefs) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
   prefs->RegisterStringPref(prefs::kRecoveryComponentVersion, "0.0.0.0");

[Mattias Nissler (ping if slow), 2012/12/19 19:50:22]

random note: This shouldn't be registered here but handled via
browser_prefs::RegisterLocalState()

[Jói, 2012/12/20 16:30:31]

Agreed. Added a TODO to fix it.

   Version version(prefs->GetString(prefs::kRecoveryComponentVersion));
@@ -130,7 +131,7 @@ bool RecoveryComponentInstaller::Install(base::DictionaryValue* manifest,
 }
 
 void RegisterRecoveryComponent(ComponentUpdateService* cus,
-                               PrefService* prefs) {
+                               PrefServiceSimple* prefs) {
 #if !defined(OS_CHROMEOS)
   // We delay execute the registration because we are not required in
   // the critical path during browser startup.