arc: bluetooth: Implement Gatt server request to read/write

components/arc/bluetooth/arc_bluetooth_bridge.h

 // Copyright 2016 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 COMPONENTS_ARC_BLUETOOTH_ARC_BLUETOOTH_BRIDGE_H_
 #define COMPONENTS_ARC_BLUETOOTH_ARC_BLUETOOTH_BRIDGE_H_
 
 #include <stdint.h>
 
 #include <map>
@@ skipping 22 matching lines @@
 class ArcBridgeService;
 
 class ArcBluetoothBridge
     : public ArcService,
       public InstanceHolder<mojom::BluetoothInstance>::Observer,
       public device::BluetoothAdapter::Observer,
       public device::BluetoothAdapterFactory::AdapterCallback,
       public device::BluetoothLocalGattService::Delegate,
       public mojom::BluetoothHost {
  public:
+  // Store callback functions for Gatt Server request read / write
+  template <class SuccessCallback>
+  struct GattServerCallbacks {
+    GattServerCallbacks() = default;
+    GattServerCallbacks(const SuccessCallback& success_callback,
+                        const ErrorCallback& error_callback)
+        : success_callback(success_callback), error_callback(error_callback) {}
+    SuccessCallback success_callback;
+    ErrorCallback error_callback;
+  };
+
   using GattStatusCallback = base::Callback<void(mojom::BluetoothGattStatus)>;
   using GattReadCallback = base::Callback<void(mojom::BluetoothGattValuePtr)>;
+  using GattServerReadCallbacks = GattServerCallbacks<ValueCallback>;
+  using GattServerWriteCallbacks = GattServerCallbacks<base::Closure>;
 
   explicit ArcBluetoothBridge(ArcBridgeService* bridge_service);
   ~ArcBluetoothBridge() override;
 
   // Overridden from
   // InstanceHolder<mojom::BluetoothInstance>::Observer:
   void OnInstanceReady() override;
 
   void OnAdapterInitialized(scoped_refptr<device::BluetoothAdapter> adapter);
 
@@ skipping 59 matching lines @@
       device::BluetoothAdapter* adapter,
       device::BluetoothRemoteGattCharacteristic* characteristic,
       const std::vector<uint8_t>& value) override;
 
   void GattDescriptorValueChanged(
       device::BluetoothAdapter* adapter,
       device::BluetoothRemoteGattDescriptor* descriptor,
       const std::vector<uint8_t>& value) override;
 
   // Overridden from device::BluetoothLocalGattService::Delegate
+  template <class LocalGattAttribute>
+  void OnGattAttributeReadRequest(const device::BluetoothDevice* device,
+                                  const LocalGattAttribute* gatt_attr,
+                                  int offset,
+                                  const GattServerReadCallbacks& callbacks);

[Luis Héctor Chávez, 2016/07/19 01:49:17]

override?

[puthik_chromium, 2016/07/19 22:21:47]

Function defined in wrong place. Fixed.
On{Characteristic,Descriptor}ReadRequest is the overridden function.
This is the templace function to avoid duplicate code.

+
+  template <class LocalGattAttribute>
+  void OnGattAttributeWriteRequest(
+      const device::BluetoothDevice* device,
+      const LocalGattAttribute* gatt_attr,
+      const std::vector<uint8_t>& value,
+      int offset,
+      const GattServerWriteCallbacks& callbacks) override;
+
   void OnCharacteristicReadRequest(
       const device::BluetoothDevice* device,
       const device::BluetoothLocalGattCharacteristic* characteristic,
       int offset,
       const ValueCallback& callback,
       const ErrorCallback& error_callback) override;
 
   void OnCharacteristicWriteRequest(
       const device::BluetoothDevice* device,
       const device::BluetoothLocalGattCharacteristic* characteristic,
@@ skipping 224 matching lines @@
       mojom::BluetoothAddressPtr remote_addr,
       mojom::BluetoothGattServiceIDPtr service_id,
       mojom::BluetoothGattIDPtr char_id,
       mojom::BluetoothGattIDPtr desc_id) const;
 
   // Propagates the list of paired device to Android.
   void SendCachedPairedDevices() const;
 
   template <class LocalGattAttribute>
   int32_t CreateGattAttributeHandle(LocalGattAttribute* gatt_attr);
+  int32_t CreateGattReadTransaction(const GattServerReadCallbacks& callbacks);
+  int32_t CreateGattWriteTransaction(const GattServerWriteCallbacks& callbacks);
 
   mojo::Binding<mojom::BluetoothHost> binding_;
 
   scoped_refptr<bluez::BluetoothAdapterBlueZ> bluetooth_adapter_;
   scoped_refptr<device::BluetoothAdvertisement> advertisment_;
   std::unique_ptr<device::BluetoothDiscoverySession> discovery_session_;
   std::map<std::string, std::unique_ptr<device::BluetoothGattNotifySession>>
       notification_session_;
   // Map from android int handle to Chrome (BlueZ) string identifier.
   std::map<int32_t, std::string> gatt_identifier_;
+  // Map from Chrome (BlueZ) string identifier to android int handle.
+  std::map<std::string, int32_t> gatt_handle_;

[Luis Héctor Chávez, 2016/07/19 01:49:17]

std::unordered_map?

[puthik_chromium, 2016/07/19 22:21:47]

Done.

   // Store last GattCharacteristic added to each GattService for GattServer.
   std::map<int32_t, int32_t> last_characteristic_;
   // Running number for handle to give to Android side.
   int32_t gatt_server_attr_handle = 0;
+  std::map<int32_t, GattServerReadCallbacks> gatt_server_read_callbacks_;
+  std::map<int32_t, GattServerWriteCallbacks> gatt_server_write_callbacks_;
 
   // WeakPtrFactory to use for callbacks.
   base::WeakPtrFactory<ArcBluetoothBridge> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(ArcBluetoothBridge);
 };
 
 }  // namespace arc
 
 #endif  // COMPONENTS_ARC_BLUETOOTH_ARC_BLUETOOTH_BRIDGE_H_