Added PermissionIDSet to APIPermissions

extensions/common/permissions/usb_device_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_USB_DEVICE_PERMISSION_H_
 #define EXTENSIONS_COMMON_PERMISSIONS_USB_DEVICE_PERMISSION_H_
 
 #include "base/basictypes.h"
 #include "extensions/common/permissions/api_permission.h"
 #include "extensions/common/permissions/set_disjunction_permission.h"
@@ skipping 12 matching lines @@
         interface_id(interface_id) {}
     const uint16 vendor_id;
     const uint16 product_id;
     const int interface_id;
   };
 
   explicit UsbDevicePermission(const APIPermissionInfo* info);
   ~UsbDevicePermission() override;
 
   // APIPermission overrides
+  PermissionIDSet GetPermissions() const override;
   PermissionMessages GetMessages() const override;
+
+ private:
+  // TODO(sashab): Once GetMessages() is deprecated, remove this function and
+  // move all logic back into GetPermissions().
+  void AddPermissionsToLists(PermissionIDSet& ids,

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

Similarly, this could be an anonymous namespace function and not appear in the
header.

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

Done. Also made it slightly more efficient by taking pointers instead of
references, so the callsites can pass NULL if they don't want a particular type
of message, like in MediaGalleriesPermission.

+                             PermissionMessages& messages) const;
 };
 
 }  // namespace extensions
 
 #endif  // EXTENSIONS_COMMON_PERMISSIONS_USB_DEVICE_PERMISSION_H_