Store Media gallery permissions in extension prefs.

chrome/browser/extensions/extension_prefs.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_EXTENSIONS_EXTENSION_PREFS_H_
 #define CHROME_BROWSER_EXTENSIONS_EXTENSION_PREFS_H_
 
 #include <set>
 #include <string>
 #include <vector>
 
 #include "base/memory/linked_ptr.h"
 #include "base/time.h"
 #include "base/values.h"
 #include "chrome/browser/extensions/api/content_settings/content_settings_store.h"
 #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"
 
 class ExtensionPrefValueMap;
 class ExtensionSorting;
 class PrefService;
 class URLPatternSet;
 
 namespace extensions {
@@ skipping 285 matching lines @@
 
   void SetLaunchType(const std::string& extension_id, LaunchType launch_type);
 
   // Find the right launch container based on the launch type.
   // If |extension|'s prefs do not have a launch type set, then
   // use |default_pref_value|.
   extension_misc::LaunchContainer GetLaunchContainer(
       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.
   // Caller takes ownership of returned structure.
   ExtensionsInfo* GetInstalledExtensionsInfo();
 
   // Returns the ExtensionInfo from the prefs for the given extension. If the
   // extension is not present, NULL is returned.
   ExtensionInfo* GetInstalledExtensionInfo(const std::string& extension_id);
 
@@ skipping 224 matching lines @@
   scoped_ptr<ExtensionSorting> extension_sorting_;
 
   scoped_refptr<ContentSettingsStore> content_settings_store_;
 
   DISALLOW_COPY_AND_ASSIGN(ExtensionPrefs);
 };
 
 }  // namespace extensions
 
 #endif  // CHROME_BROWSER_EXTENSIONS_EXTENSION_PREFS_H_