renderer: implement multiple permission requesting

content/child/permissions/permission_dispatcher.cc

 // 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.
 
 #include "content/child/permissions/permission_dispatcher.h"
 
 #include "base/callback.h"
 #include "content/child/worker_task_runner.h"
 #include "content/public/common/service_registry.h"
 #include "third_party/WebKit/public/platform/WebURL.h"
@@ skipping 59 matching lines @@
 }  // anonymous namespace
 
 // static
 bool PermissionDispatcher::IsObservable(blink::WebPermissionType type) {
   return type == blink::WebPermissionTypeGeolocation ||
          type == blink::WebPermissionTypeNotifications ||
          type == blink::WebPermissionTypePushNotifications ||
          type == blink::WebPermissionTypeMidiSysEx;
 }
 
-PermissionDispatcher::CallbackInformation::CallbackInformation(
-    blink::WebPermissionCallback* callback,
-    int worker_thread_id)
-  : callback_(callback),
-    worker_thread_id_(worker_thread_id) {
-}
-
-blink::WebPermissionCallback*
-PermissionDispatcher::CallbackInformation::callback() const {
-  return callback_.get();
-}
-
-int PermissionDispatcher::CallbackInformation::worker_thread_id() const {
-  return worker_thread_id_;
-}
-
-blink::WebPermissionCallback*
-PermissionDispatcher::CallbackInformation::ReleaseCallback() {
-  return callback_.release();
-}
-
-PermissionDispatcher::CallbackInformation::~CallbackInformation() {
-}
-
 PermissionDispatcher::PermissionDispatcher(ServiceRegistry* service_registry)
     : service_registry_(service_registry) {
 }
 
 PermissionDispatcher::~PermissionDispatcher() {
 }
 
 void PermissionDispatcher::queryPermission(
     blink::WebPermissionType type,
     const blink::WebURL& origin,
     blink::WebPermissionCallback* callback) {
   QueryPermissionInternal(
       type, origin.string().utf8(), callback, kNoWorkerThread);
 }
 
 void PermissionDispatcher::requestPermission(
     blink::WebPermissionType type,
     const blink::WebURL& origin,
     blink::WebPermissionCallback* callback) {
   RequestPermissionInternal(
       type, origin.string().utf8(), callback, kNoWorkerThread);
 }
 
+void PermissionDispatcher::requestPermissions(
+    const blink::WebVector<blink::WebPermissionType>& types,
+    const blink::WebURL& origin,
+    blink::WebPermissionsCallback* callback) {
+  RequestPermissionsInternal(
+      types, origin.string().utf8(), callback, kNoWorkerThread);
+}
+
 void PermissionDispatcher::revokePermission(
     blink::WebPermissionType type,
     const blink::WebURL& origin,
     blink::WebPermissionCallback* callback) {
   RevokePermissionInternal(
       type, origin.string().utf8(), callback, kNoWorkerThread);
 }
 
 void PermissionDispatcher::startListening(
     blink::WebPermissionType type,
@@ skipping 29 matching lines @@
 }
 
 void PermissionDispatcher::RequestPermissionForWorker(
     blink::WebPermissionType type,
     const std::string& origin,
     blink::WebPermissionCallback* callback,
     int worker_thread_id) {
   RequestPermissionInternal(type, origin, callback, worker_thread_id);
 }
 
+void PermissionDispatcher::RequestPermissionsForWorker(
+    const blink::WebVector<blink::WebPermissionType>& types,
+    const std::string& origin,
+    blink::WebPermissionsCallback* callback,
+    int worker_thread_id) {
+  RequestPermissionsInternal(types, origin, callback, worker_thread_id);
+}
+
 void PermissionDispatcher::RevokePermissionForWorker(
     blink::WebPermissionType type,
     const std::string& origin,
     blink::WebPermissionCallback* callback,
     int worker_thread_id) {
   RevokePermissionInternal(type, origin, callback, worker_thread_id);
 }
 
 void PermissionDispatcher::StartListeningForWorker(
     blink::WebPermissionType type,
@@ skipping 31 matching lines @@
 }
 
 // static
 void PermissionDispatcher::RunCallbackOnWorkerThread(
     blink::WebPermissionCallback* callback,
     scoped_ptr<blink::WebPermissionStatus> status) {
   callback->onSuccess(status.release());
   delete callback;
 }
 
