Extensions: Switch to new permission message system, part II

extensions/common/permissions/api_permission_set.h

Index: extensions/common/permissions/api_permission_set.h
diff --git a/extensions/common/permissions/api_permission_set.h b/extensions/common/permissions/api_permission_set.h
index fb8bde9b386eb6c12ff2ed3d667fbcf96fbf36b6..5b21db1d9a001093eb40e72b948fc2a622d14133 100644
--- a/extensions/common/permissions/api_permission_set.h
+++ b/extensions/common/permissions/api_permission_set.h
@@ -111,6 +111,8 @@ class PermissionID : public std::pair<APIPermission::ID, base::string16> {
 // instead of pointers and change this to extend BaseSetOperators<PermissionID>.
 class PermissionIDSet {
  public:
+  using const_iterator = std::set<PermissionID>::const_iterator;

[Devlin, 2015/04/02 21:00:23]

Makes me wonder if this can/should inherit from BaseSetOperators.  Ah well -
that's out of this CL's scope.

[Marc Treib, 2015/04/07 11:34:47]

It probably should (eventually), see the TODO above.

[Devlin, 2015/04/07 15:55:56]

Haha oh, look at that. :)

+
   PermissionIDSet();
   virtual ~PermissionIDSet();
 
@@ -132,6 +134,7 @@ class PermissionIDSet {
 
   // Convenience functions that call their stl_util counterparts.
   bool Includes(const PermissionIDSet& subset) const;
+  bool Equals(const PermissionIDSet& set) const;
   static PermissionIDSet Difference(const PermissionIDSet& set_1,
                                     const PermissionIDSet& set_2);
   static PermissionIDSet Intersection(const PermissionIDSet& set_1,
@@ -142,6 +145,9 @@ class PermissionIDSet {
   size_t size() const;
   bool empty() const;
 
+  const_iterator begin() const { return permissions_.begin(); }
+  const_iterator end() const { return permissions_.end(); }
+
  private:
   PermissionIDSet(std::set<PermissionID> permissions);