bluetooth: Use BluetoothUUID instead of string when sending uuids

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

 // Copyright 2016 The Chromium Authors. All rights reserved.

[jbroman, 2016/06/08 21:08:08]

I assume the plan is ultimately for Blink to directly invoke this service?
(Currently a lot of this stuff seems to be proxied via content/renderer/ which
actually does the IPC.)

[ortuno, 2016/06/09 16:15:59]

Yup. There is so much code that we decided to do it in two phases:

1. Use mojo for IPC. (Almost done. Some clean up, like this one, remains.)
2. Onion Soup

 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 module blink.mojom;
 
+import "device/bluetooth/public/interfaces/bluetooth_uuid.mojom";
+
 // Errors that can occur during Web Bluetooth execution, which are transformed
 // to a DOMException in Source/modules/bluetooth/BluetoothError.cpp.
 //
 // These errors all produce constant message strings. If a particular message
 // needs a dynamic component, we should add a separate enum so type-checking the
 // IPC ensures the dynamic component is passed.
 enum WebBluetoothError {
   SUCCESS,
   // AbortError:
   // InvalidModificationError:
@@ skipping 45 matching lines @@
   NOT_ALLOWED_TO_ACCESS_SERVICE,
   REQUEST_DEVICE_WITH_BLACKLISTED_UUID,
   REQUEST_DEVICE_FROM_CROSS_ORIGIN_IFRAME,
   REQUEST_DEVICE_WITHOUT_FRAME,
   // SyntaxError:
   // TODO(ortuno): Remove once we no longer use IPC.
   ENUM_MAX_VALUE = REQUEST_DEVICE_WITHOUT_FRAME,
 };
 
 struct WebBluetoothScanFilter {
-  array<string>? services;
+  array<device.mojom.BluetoothUUID>? services;
   string? name;
   string? name_prefix;
 };
 
 struct WebBluetoothRequestDeviceOptions {
   array<WebBluetoothScanFilter> filters;
-  array<string> optional_services;
+  array<device.mojom.BluetoothUUID> optional_services;
 };
 
 // Indicates if the function will return a single or multiple
 // GATT objects.
 enum WebBluetoothGATTQueryQuantity {
   SINGLE,
   MULTIPLE
 };
 
 struct WebBluetoothDevice {
@@ skipping 34 matching lines @@
   RemoteServerConnect(string device_id) => (WebBluetoothError error);
 
   // If a GATT connection exists for Device with |device_id| then decreases
   // the ref count for that connection.
   RemoteServerDisconnect(string device_id);
 
   // Returns the first GATT Service with |service_uuid| of a Bluetooth Device
   // with |device_id|.
   RemoteServerGetPrimaryService(
     string device_id,
-    string service_uuid) => (
+    device.mojom.BluetoothUUID service_uuid) => (
       WebBluetoothError error,
       WebBluetoothRemoteGATTService? service);
 
   // Returns the Characteristics of a GATT Service with |service_instance_id|.
   // If |quantity| == WebBluetoothGATTQueryQuantity::SINGLE, only one
   // characteristic will be returned.
   RemoteServiceGetCharacteristics(
     string service_instance_id,
     WebBluetoothGATTQueryQuantity quantity,
-    string? characteristics_uuid) => (
+    device.mojom.BluetoothUUID? characteristics_uuid) => (
       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
   // characteristic's value. If the value is not successfully read the
   // callback with be run with the corresponding error and nullptr for value.
   RemoteCharacteristicReadValue(
     string characteristic_instance_id) => (
@@ skipping 20 matching lines @@
 };
 
 // Classes should implement this interface and pass an associated pointer
 // bound to them to the WebBluetoothService by using SetClient. Classes
 // that do this will be notified of device events e.g. device disconnection.
 interface WebBluetoothServiceClient {
   RemoteCharacteristicValueChanged(string characteristic_instance_id,
                                    array<uint8> value);
   GattServerDisconnected(string device_id);
 };