Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1834)

Unified Diff: content/child/permissions/permissions_manager.h

Issue 801613004: Implement WebPermissionClient for Document and Worker contexts (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/child/permissions/permissions_manager.h
diff --git a/content/child/permissions/permissions_manager.h b/content/child/permissions/permissions_manager.h
new file mode 100644
index 0000000000000000000000000000000000000000..fde75f496019ed8951e12bd2e72ab95fd243d2f8
--- /dev/null
+++ b/content/child/permissions/permissions_manager.h
@@ -0,0 +1,83 @@
+// 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_CHILD_PERMISSIONS_PERMISSIONS_MANAGER_H_
+#define CONTENT_CHILD_PERMISSIONS_PERMISSIONS_MANAGER_H_
+
+#include <string>
+
+#include "base/compiler_specific.h"
+#include "base/id_map.h"
+#include "base/macros.h"
+#include "base/memory/scoped_ptr.h"
+#include "content/common/permission_service.mojom.h"
+#include "third_party/WebKit/public/platform/modules/permissions/WebPermissionClient.h"
+
+namespace content {
+
+class ServiceRegistry;
+
+// The PermissionManager is a layer between Blink and the Mojo PermissionService
+// It implements blink::WebPermissionClient. It is being used from workers and
+// frames independently. When called outside of the main thread,
+// QueryPermissionForWorker is meant to be called. It will handle the thread
+// jumping.
+class PermissionsManager :
+ public NON_EXPORTED_BASE(blink::WebPermissionClient) {
+ public:
+ PermissionsManager(ServiceRegistry& service_registry);
+ virtual ~PermissionsManager();
+
+ // blink::WebPermissionClient implementation.
+ virtual void queryPermission(blink::WebPermissionType type,
+ const blink::WebURL& origin,
+ blink::WebPermissionQueryCallback* callback);
+
+ void QueryPermissionForWorker(blink::WebPermissionType type,
+ const std::string& origin,
+ blink::WebPermissionQueryCallback* callback,
+ int worker_thread_id);
+
+ protected:
+ void QueryPermissionInternal(blink::WebPermissionType type,
+ const std::string& origin,
+ blink::WebPermissionQueryCallback* callback,
+ int worker_thread_id);
+
+ void OnQueryPermission(int request_id, PermissionStatus status);
+
+ // Called from the main thread in order to run the callback in the thread it
+ // was created on.
+ static void RunCallbackOnWorkerThread(
+ blink::WebPermissionQueryCallback* callback,
+ scoped_ptr<blink::WebPermissionStatus> status);
+
+ // Saves some basic information about the callback in order to be able to run
+ // it in the right thread.
+ class CallbackInformation {
+ public:
+ CallbackInformation(blink::WebPermissionQueryCallback* callback,
+ int worker_thread_id);
+ ~CallbackInformation();
+
+ blink::WebPermissionQueryCallback* callback() const;
+ int worker_thread_id() const;
+
+ blink::WebPermissionQueryCallback* ReleaseCallback();
+
+ private:
+ scoped_ptr<blink::WebPermissionQueryCallback> callback_;
+ int worker_thread_id_;
+ };
jochen (gone - plz use gerrit) 2015/03/09 15:36:31 disallow copy/assign
mlamouri (slow - plz ping) 2015/03/11 11:08:13 Done.
+ using CallbackMap = IDMap<CallbackInformation, IDMapOwnPointer>;
+ CallbackMap pending_callbacks_;
+
+ PermissionServicePtr permission_service_;
+
+ DISALLOW_COPY_AND_ASSIGN(PermissionsManager);
+};
+
+} // namespace content
+
+#endif // CONTENT_CHILD_PERMISSIONS_PERMISSIONS_MANAGER_H_

Powered by Google App Engine
This is Rietveld 408576698