Added PermissionIDSet to APIPermissions

extensions/common/permissions/media_galleries_permission.h

 // Copyright 2014 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 EXTENSIONS_COMMON_PERMISSIONS_MEDIA_GALLERIES_PERMISSION_H_
 #define EXTENSIONS_COMMON_PERMISSIONS_MEDIA_GALLERIES_PERMISSION_H_
 
 #include "base/basictypes.h"
 #include "extensions/common/permissions/api_permission.h"
 #include "extensions/common/permissions/media_galleries_permission_data.h"
@@ skipping 25 matching lines @@
   ~MediaGalleriesPermission() override;
 
   // SetDisjunctionPermission overrides.
   // MediaGalleriesPermission does additional checks to make sure the
   // permissions do not contain unknown values.
   bool FromValue(const base::Value* value,
                  std::string* error,
                  std::vector<std::string>* unhandled_permissions) override;
 
   // APIPermission overrides.
+  PermissionIDSet GetPermissions() const override;
   PermissionMessages GetMessages() const override;
 
   // Permission strings.
   static const char kAllAutoDetectedPermission[];
   static const char kScanPermission[];
   static const char kReadPermission[];
   static const char kCopyToPermission[];
   static const char kDeletePermission[];
+
+ private:
+  // TODO(sashab): Once GetMessages() is deprecated, move this logic back into

[Yoyo Zhou, 2014/12/11 00:54:11]

nit: seems like this TODO should go in the implementation. The implementation
could also be an anonymous namespace function.

[sashab, 2014/12/11 04:15:36]

Done. Also renamed to AddPermissionsToLists() and shortened the parameter names
to match USBDevicePermission.

+  // GetPermissions().
+  void AddMessagesAndPermissions(PermissionIDSet* permission_ids,
+                                 PermissionMessages* permission_messages) const;
 };
 
 }  // namespace extensions
 
 #endif  // EXTENSIONS_COMMON_PERMISSIONS_MEDIA_GALLERIES_PERMISSION_H_