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

Side by Side Diff: content/browser/media/midi_dispatcher_host.cc

Issue 571483002: Refactor content client to include a single Request/CancelPermission method. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@geoloc_permission
Patch Set: Created 6 years, 3 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 unified diff | Download patch
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "content/browser/media/midi_dispatcher_host.h" 5 #include "content/browser/media/midi_dispatcher_host.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "content/browser/child_process_security_policy_impl.h" 8 #include "content/browser/child_process_security_policy_impl.h"
9 #include "content/browser/frame_host/render_frame_host_impl.h" 9 #include "content/browser/frame_host/render_frame_host_impl.h"
10 #include "content/common/media/midi_messages.h" 10 #include "content/common/media/midi_messages.h"
(...skipping 46 matching lines...) Expand 10 before | Expand all | Expand 10 after
57 int bridge_id, 57 int bridge_id,
58 const GURL& origin, 58 const GURL& origin,
59 bool user_gesture) { 59 bool user_gesture) {
60 int render_process_id = render_frame_host->GetProcess()->GetID(); 60 int render_process_id = render_frame_host->GetProcess()->GetID();
61 int render_frame_id = render_frame_host->GetRoutingID(); 61 int render_frame_id = render_frame_host->GetRoutingID();
62 62
63 PendingPermission pending_permission( 63 PendingPermission pending_permission(
64 render_process_id, render_frame_id, bridge_id); 64 render_process_id, render_frame_id, bridge_id);
65 pending_permissions_.push_back(pending_permission); 65 pending_permissions_.push_back(pending_permission);
66 66
67 GetContentClient()->browser()->RequestMidiSysExPermission( 67 GetContentClient()->browser()->RequestPermission(
68 content::PERMISSION_MIDI_SYSEX,
68 web_contents(), 69 web_contents(),
69 bridge_id, 70 bridge_id,
70 origin, 71 origin,
71 user_gesture, 72 user_gesture,
72 base::Bind(&MidiDispatcherHost::WasSysExPermissionGranted, 73 base::Bind(&MidiDispatcherHost::WasSysExPermissionGranted,
73 weak_factory_.GetWeakPtr(), 74 weak_factory_.GetWeakPtr(),
74 render_process_id, render_frame_id, bridge_id), 75 render_process_id, render_frame_id, bridge_id));
75 &pending_permissions_.back().cancel);
76 } 76 }
77 77
78 void MidiDispatcherHost::OnCancelSysExPermissionRequest( 78 void MidiDispatcherHost::OnCancelSysExPermissionRequest(
79 RenderFrameHost* render_frame_host, 79 RenderFrameHost* render_frame_host,
80 int bridge_id, 80 int bridge_id,
81 const GURL& requesting_frame) { 81 const GURL& requesting_frame) {
82 int render_process_id = render_frame_host->GetProcess()->GetID(); 82 int render_process_id = render_frame_host->GetProcess()->GetID();
83 int render_frame_id = render_frame_host->GetRoutingID(); 83 int render_frame_id = render_frame_host->GetRoutingID();
84 84
85 for (size_t i = 0; i < pending_permissions_.size(); ++i) { 85 for (size_t i = 0; i < pending_permissions_.size(); ++i) {
86 if (pending_permissions_[i].render_process_id == render_process_id && 86 if (pending_permissions_[i].render_process_id == render_process_id &&
87 pending_permissions_[i].render_frame_id == render_frame_id && 87 pending_permissions_[i].render_frame_id == render_frame_id &&
88 pending_permissions_[i].bridge_id == bridge_id) { 88 pending_permissions_[i].bridge_id == bridge_id) {
89 if (!pending_permissions_[i].cancel.is_null()) 89 GetContentClient()->browser()->CancelPermissionRequest(
90 pending_permissions_[i].cancel.Run(); 90 content::PERMISSION_MIDI_SYSEX,
91 web_contents(), bridge_id, requesting_frame);
92
91 pending_permissions_.erase(pending_permissions_.begin() + i); 93 pending_permissions_.erase(pending_permissions_.begin() + i);
92 return; 94 return;
93 } 95 }
94 } 96 }
95 } 97 }
96 98
97 void MidiDispatcherHost::WasSysExPermissionGranted(int render_process_id, 99 void MidiDispatcherHost::WasSysExPermissionGranted(int render_process_id,
98 int render_frame_id, 100 int render_frame_id,
99 int bridge_id, 101 int bridge_id,
100 bool is_allowed) { 102 bool is_allowed) {
(...skipping 13 matching lines...) Expand all
114 GrantSendMidiSysExMessage(render_process_id); 116 GrantSendMidiSysExMessage(render_process_id);
115 } 117 }
116 118
117 pending_permissions_.erase(pending_permissions_.begin() + i); 119 pending_permissions_.erase(pending_permissions_.begin() + i);
118 return; 120 return;
119 } 121 }
120 } 122 }
121 } 123 }
122 124
123 } // namespace content 125 } // namespace content
OLDNEW
« no previous file with comments | « content/browser/media/midi_dispatcher_host.h ('k') | content/public/browser/content_browser_client.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698