Implement WebBluetooth getDescriptor[s]

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

 // 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.
 
 module blink.mojom;
 
 import "device/bluetooth/public/interfaces/uuid.mojom";
 
 // Result codes 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 WebBluetoothResult {
   SUCCESS,
   // AbortError:
   // InvalidModificationError:
   GATT_INVALID_ATTRIBUTE_LENGTH,
   // InvalidStateError:
   SERVICE_NO_LONGER_EXISTS,
   CHARACTERISTIC_NO_LONGER_EXISTS,
+  DESCRIPTOR_NO_LONGER_EXISTS,
   // NetworkError:
   CONNECT_ALREADY_IN_PROGRESS,
   CONNECT_ATTRIBUTE_LENGTH_INVALID,
   CONNECT_AUTH_CANCELED,
   CONNECT_AUTH_FAILED,
   CONNECT_AUTH_REJECTED,
   CONNECT_AUTH_TIMEOUT,
   CONNECT_CONNECTION_CONGESTED,
   CONNECT_INSUFFICIENT_ENCRYPTION,
   CONNECT_OFFSET_INVALID,
@@ skipping 11 matching lines @@
   NO_BLUETOOTH_ADAPTER,
   CHOSEN_DEVICE_VANISHED,
   CHOOSER_CANCELLED,
   CHOOSER_NOT_SHOWN_API_GLOBALLY_DISABLED,
   CHOOSER_NOT_SHOWN_API_LOCALLY_DISABLED,
   CHOOSER_NOT_SHOWN_USER_DENIED_PERMISSION_TO_SCAN,
   SERVICE_NOT_FOUND,
   NO_SERVICES_FOUND,
   CHARACTERISTIC_NOT_FOUND,
   NO_CHARACTERISTICS_FOUND,
+  DESCRIPTOR_NOT_FOUND,
+  NO_DESCRIPTORS_FOUND,
   WEB_BLUETOOTH_NOT_SUPPORTED,
   BLUETOOTH_LOW_ENERGY_NOT_AVAILABLE,
   // NotSupportedError:
   GATT_UNKNOWN_ERROR,
   GATT_UNKNOWN_FAILURE,
   GATT_NOT_PERMITTED,
   GATT_NOT_SUPPORTED,
   GATT_UNTRANSLATED_ERROR_CODE,
   // SecurityError:
   GATT_NOT_AUTHORIZED,
+  BLACKLISTED_DESCRIPTOR_UUID,
   BLACKLISTED_CHARACTERISTIC_UUID,
   BLACKLISTED_READ,
   BLACKLISTED_WRITE,
   NOT_ALLOWED_TO_ACCESS_SERVICE,
   REQUEST_DEVICE_WITH_BLACKLISTED_UUID,
   REQUEST_DEVICE_FROM_CROSS_ORIGIN_IFRAME,
   REQUEST_DEVICE_WITHOUT_FRAME,
 };
 
 struct WebBluetoothScanFilter {
@@ skipping 27 matching lines @@
   string instance_id;
   string uuid;
 };
 
 struct WebBluetoothRemoteGATTCharacteristic {
   string instance_id;
   string uuid;
   uint32 properties;
 };
 
+struct WebBluetoothRemoteGATTDescriptor {
+  string instance_id;
+  string uuid;
+};
+
 // Web Bluetooth Interface that Blink can use to perform
 // Bluetooth GATT Operations on Bluetooth Devices.
 interface WebBluetoothService {
   // Sets the client for this WebBluetoothService. The service will notify the
   // client of device events e.g. when a Characteristic's value changes or when
   // a device disconnects.
   SetClient(associated WebBluetoothServiceClient client);
 
   RequestDevice(WebBluetoothRequestDeviceOptions options)
     => (WebBluetoothResult result, WebBluetoothDevice? device);
@@ skipping 49 matching lines @@
 
   // Starts notifications for the characteristic with
   // |characteristic_instance_id|.
   RemoteCharacteristicStartNotifications(
     string characteristic_instance_id) => (WebBluetoothResult result);
 
   // Stops notifications for the characteristic with
   // |characteristic_instance_id|.
   RemoteCharacteristicStopNotifications(
     string characteristic_instance_id) => ();
+
+  // Returns the Descriptors of a GATT Characteristics with |service_instance_id|.
+  // If |quantity| == WebBluetoothGATTQueryQuantity::SINGLE, only one
+  // descriptor will be returned.
+  RemoteCharacteristicGetDescriptors(
+    string characteristics_instance_id,
+    WebBluetoothGATTQueryQuantity quantity,
+    bluetooth.mojom.UUID? characteristics_uuid) => (
+      WebBluetoothResult result,
+      array<WebBluetoothRemoteGATTDescriptor>? descriptors);
+
+  // Reads the value for descriptor with
+  // |descriptor_instance_id|. If the value is successfully read the
+  // callback will be run with WebBluetoothResult::SUCCESS and the
+  // descriptor's value. If the value is not successfully read the
+  // callback with be run with the corresponding error and nullptr for value.
+  RemoteDescriptorReadValue(
+    string descriptor_instance_id) => (
+    WebBluetoothResult result,
+    array<uint8>? value);
+
+  // Writes a value to the descriptor with
+  // |descriptor_instance_id|. The callback is run with
+  // WebBluetoothResult::SUCCESS if the value was successfully
+  // written.
+  RemoteDescriptorWriteValue(
+    string descriptor_instance_id,
+    array<uint8> value) => (WebBluetoothResult result);
 };
 
 // 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);
+  RemoteDescriptorValueChanged(string descriptor_instance_id,

[ortuno, 2016/11/21 03:34:09]

I don't think we need this. We do it for characteristics because we need to get
characteristics notifications but there is no such a thing for descriptors.

[dougt, 2016/11/22 01:47:17]

Acknowledged.

+                               array<uint8> value);
   GattServerDisconnected(WebBluetoothDeviceId device_id);
 };