+void PermissionDispatcher::RunMultiCallbackOnWorkerThread(
+    blink::WebPermissionsCallback* callback,
+    scoped_ptr<blink::WebVector<blink::WebPermissionStatus>> statuses) {
+  callback->onSuccess(
+      blink::adoptWebPtr<blink::WebVector<blink::WebPermissionStatus>>(
+          statuses.release()));
+  delete callback;
+}
+
 PermissionServicePtr& PermissionDispatcher::GetPermissionServicePtr() {
   if (!permission_service_.get()) {
     service_registry_->ConnectToRemoteService(
         mojo::GetProxy(&permission_service_));
   }
   return permission_service_;
 }
 
 void PermissionDispatcher::QueryPermissionInternal(
     blink::WebPermissionType type,
     const std::string& origin,
     blink::WebPermissionCallback* callback,
     int worker_thread_id) {
   // We need to save the |callback| in an IDMap so if |this| gets deleted, the
   // callback will not leak. In the case of |this| gets deleted, the
   // |permission_service_| pipe will be destroyed too so OnQueryPermission will
   // not be called.
   int request_id = pending_callbacks_.Add(
-      new CallbackInformation(callback, worker_thread_id));
+      new CallbackInformation<blink::WebPermissionCallback>(
+          callback, worker_thread_id));
   GetPermissionServicePtr()->HasPermission(
       GetPermissionName(type),
       origin,
       base::Bind(&PermissionDispatcher::OnPermissionResponse,
                  base::Unretained(this),
                  request_id));
 }
 
 void PermissionDispatcher::RequestPermissionInternal(
     blink::WebPermissionType type,
     const std::string& origin,
     blink::WebPermissionCallback* callback,
     int worker_thread_id) {
   // We need to save the |callback| in an IDMap so if |this| gets deleted, the
   // callback will not leak. In the case of |this| gets deleted, the
   // |permission_service_| pipe will be destroyed too so OnQueryPermission will
   // not be called.
   int request_id = pending_callbacks_.Add(
-      new CallbackInformation(callback, worker_thread_id));
+      new SingleCallbackInfo(callback, worker_thread_id));
   GetPermissionServicePtr()->RequestPermission(
       GetPermissionName(type),
       origin,
       blink::WebUserGestureIndicator::isProcessingUserGesture(),
       base::Bind(&PermissionDispatcher::OnPermissionResponse,
                  base::Unretained(this),
                  request_id));
 }
 
