renderer: implement multiple permission requesting

content/public/browser/permission_manager.h

Index: content/public/browser/permission_manager.h
diff --git a/content/public/browser/permission_manager.h b/content/public/browser/permission_manager.h
index 5a2e078718dab32004594a0166f442b26335da50..5e0f16386e16319c22d6ec1f6a8ad551ac7a1ecd 100644
--- a/content/public/browser/permission_manager.h
+++ b/content/public/browser/permission_manager.h
@@ -34,6 +34,15 @@ class CONTENT_EXPORT PermissionManager {
       bool user_gesture,
       const base::Callback<void(PermissionStatus)>& callback) = 0;
 
+  using StatusVector = std::vector<PermissionStatus>;
+  virtual void RequestPermissions(
+      const std::vector<PermissionType>& permission,
+      RenderFrameHost* render_frame_host,
+      int request_id,
+      const GURL& requesting_origin,
+      bool user_gesture,
+      const base::Callback<void(const StatusVector&)>& callback) = 0;
+
   // Cancels a previously requested permission. The given parameter must match
   // the ones passed to the RequestPermission call.
   virtual void CancelPermissionRequest(PermissionType permission,
@@ -41,6 +50,12 @@ class CONTENT_EXPORT PermissionManager {
                                        int request_id,
                                        const GURL& requesting_origin) = 0;
 
+   // Cancels a previously requested batch permission request.
+   // The given parameters must match the ones passed to the
+   // RequestPermission call.
+   virtual void CancelPermissionsRequest(RenderFrameHost* render_frame_host,
+                                         int request_id) = 0;

[mlamouri (slow - plz ping), 2015/08/18 13:37:15]

Would it make sense to change CancelPermission to that or is PermissionType
required for something? (I assume in both case we can pass the origin).

[Lalit Maganti, 2015/08/20 14:23:30]

In CancelPermission, it is used to retrieve the context.

In any case, with unification of code paths this has changed.

+
   // Returns the permission status of a given requesting_origin/embedding_origin
   // tuple. This is not taking a RenderFrameHost because the call might happen
   // outside of a frame context.