Added PermissionIDSet to APIPermissions

extensions/common/permissions/coalesced_permission_message.h

Index: extensions/common/permissions/coalesced_permission_message.h
diff --git a/extensions/common/permissions/coalesced_permission_message.h b/extensions/common/permissions/coalesced_permission_message.h
index 7ff24c0aff8bf78e7dbf52742d19ac459457c75a..e7e61814fb7ce1dfd206e6b5440d36aefd1d25f7 100644
--- a/extensions/common/permissions/coalesced_permission_message.h
+++ b/extensions/common/permissions/coalesced_permission_message.h
@@ -5,11 +5,9 @@
 #ifndef EXTENSIONS_COMMON_PERMISSIONS_COALESCED_PERMISSION_MESSAGE_H_
 #define EXTENSIONS_COMMON_PERMISSIONS_COALESCED_PERMISSION_MESSAGE_H_
 
-#include <set>
+#include <list>
 #include <string>
 
-#include "base/memory/scoped_vector.h"
-#include "extensions/common/permissions/api_permission.h"
 #include "extensions/common/permissions/api_permission_set.h"
 
 namespace extensions {
@@ -42,6 +40,7 @@ namespace extensions {
 // this to PermissionMessage.
 class CoalescedPermissionMessage {
  public:
+  CoalescedPermissionMessage();

[sashab, 2014/12/10 08:57:49]

Not sure if we should include this... Technically, there should be no such thing
as an empty permission message. Will remove in a follow-up patch.

   CoalescedPermissionMessage(const base::string16& message,
                              const PermissionIDSet& permissions);
   CoalescedPermissionMessage(const base::string16& message,
@@ -59,11 +58,11 @@ class CoalescedPermissionMessage {
   const base::string16 message_;
   const PermissionIDSet permissions_;
   const std::vector<base::string16> submessages_;
-
-  DISALLOW_COPY_AND_ASSIGN(CoalescedPermissionMessage);
 };
 
-typedef std::vector<CoalescedPermissionMessage> CoalescedPermissionMessages;
+// Use a linked list to store our list of messages, since we will commonly be
+// iterating/removing elements but should never be accessing by index.
+typedef std::list<CoalescedPermissionMessage> CoalescedPermissionMessages;
 
 }  // namespace extensions