+void PermissionDispatcher::RequestPermissionsInternal(
+    const blink::WebVector<blink::WebPermissionType>& types,
+    const std::string& origin,
+    blink::WebPermissionsCallback* callback,
+    int worker_thread_id) {
+  // We need to save the |callback| in an IDMap so if |this| gets deleted, the
+  // callback will not leak. In the case of |this| gets deleted, the
+  // |permission_service_| pipe will be destroyed too so OnQueryPermission will
+  // not be called.
+  int request_id = multiple_pending_callbacks_.Add(
+      new MultiCallbackInfo(callback, worker_thread_id));
+
+  mojo::Array<PermissionName> names(types.size());
+  for (size_t i = 0; i < types.size(); ++i) {
+    names[i] = GetPermissionName(types[i]);
+  }

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

nit: no { }

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

Done.

+
+  GetPermissionServicePtr()->RequestPermissions(
+      names.Pass(),
+      origin,
+      blink::WebUserGestureIndicator::isProcessingUserGesture(),
+      base::Bind(&PermissionDispatcher::OnPermissionsResponse,
+                 base::Unretained(this),
+                 request_id));
+}
+
 void PermissionDispatcher::RevokePermissionInternal(
     blink::WebPermissionType type,
     const std::string& origin,
     blink::WebPermissionCallback* callback,
     int worker_thread_id) {
   // We need to save the |callback| in an IDMap so if |this| gets deleted, the
   // callback will not leak. In the case of |this| gets deleted, the
   // |permission_service_| pipe will be destroyed too so OnQueryPermission will
   // not be called.
   int request_id = pending_callbacks_.Add(
-      new CallbackInformation(callback, worker_thread_id));
+      new SingleCallbackInfo(callback, worker_thread_id));
   GetPermissionServicePtr()->RevokePermission(
       GetPermissionName(type),
       origin,
       base::Bind(&PermissionDispatcher::OnPermissionResponse,
                  base::Unretained(this),
                  request_id));
 }
 
 void PermissionDispatcher::OnPermissionResponse(int request_id,
                                                 PermissionStatus result) {
-  CallbackInformation* callback_information =
+  SingleCallbackInfo* callback_information =
       pending_callbacks_.Lookup(request_id);
   DCHECK(callback_information && callback_information->callback());
   scoped_ptr<blink::WebPermissionStatus> status(
       new blink::WebPermissionStatus(GetWebPermissionStatus(result)));
 
   if (callback_information->worker_thread_id() != kNoWorkerThread) {
     blink::WebPermissionCallback* callback =
         callback_information->ReleaseCallback();
     int worker_thread_id = callback_information->worker_thread_id();
     pending_callbacks_.Remove(request_id);
 
     // If the worker is no longer running, ::PostTask() will return false and
     // gracefully fail, destroying the callback too.
     WorkerTaskRunner::Instance()->PostTask(
         worker_thread_id,
         base::Bind(&PermissionDispatcher::RunCallbackOnWorkerThread,
                    base::Unretained(callback),
                    base::Passed(&status)));
     return;
   }
 
   callback_information->callback()->onSuccess(status.release());
   pending_callbacks_.Remove(request_id);
 }
 
+void PermissionDispatcher::OnPermissionsResponse(
+    int request_id,
+    const mojo::Array<PermissionStatus>& result) {
+  MultiCallbackInfo* callback_information =
+      multiple_pending_callbacks_.Lookup(request_id);
+  DCHECK(callback_information && callback_information->callback());
+
+  scoped_ptr<blink::WebVector<blink::WebPermissionStatus>> status(

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

nit: statuses

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

Done.

+      new blink::WebVector<blink::WebPermissionStatus>(result.size()));
+  for (size_t i = 0; i < result.size(); i++) {
+    (*status)[i] = GetWebPermissionStatus(result[i]);
+  }

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

nit: remove { }

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

Done.

+
+  if (callback_information->worker_thread_id() != kNoWorkerThread) {
+    blink::WebPermissionsCallback* callback =
+        callback_information->ReleaseCallback();
+    int worker_thread_id = callback_information->worker_thread_id();
+    multiple_pending_callbacks_.Remove(request_id);
+
+    // If the worker is no longer running, ::PostTask() will return false and
+    // gracefully fail, destroying the callback too.
+    WorkerTaskRunner::Instance()->PostTask(
+        worker_thread_id,
+        base::Bind(&PermissionDispatcher::RunMultiCallbackOnWorkerThread,
+                   base::Unretained(callback),
+                   base::Passed(&status)));
+    return;
+  }
+
+  callback_information->callback()->onSuccess(
+      blink::adoptWebPtr<blink::WebVector<blink::WebPermissionStatus>>(
+          status.release()));

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

wouldn't status.release be enough?

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

Although that would make so much sense, WebPtrs don't have implicit conversion
implemented.

+  multiple_pending_callbacks_.Remove(request_id);
+}
+
 void PermissionDispatcher::OnPermissionChanged(
     blink::WebPermissionType type,
     const std::string& origin,
     WebPermissionObserver* observer,
     PermissionStatus status) {
   if (!IsObserverRegistered(observer))
     return;
 
   observer->permissionChanged(type, GetWebPermissionStatus(status));
 
@@ skipping 20 matching lines @@
       origin,
       current_status,
       base::Bind(&PermissionDispatcher::OnPermissionChanged,
                  base::Unretained(this),
                  type,
                  origin,
                  base::Unretained(observer)));
 }
 
 }  // namespace content