browser: implement multiple permission requesting

content/public/browser/permission_manager.h

 // Copyright 2015 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 CONTENT_PUBLIC_BROWSER_PERMISSION_MANAGER_H_
 #define CONTENT_PUBLIC_BROWSER_PERMISSION_MANAGER_H_
 
 #include "content/common/content_export.h"
 #include "content/public/common/permission_status.mojom.h"
 
@@ skipping 18 matching lines @@
   // CancelPermissionRequest). This can be kNoPendingRequestOrSubscription if
   // there is no further need to cancel the permission in which case |callback|
   // was invoked.
   virtual int RequestPermission(
       PermissionType permission,
       RenderFrameHost* render_frame_host,
       const GURL& requesting_origin,
       bool user_gesture,
       const base::Callback<void(PermissionStatus)>& callback) = 0;
 
+  // Requests multiple permissions on behalf of a frame identified by
+  // render_frame_host.
+  // When the permission request is handled, whether it failed, timed out or
+  // succeeded, the |callback| will be run. The order of statuses in the
+  // returned vector will correspond to the order of requested permission
+  // types.
+  // Returns a request id which can be used to cancel the request (see
+  // CancelPermissionRequest). This can be kNoPendingRequestOrSubscription if
+  // there is no further need to cancel the permission in which case |callback|
+  // was invoked.
+  virtual int RequestPermissions(
+      const std::vector<PermissionType>& permission,
+      RenderFrameHost* render_frame_host,
+      const GURL& requesting_origin,
+      bool user_gesture,
+      const base::Callback<void(
+          const std::vector<PermissionStatus>&)>& callback) = 0;

[michaelbai, 2015/09/23 17:51:52]

Are you going to combine RequestPermission into RequestPermissions? they are
same if vector<> only has one permission.

[Lalit Maganti, 2015/09/23 17:57:25]

Yes you're right but no we are not going to do this. We did consider it but we
decided against forcing implementations from supporting a multiple permission
request if they did not want to.

Mounir: anything to add from your end here? 

+
   // Cancels a previous permission request specified by |request_id|. Cancelling
   // an already cancelled request or providing the |request_id|
   // kNoPendingRequestOrSubscription is a no-op.
   virtual void CancelPermissionRequest(int request_id) = 0;
 
   // 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.
   virtual PermissionStatus GetPermissionStatus(
       PermissionType permission,
@@ skipping 30 matching lines @@
   virtual void UnsubscribePermissionStatusChange(int subscription_id) = 0;
 
   // Constant retured when registering and subscribing if
   // cancelling/unsubscribing at a later stage would have no effect.
   static const int kNoPendingRequestOrSubscription = -1;
 };
 
 }  // namespace content
 
 #endif // CONTENT_PUBLIC_BROWSER_PERMISSION_MANAGER_H_