Add extension permissions for Media Gallery API.

chrome/common/extensions/permissions/api_permission.cc

Index: chrome/common/extensions/permissions/api_permission.cc
diff --git a/chrome/common/extensions/permissions/api_permission.cc b/chrome/common/extensions/permissions/api_permission.cc
index 03b141a9fc950dce67c86335e875dc36b7ff8b47..cdfb49a231d6a98bf6c2f82fbb697385e0b955fd 100644
--- a/chrome/common/extensions/permissions/api_permission.cc
+++ b/chrome/common/extensions/permissions/api_permission.cc
@@ -99,6 +99,8 @@ void APIPermission::RegisterAllPermissions(
     { kManagement, "management", kFlagNone,
       IDS_EXTENSION_PROMPT_WARNING_MANAGEMENT,
       PermissionMessage::kManagement },
+    { kMediaGalleries, "mediaGalleries" },
+    { kMediaGalleriesRead, "mediaGalleriesRead" },

[Matt Perry, 2012/07/24 00:21:07]

Does the read permission have no warning message?

[vandebo (ex-Chrome), 2012/07/24 00:32:57]

No - By itself it only conveys permission to request permission for galleries at
runtime (instead of install time).  (aa@ et al are aware of this concept and
signed off on it.)

     { kPageCapture, "pageCapture", kFlagNone,
       IDS_EXTENSION_PROMPT_WARNING_ALL_PAGES_CONTENT,
       PermissionMessage::kAllPageContent },
@@ -167,6 +169,10 @@ void APIPermission::RegisterAllPermissions(
     { kFileSystemWrite, "fileSystemWrite", kFlagNone,
       IDS_EXTENSION_PROMPT_WARNING_FILE_SYSTEM_WRITE,
       PermissionMessage::kFileSystemWrite },
+    { kMediaGalleriesAccessEverything, "mediaGalleriesAccessEverything",

[Matt Perry, 2012/07/24 00:21:07]

I don't see this actually used in this CL. What does it do? The name is a bit
confusing. Does it just combine read/write?

[vandebo (ex-Chrome), 2012/07/24 00:32:57]

Sorry, working on another CL to hook this up.  In the long run this will
probably remain a platform app only permission (whereas read, write, append will
be available for regular extensions). This permission will grant access to all
of the users media galleries without further prompting.  It will be
read/write/append access depending on the the other media gallery permissions.

Commentary: I'd be happy to add comments in the code explaining how/what these
permissions do, but it doesn't seem to be the norm.

[Matt Perry, 2012/07/24 00:51:20]

maybe mediaGalleriesAllGalleries then? Agreed that adding a comment here is
unusual.

[vandebo (ex-Chrome), 2012/07/24 01:07:59]

Done.

+      kFlagCannotBeOptional,
+      IDS_EXTENSION_PROMPT_WARNING_MEDIA_GALLERIES_ACCESS_EVERYTHING,
+      PermissionMessage::kMediaGalleriesAccessEverything },
   };
 
   for (size_t i = 0; i < ARRAYSIZE_UNSAFE(PermissionsToRegister); ++i) {