bluetooth: Impl uses new WebBluetoothGATTRemoteServer content/browser.

content/child/bluetooth/bluetooth_dispatcher.h

 // 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.
 
 #ifndef CONTENT_CHILD_BLUETOOTH_BLUETOOTH_DISPATCHER_H_
 #define CONTENT_CHILD_BLUETOOTH_BLUETOOTH_DISPATCHER_H_
 
 #include "base/id_map.h"
 #include "base/memory/ref_counted.h"
 #include "content/child/worker_task_runner.h"
@@ skipping 30 matching lines @@
   // |thread_safe_sender| is required when constructing a BluetoothDispatcher.
   static BluetoothDispatcher* GetOrCreateThreadSpecificInstance(
       ThreadSafeSender* thread_safe_sender);
 
   // IPC Send and Receiving interface, see IPC::Sender and IPC::Listener.
   bool Send(IPC::Message* msg);
   void OnMessageReceived(const IPC::Message& msg);
 
   // 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.
   void OnWorkerRunLoopStopped() override;
 
  private:
   // IPC Handlers, see definitions in bluetooth_messages.h.
   void OnRequestDeviceSuccess(int thread_id,
                               int request_id,
                               const BluetoothDevice& device);
   void OnRequestDeviceError(int thread_id,
                             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);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_CHILD_BLUETOOTH_BLUETOOTH_DISPATCHER_H_