Added Protector backup for Preferences.

chrome/browser/extensions/extension_prefs.h

Index: chrome/browser/extensions/extension_prefs.h
diff --git a/chrome/browser/extensions/extension_prefs.h b/chrome/browser/extensions/extension_prefs.h
index 19b79386ce578b17f9d30a1719ae1b9706e01308..7410da3a2824b8e1b2f346393322c2f9596ee763 100644
--- a/chrome/browser/extensions/extension_prefs.h
+++ b/chrome/browser/extensions/extension_prefs.h
@@ -121,6 +121,12 @@ class ExtensionPrefs : public ExtensionContentSettingsStore::Observer,
   // Returns all installed extensions
   void GetExtensions(ExtensionIdSet* out);
 
+  // Returns all installed extensions from |extension_prefs|. This is exposed
+  // for ProtectedPrefsWatcher because it needs access to the extension ID list
+  // before the ExtensionService is initialized.
+  static void GetExtensionsFrom(const base::DictionaryValue* extension_prefs,

[Aaron Boodman, 2012/03/13 19:45:15]

Just return ExtensionIdSet. Most compilers will optimize away the copy, and even
if they don't clarity is more important anyway.

[Ivan Korotkov, 2012/03/14 13:06:45]

Done. Is it ok that there is inconsistency between this and GetExtensions() now?

+                                ExtensionIdSet* out);
+
   // Getter and setter for browser action visibility.
   bool GetBrowserActionVisibility(const Extension* extension);
   void SetBrowserActionVisibility(const Extension* extension, bool visible);