Store Media gallery permissions in extension prefs.

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 70565fa95f8822c8b3f26ac17c9ad31b6136cfd5..cf630dc154ea542f470340f5425e5d07eecfef97 100644
--- a/chrome/browser/extensions/extension_prefs.h
+++ b/chrome/browser/extensions/extension_prefs.h
@@ -16,6 +16,7 @@
 #include "chrome/browser/extensions/extension_prefs_scope.h"
 #include "chrome/browser/extensions/extension_scoped_prefs.h"
 #include "chrome/browser/extensions/management_policy.h"
+#include "chrome/browser/media_gallery/media_galleries_preferences.h"
 #include "chrome/common/extensions/extension.h"
 #include "chrome/common/extensions/url_pattern_set.h"
 #include "chrome/common/string_ordinal.h"
@@ -321,6 +322,14 @@ class ExtensionPrefs : public ContentSettingsStore::Observer,
       const Extension* extension,
       LaunchType default_pref_value);
 
+  // Set and retrieve permissions for media galleries as identified by the
+  // gallery id.
+  void SetMediaGalleryPermission(const std::string& extension_id,
+                                 MediaGalleryPrefId gallery, bool has_access);

[Evan Stade, 2012/07/31 20:29:47]

The object should be a const&, and I think it makes sense to pass a
MediaGalleryPermission struct (although only the id would be used).

[vandebo (ex-Chrome), 2012/08/01 01:01:10]

The extension prefs don't need to know any of the gallery metadata, I think
passing just the id is the right abstraction here.

[Evan Stade, 2012/08/01 04:29:37]

yea, you're right. I failed to look at what file this was.

+  std::vector<MediaGalleryPermission> GetMediaGalleryPermissions(
+          const std::string& extension_id);

[Evan Stade, 2012/07/31 20:29:47]

nit: too much indent

[vandebo (ex-Chrome), 2012/08/01 18:28:25]

Oops, need to fix my home vim settings.  Fixed.

+  void RemoveMediaGalleryPermissions(MediaGalleryPrefId gallery_id);
+
   // Saves ExtensionInfo for each installed extension with the path to the
   // version directory and the location. Blacklisted extensions won't be saved
   // and neither will external extensions the user has explicitly uninstalled.