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

Unified Diff: content/browser/permissions/permission_service_context.h

Issue 722153003: Implement basic mojo Permission service and use it for Geolocation. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: review comments Created 6 years, 1 month 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/browser/permissions/permission_service_context.h
diff --git a/content/browser/permissions/permission_service_context.h b/content/browser/permissions/permission_service_context.h
new file mode 100644
index 0000000000000000000000000000000000000000..adf4f564343afc7e403bb298429487d0794160a0
--- /dev/null
+++ b/content/browser/permissions/permission_service_context.h
@@ -0,0 +1,44 @@
+// Copyright 2014 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 "base/macros.h"
+#include "content/public/browser/web_contents_observer.h"
+
+#ifndef CONTENT_BROWSER_PERMISSIONS_PERMISSION_SERVICE_CONTEXT_H_
+#define CONTENT_BROWSER_PERMISSIONS_PERMISSION_SERVICE_CONTEXT_H_
+
+namespace content {
+
+class PermissionServiceImpl;
+class RenderFrameHost;
+
+// Provides information to a PermissionService. It is used by the
+// PermissionService to handle request permission UI.
+// The RenderFrameHost creates an instance when setting up the Permission mojo
+// service and then passes ownership to the service.
+class PermissionServiceContext : public WebContentsObserver {
+ public:
+ explicit PermissionServiceContext(RenderFrameHost* render_frame_host);
+ virtual ~PermissionServiceContext();
+
+ // WebContentsObserver
+ void RenderFrameDeleted(RenderFrameHost* render_frame_host) override;
+ void DidNavigateAnyFrame(RenderFrameHost* render_frame_host,
+ const LoadCommittedDetails& details,
+ const FrameNavigateParams& params) override;
+
+ RenderFrameHost* render_frame_host();
+
+ void set_service(PermissionServiceImpl* service);
qsr 2014/11/14 09:20:26 This seems weird. There is a single context for mu
mlamouri (slow - plz ping) 2014/11/14 11:37:11 That's not the model I had in mind. Contrary to th
+
+ private:
+ RenderFrameHost* render_frame_host_;
+ PermissionServiceImpl* service_;
+
+ DISALLOW_COPY_AND_ASSIGN(PermissionServiceContext);
+};
+
+} // namespace content
+
+#endif // CONTENT_BROWSER_PERMISSIONS_PERMISSION_SERVICE_CONTEXT_H_

Powered by Google App Engine
This is Rietveld 408576698