Introduce PrefRegistrySyncable, simplifying PrefServiceSyncable.

chrome/browser/bookmarks/bookmark_model_factory.h

Index: chrome/browser/bookmarks/bookmark_model_factory.h
diff --git a/chrome/browser/bookmarks/bookmark_model_factory.h b/chrome/browser/bookmarks/bookmark_model_factory.h
index 5ad84d7725f770b7ed771bfe9c4b4741d98a551e..f38bcb57675b242109bd4d3607964be8392cc233 100644
--- a/chrome/browser/bookmarks/bookmark_model_factory.h
+++ b/chrome/browser/bookmarks/bookmark_model_factory.h
@@ -11,7 +11,7 @@
 
 template <typename T> struct DefaultSingletonTraits;
 
-class PrefServiceSyncable;
+class PrefRegistrySyncable;

[tfarina, 2013/02/09 17:06:24]

you should not need this forward declaration since it's part of the public API
of ProfileKeyedServiceFactory

 class Profile;
 class BookmarkModel;
 
@@ -34,7 +34,7 @@ class BookmarkModelFactory : public ProfileKeyedServiceFactory {
   // ProfileKeyedServiceFactory:
   virtual ProfileKeyedService* BuildServiceInstanceFor(
       Profile* profile) const OVERRIDE;
-  virtual void RegisterUserPrefs(PrefServiceSyncable* user_prefs) OVERRIDE;
+  virtual void RegisterUserPrefs(PrefRegistrySyncable* registry) OVERRIDE;
   virtual bool ServiceRedirectedInIncognito() const OVERRIDE;
   virtual bool ServiceIsNULLWhileTesting() const OVERRIDE;