Added PermissionIDSet to APIPermissions

extensions/common/permissions/usb_device_permission.h

Index: extensions/common/permissions/usb_device_permission.h
diff --git a/extensions/common/permissions/usb_device_permission.h b/extensions/common/permissions/usb_device_permission.h
index cb19f3e937bb81621ea19cdea71f00f8bb2458dc..0a04ec5d7523f8595592afe292f28d5dfe6c8660 100644
--- a/extensions/common/permissions/usb_device_permission.h
+++ b/extensions/common/permissions/usb_device_permission.h
@@ -30,7 +30,14 @@ class UsbDevicePermission
   ~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