Remove PrefServiceSimple, replacing it with PrefService and PrefRegistrySimple.

chrome/browser/prefs/chrome_pref_service_factory.h

Index: chrome/browser/prefs/chrome_pref_service_factory.h
diff --git a/chrome/browser/prefs/chrome_pref_service_factory.h b/chrome/browser/prefs/chrome_pref_service_factory.h
index 84e5a637400ef75d6be8aaccf3939d91d9212a69..61ec8b47e2b02cc37c20e6773d24f6a0394ba649 100644
--- a/chrome/browser/prefs/chrome_pref_service_factory.h
+++ b/chrome/browser/prefs/chrome_pref_service_factory.h
@@ -5,6 +5,8 @@
 #ifndef CHROME_BROWSER_PREFS_CHROME_PREF_SERVICE_FACTORY_H_
 #define CHROME_BROWSER_PREFS_CHROME_PREF_SERVICE_FACTORY_H_
 
+#include "base/memory/scoped_ptr.h"
+
 namespace base {
 class SequencedTaskRunner;
 }
@@ -14,7 +16,8 @@ class PolicyService;
 }
 
 class FilePath;
-class PrefServiceSimple;
+class PrefRegistry;
+class PrefService;
 class PrefServiceSyncable;
 class PrefStore;
 
@@ -28,17 +31,21 @@ namespace chrome_prefs {
 // preferences and may be NULL.
 // |policy_service| is used as the source for mandatory or recommended
 // policies.
+// |pref_registry| is the registry object to retrieve registration

[Mattias Nissler (ping if slow), 2013/01/29 18:10:45]

This sentence is a bit cloudy, can we just say that |pref_registry| keeps the
list of registered prefs and their default values?

[Jói, 2013/01/30 14:23:33]

Done.

+// information from; PrefService takes ownership (not used for

[Mattias Nissler (ping if slow), 2013/01/29 18:10:45]

PrefRegistry is RefCounted vs. ownership transfer

[Jói, 2013/01/30 14:23:33]

Done.

+// CreateProfilePrefs yet).
 // If |async| is true, asynchronous version is used.
 // Notifies using PREF_INITIALIZATION_COMPLETED in the end. Details is set to
 // the created PrefService or NULL if creation has failed. Note, it is
 // guaranteed that in asynchronous version initialization happens after this
 // function returned.
 
-PrefServiceSimple* CreateLocalState(
+PrefService* CreateLocalState(
     const FilePath& pref_filename,
     base::SequencedTaskRunner* pref_io_task_runner,
     policy::PolicyService* policy_service,
     PrefStore* extension_prefs,
+    PrefRegistry* pref_registry,

[Mattias Nissler (ping if slow), 2013/01/29 18:10:45]

Isn't this now refcounted? I think the rule is to pass the pointer wrapped in a
scoped_refptr then.

[Jói, 2013/01/30 14:23:33]

Done.

     bool async);
 
 PrefServiceSyncable* CreateProfilePrefs(