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

Unified Diff: content/browser/media/midi_dispatcher_host.cc

Issue 335993002: Convert MIDI permission requests to use WebContents in preparation for switching the code to using … (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 6 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
« no previous file with comments | « content/browser/media/midi_dispatcher_host.h ('k') | content/browser/media/midi_host.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/browser/media/midi_dispatcher_host.cc
diff --git a/content/browser/media/midi_dispatcher_host.cc b/content/browser/media/midi_dispatcher_host.cc
new file mode 100644
index 0000000000000000000000000000000000000000..2169f397a1fc0c5bd0e0815ee03f156713d08d42
--- /dev/null
+++ b/content/browser/media/midi_dispatcher_host.cc
@@ -0,0 +1,118 @@
+// Copyright 2013 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/browser/media/midi_dispatcher_host.h"
+
+#include "base/bind.h"
+#include "content/browser/child_process_security_policy_impl.h"
+#include "content/browser/renderer_host/render_view_host_impl.h"
+#include "content/common/media/midi_messages.h"
+#include "content/public/browser/browser_context.h"
+#include "content/public/browser/browser_thread.h"
+#include "content/public/browser/content_browser_client.h"
+#include "content/public/browser/render_process_host.h"
+#include "content/public/browser/render_view_host.h"
+#include "content/public/browser/web_contents.h"
+#include "url/gurl.h"
+
+namespace content {
+
+MidiDispatcherHost::PendingPermission::PendingPermission(
+ int render_process_id,
+ int render_view_id,
+ int bridge_id)
+ : render_process_id(render_process_id),
+ render_view_id(render_view_id),
+ bridge_id(bridge_id) {
+}
+
+MidiDispatcherHost::PendingPermission::~PendingPermission() {
+}
+
+MidiDispatcherHost::MidiDispatcherHost(WebContents* web_contents)
+ : WebContentsObserver(web_contents),
+ weak_factory_(this) {
+}
+
+MidiDispatcherHost::~MidiDispatcherHost() {
+}
+
+bool MidiDispatcherHost::OnMessageReceived(const IPC::Message& message) {
+ bool handled = true;
+ IPC_BEGIN_MESSAGE_MAP(MidiDispatcherHost, message)
+ IPC_MESSAGE_HANDLER(MidiHostMsg_RequestSysExPermission,
+ OnRequestSysExPermission)
+ IPC_MESSAGE_HANDLER(MidiHostMsg_CancelSysExPermissionRequest,
+ OnCancelSysExPermissionRequest)
+ IPC_MESSAGE_UNHANDLED(handled = false)
+ IPC_END_MESSAGE_MAP()
+ return handled;
+}
+
+void MidiDispatcherHost::OnRequestSysExPermission(int bridge_id,
+ const GURL& origin,
+ bool user_gesture) {
+ int render_process_id = web_contents()->GetRenderProcessHost()->GetID();
+ int render_view_id = web_contents()->GetRenderViewHost()->GetRoutingID();
+
+ PendingPermission pending_permission(
+ render_process_id, render_view_id, bridge_id);
+ pending_permissions_.push_back(pending_permission);
+
+ GetContentClient()->browser()->RequestMidiSysExPermission(
+ web_contents(),
+ bridge_id,
+ origin,
+ user_gesture,
+ base::Bind(&MidiDispatcherHost::WasSysExPermissionGranted,
+ weak_factory_.GetWeakPtr(),
+ render_process_id, render_view_id, bridge_id),
+ &pending_permissions_.back().cancel);
+}
+
+void MidiDispatcherHost::OnCancelSysExPermissionRequest(
+ int bridge_id,
+ const GURL& requesting_frame) {
+ int render_process_id = web_contents()->GetRenderProcessHost()->GetID();
+ int render_view_id = web_contents()->GetRenderViewHost()->GetRoutingID();
+
+ for (size_t i = 0; i < pending_permissions_.size(); ++i) {
+ if (pending_permissions_[i].render_process_id == render_process_id &&
+ pending_permissions_[i].render_view_id == render_view_id &&
+ pending_permissions_[i].bridge_id == bridge_id) {
+ if (!pending_permissions_[i].cancel.is_null())
+ pending_permissions_[i].cancel.Run();
+ pending_permissions_.erase(pending_permissions_.begin() + i);
+ return;
+ }
+ }
+}
+
+void MidiDispatcherHost::WasSysExPermissionGranted(int render_process_id,
+ int render_view_id,
+ int bridge_id,
+ bool is_allowed) {
+ for (size_t i = 0; i < pending_permissions_.size(); ++i) {
+ if (pending_permissions_[i].render_process_id == render_process_id &&
+ pending_permissions_[i].render_view_id == render_view_id &&
+ pending_permissions_[i].bridge_id == bridge_id) {
+ RenderViewHost* render_view_host =
+ RenderViewHost::FromID(render_process_id, render_view_id);
+ if (render_view_host) {
+ render_view_host->Send(new MidiMsg_SysExPermissionApproved(
+ render_view_id, bridge_id, is_allowed));
+ }
+
+ if (is_allowed) {
+ ChildProcessSecurityPolicyImpl::GetInstance()->
+ GrantSendMidiSysExMessage(render_process_id);
+ }
+
+ pending_permissions_.erase(pending_permissions_.begin() + i);
+ return;
+ }
+ }
+}
+
+} // namespace content
« no previous file with comments | « content/browser/media/midi_dispatcher_host.h ('k') | content/browser/media/midi_host.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698