bluetooth: Move writeValue to mojo

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;
 
 // 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 {

[scheib, 2016/03/31 04:17:51]

Not always an Error, so Result?

[ortuno, 2016/03/31 16:56:44]

Opened http://crbug.com/599489

   SUCCESS,
   // AbortError:
   // InvalidModificationError:
   GATT_INVALID_ATTRIBUTE_LENGTH,
   // InvalidStateError:
   SERVICE_NO_LONGER_EXISTS,
   CHARACTERISTIC_NO_LONGER_EXISTS,
   // NetworkError:
   CONNECT_ALREADY_IN_PROGRESS,
   CONNECT_ATTRIBUTE_LENGTH_INVALID,
@@ skipping 38 matching lines @@
   REQUEST_DEVICE_WITH_BLACKLISTED_UUID,
   REQUEST_DEVICE_WITH_UNIQUE_ORIGIN,
   REQUEST_DEVICE_WITHOUT_FRAME,
   // SyntaxError:
   // TODO(ortuno): Remove once we no longer use IPC.
   ENUM_MAX_VALUE = REQUEST_DEVICE_WITHOUT_FRAME,
 };
 
 // TODO(ortuno): Define Bluetooth Service.
 // https://crbug.com/508771
+
+// Web Bluetooth Interface that Blink can use to perform
+// Bluetooth GATT Operations on Bluetooth Devices.
+interface WebBluetoothService {

[scheib, 2016/03/31 04:17:51]

In person we discussed and I thought you said the plan was a mojo interface each
for adapter, device, characteristic, descriptor? 

[ortuno, 2016/03/31 16:56:44]

Yes. That's the long term plan. Since that + mojo would be too much to refactor
at once, I started by only converting from IPC::Message to Mojo.

+
+  // Writes a value to the characteristic with
+  // |characteristic_instance_id|. The callback is run with
+  // WebBluetoothError::SUCCESS if the value was successfully
+  // written.
+  RemoteCharacteristicWriteValue(
+    string characteristic_instance_id,
+    array<uint8> value) => (WebBluetoothError error);

[scheib, 2016/03/31 04:17:51]

error -> result

[ortuno, 2016/03/31 16:56:44]

Will do in http://crbug.com/599489

+
+};