bluetooth: Impl uses new WebBluetoothGATTRemoteServer content/browser.

content/child/bluetooth/bluetooth_dispatcher.h

Index: content/child/bluetooth/bluetooth_dispatcher.h
diff --git a/content/child/bluetooth/bluetooth_dispatcher.h b/content/child/bluetooth/bluetooth_dispatcher.h
index 36c2ec3c42e44ef8902acbf62b7b0f9ae2280dbf..b42ed58ec5808219a61841b6b1de2a3f0ce67574 100644
--- a/content/child/bluetooth/bluetooth_dispatcher.h
+++ b/content/child/bluetooth/bluetooth_dispatcher.h
@@ -48,6 +48,8 @@ class BluetoothDispatcher : public WorkerTaskRunner::Observer {
 
   // Corresponding to WebBluetoothImpl methods.
   void requestDevice(blink::WebBluetoothRequestDeviceCallbacks* callbacks);
+  void connectGATT(std::string instance_id,
+                   blink::WebBluetoothConnectGATTCallbacks* callbacks);
   void SetBluetoothMockDataSetForTesting(const std::string& name);
 
   // WorkerTaskRunner::Observer implementation.
@@ -62,12 +64,19 @@ class BluetoothDispatcher : public WorkerTaskRunner::Observer {
                             int request_id,
                             BluetoothError error_type);
 
+  void OnConnectGATTSuccess(int thread_id,
+                            int request_id,
+                            const std::string& message);
+
   scoped_refptr<ThreadSafeSender> thread_safe_sender_;
 
   // Tracks requests sent to browser to match replies with callbacks.
   // Owns callback objects.
   IDMap<blink::WebBluetoothRequestDeviceCallbacks, IDMapOwnPointer>
       pending_requests_;
+  // TODO(ortuno): Create an callback interface to use in the map.

[scheib, 2015/04/19 04:44:19]

Drop the TODO comment for now. I'm not certain we'll end up using an interface.
Looking around some I see several examples where simple strong typed IDMap
members are used and duplicated for each call. I also see some that merge the
callbacks, (WebIDBCallbacks, PermissionDispatcher::CallbackInformation) but I'm
not convinced the code complication there is worth it. To be complete, it would
be trivial to static cast the types in an IDMap<void*, but this doesn't seem
like the situation I'd be willing to do that.

[ortuno, 2015/04/20 20:09:33]

Done.

+  IDMap<blink::WebBluetoothConnectGATTCallbacks, IDMapOwnPointer>
+      pending_connect_requests_;
 
   DISALLOW_COPY_AND_ASSIGN(BluetoothDispatcher);
 };