bluetooth: Move GetCharacteristic(s) over to Mojo

third_party/WebKit/public/platform/modules/bluetooth/web_bluetooth.mojom

Index: third_party/WebKit/public/platform/modules/bluetooth/web_bluetooth.mojom
diff --git a/third_party/WebKit/public/platform/modules/bluetooth/web_bluetooth.mojom b/third_party/WebKit/public/platform/modules/bluetooth/web_bluetooth.mojom
index bdd829d20c149f7dd8ee6fdac0c444e22373004f..a5046a9d7e1c29ffb880be36e90671cc068e93f8 100644
--- a/third_party/WebKit/public/platform/modules/bluetooth/web_bluetooth.mojom
+++ b/third_party/WebKit/public/platform/modules/bluetooth/web_bluetooth.mojom
@@ -68,6 +68,12 @@ enum WebBluetoothError {
   ENUM_MAX_VALUE = REQUEST_DEVICE_WITHOUT_FRAME,
 };
 
+struct WebBluetoothRemoteGATTCharacteristic {
+  string instance_id;
+  string? uuid;
+  uint32 properties;
+};
+
 // TODO(ortuno): Define Bluetooth Service.
 // https://crbug.com/508771
 
@@ -79,6 +85,16 @@ interface WebBluetoothService {
   // a device disconnects.
   SetClient(associated WebBluetoothServiceClient client);
 
+  // Returns the Characteristics of a GATT Service with |service_instance_id|.
+  // If |single_characteristic| is true, only one characteristic will be
+  // returned.
+  RemoteServiceGetCharacteristics(
+    string service_instance_id,
+    bool single_characteristic,

[Jeffrey Yasskin, 2016/04/22 01:16:38]

We should make this an enum so the call sites are clearer than 'true' and
'false'.

[ortuno, 2016/04/25 15:29:45]

Done.

+    string? characteristics_uuid) => (

[Jeffrey Yasskin, 2016/04/22 01:16:38]

We should get this type-checked. I'm sorry for not noticing when it went into
bluetooth_messages.h, but that should have been a BluetoothUUID, not just a
string. IIRC, Mojo now has support for validators on particular arguments.

[ortuno, 2016/04/25 15:29:45]

No problem! I already noticed :) Yes, Mojo does have support but it's been
changing a lot lately so I've been advised to stay away from it until it
stabilizes a bit. I was planning on passing strings for now and once the move is
complete and typemaps are more stable I was going to replace all instances with
BluetoothUUID. Do you think that's acceptable? Otherwise I can start now, but
this patch will get slightly bigger.

[Jeffrey Yasskin, 2016/04/25 17:26:11]

Waiting sounds good.

[ortuno, 2016/04/25 20:37:41]

I opened an issue so that we don't forget.

+      WebBluetoothError error,
+      array<WebBluetoothRemoteGATTCharacteristic>? characteristics);
+
   // Reads the value for characteristic with
   // |characteristic_instance_id|. If the value is successfully read the
   // callback will be run with WebBluetoothError::SUCCESS and the