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

Unified Diff: content/child/bluetooth/web_bluetooth_impl.cc

Issue 1228113004: Put the RenderFrame's ID into a per-frame WebBluetooth, and plumb that back to the browser. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@store-request-device-state-in-dispatcher
Patch Set: Initial Created 5 years, 5 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/child/bluetooth/web_bluetooth_impl.h ('k') | content/common/bluetooth/bluetooth_messages.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/child/bluetooth/web_bluetooth_impl.cc
diff --git a/content/child/bluetooth/web_bluetooth_impl.cc b/content/child/bluetooth/web_bluetooth_impl.cc
index c307461a5345fe98d311dd38defbf9fbe3c55ba8..166018ec23a1304a7e01f2df3f2f17ee9db9598e 100644
--- a/content/child/bluetooth/web_bluetooth_impl.cc
+++ b/content/child/bluetooth/web_bluetooth_impl.cc
@@ -6,11 +6,18 @@
#include "content/child/bluetooth/bluetooth_dispatcher.h"
#include "content/child/thread_safe_sender.h"
+#include "ipc/ipc_message.h"
namespace content {
WebBluetoothImpl::WebBluetoothImpl(ThreadSafeSender* thread_safe_sender)
- : thread_safe_sender_(thread_safe_sender) {
+ : WebBluetoothImpl(thread_safe_sender, MSG_ROUTING_NONE) {
+}
+
+WebBluetoothImpl::WebBluetoothImpl(ThreadSafeSender* thread_safe_sender,
+ int frame_routing_id)
+ : thread_safe_sender_(thread_safe_sender),
+ frame_routing_id_(frame_routing_id) {
jochen (gone - plz use gerrit) 2015/07/15 13:40:55 stuff in child/ shouldn't know about frames. why
Jeffrey Yasskin 2015/07/15 15:47:16 Would this be better if I called it a "routing_id"
}
WebBluetoothImpl::~WebBluetoothImpl() {
@@ -19,7 +26,7 @@ WebBluetoothImpl::~WebBluetoothImpl() {
void WebBluetoothImpl::requestDevice(
const blink::WebRequestDeviceOptions& options,
blink::WebBluetoothRequestDeviceCallbacks* callbacks) {
- GetDispatcher()->requestDevice(options, callbacks);
+ GetDispatcher()->requestDevice(frame_routing_id_, options, callbacks);
}
void WebBluetoothImpl::connectGATT(const blink::WebString& device_instance_id,
« no previous file with comments | « content/child/bluetooth/web_bluetooth_impl.h ('k') | content/common/bluetooth/bluetooth_messages.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698