arc: bluetooth: Add Gatt server mojo API

components/arc/common/bluetooth.mojom

Index: components/arc/common/bluetooth.mojom
diff --git a/components/arc/common/bluetooth.mojom b/components/arc/common/bluetooth.mojom
index 67bc7ff87234b306809d13909d18f71f7916b729..1868e3e2b2a23c7af929365f529709eed8314c6f 100644
--- a/components/arc/common/bluetooth.mojom
+++ b/components/arc/common/bluetooth.mojom
@@ -287,6 +287,36 @@ interface BluetoothHost {
   [MinVersion=1] ReadRemoteRssi@28(BluetoothAddress remote_addr)
       => (int32 rssi);
 
+  // Bluetooth Gatt Server functions
+  // TODO connect
+  [MinVersion=3] AddService@29(BluetoothGattServiceID service_id,
+                               int32 num_handles)

[rkc, 2016/06/29 21:54:17]

What is num_handles? Is this the handles of the included service? or?

Please add documentation to each of these functions clarifying what they do at
some location. Either here, or the header file.

[puthik_chromium, 2016/07/01 21:12:42]

Added at the header file.

+      => (int32 service_handle);
+  [MinVersion=3] AddCharacteristic@30(int32 service_handle,
+                                      BluetoothUUID uuid,
+                                      int32 properties,
+                                      int32 permissions)
+      => (int32 characteristic_handle);
+  [MinVersion=3] AddDescriptor@31(int32 service_handle,
+                                  BluetoothUUID uuid,
+                                  int32 permissions)
+      => (int32 descriptor_handle);
+  [MinVersion=3] StartService@32(int32 service_handle, int32 transport)

[rkc, 2016/06/29 21:54:17]

What is transport here?

[Eric Caruso, 2016/06/30 23:19:09]

It looks like this is BR/EDR, LE, etc.

I refactored the transport type so we have an enum for this in
device/bluetooth/bluetooth_common.h. Why don't we use the mojo DeviceType, which
gets typemapped to one of these?

[puthik_chromium, 2016/07/01 21:12:42]

Done.

+      => (BluetoothGattStatus status);
+  [MinVersion=3] StopService@33(int32 service_handle)
+      => (BluetoothGattStatus status);
+  [MinVersion=3] DeleteService@34(int32 service_handle)
+      => (BluetoothGattStatus status);
+  [MinVersion=3] SendIndication@35(int32 attribute_handle,

[rkc, 2016/06/29 21:54:17]

Should this be called SendNotification? The Android Java API uses the word
notify and the argument confirm.

Also, why is confirm an int32?

[Eric Caruso, 2016/06/30 23:19:09]

http://androidxref.com/6.0.1_r10/xref/hardware/libhardware/include/hardware/b...

Looks like this is libhardware's term for this function.

[puthik_chromium, 2016/07/01 21:12:42]

It's int32 in Android HAL, but the upper Java layer in Android is a boolean.
Boolean make more sense here so I change it to that.

+                                   BluetoothAddress address,
+                                   int32 confirm,
+                                   array<uint8> value)
+      => (BluetoothGattStatus status);
+  [MinVersion=3] SendResponse@36(int32 trans_id,
+                                 int32 status,
+                                 array<uint8> value)
+      => (BluetoothGattStatus status);
+
 };
 
 interface BluetoothInstance {
@@ -326,4 +356,16 @@ interface BluetoothInstance {
                                  BluetoothGattID char_id,
                                  bool is_notify,
                                  array<uint8> value);
+
+  // Bluetooth Gatt Server functions
+  [MinVersion=3] RequestGattRead@14(BluetoothAddress address,
+                                    int32 transaction_id,
+                                    int32 attribute_handle,
+                                    int32 offset,
+                                    bool is_long);
+  [MinVersion=3] RequestGattWrite@15(BluetoothAddress address,
+                                     int32 transaction_id,
+                                     int32 attribute_handle,
+                                     int32 offset,
+                                     array<uint8> value);
 };