bluetooth: Implement Service.getCharacteristics

third_party/WebKit/Source/modules/bluetooth/BluetoothRemoteGATTService.cpp

 // Copyright 2015 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.
 
 #include "modules/bluetooth/BluetoothRemoteGATTService.h"
 
 #include "bindings/core/v8/CallbackPromiseAdapter.h"
 #include "bindings/core/v8/ScriptPromise.h"
 #include "bindings/core/v8/ScriptPromiseResolver.h"
 #include "core/dom/DOMException.h"
@@ skipping 28 matching lines @@
     if (exceptionState.hadException())
         return exceptionState.reject(scriptState);
 
     ScriptPromiseResolver* resolver = ScriptPromiseResolver::create(scriptState);
     ScriptPromise promise = resolver->promise();
     webbluetooth->getCharacteristic(m_webService->serviceInstanceID, characteristicUUID, new CallbackPromiseAdapter<BluetoothRemoteGATTCharacteristic, BluetoothError>(resolver));
 
     return promise;
 }
 
+ScriptPromise BluetoothRemoteGATTService::getCharacteristics(ScriptState* scriptState, ExceptionState&)
+{
+    return getCharacteristicsImpl(scriptState, String());
+}
+
+ScriptPromise BluetoothRemoteGATTService::getCharacteristics(ScriptState* scriptState, const StringOrUnsignedLong& characteristic, ExceptionState& exceptionState)
+{
+    String characteristicUUID = BluetoothUUID::getCharacteristic(characteristic, exceptionState);
+    if (exceptionState.hadException())
+        return exceptionState.reject(scriptState);
+
+    return getCharacteristicsImpl(scriptState, characteristicUUID);
+}
+
+// Class that allows us to use CallbackPromiseAdapter to resolve a promise with a
+// vector owning BluetoothRemoteGATTCharacteristics.
+class RemoteCharacteristicArray {
+    STATIC_ONLY(RemoteCharacteristicArray);
+public:
+    using WebType = OwnPtr<WebVector<WebBluetoothRemoteGATTCharacteristicInit*>>;
+    static HeapVector<Member<BluetoothRemoteGATTCharacteristic>> take(ScriptPromiseResolver* resolver, PassOwnPtr<WebVector<WebBluetoothRemoteGATTCharacteristicInit*>> webCharacteristics)
+    {
+        HeapVector<Member<BluetoothRemoteGATTCharacteristic>> characteristics;

[Jeffrey Yasskin, 2016/02/24 23:20:30]

Use characteristics.reserveInitialCapacity(webCharacteristics.size()) to avoid
reallocations and make sure the result is sized exactly.

[ortuno, 2016/02/25 23:08:27]

Done.

+        for (WebBluetoothRemoteGATTCharacteristicInit* webCharacteristic : *webCharacteristics) {
+            characteristics.append(BluetoothRemoteGATTCharacteristic::take(resolver, adoptPtr(webCharacteristic)));
+        }
+        return characteristics;
+    }
+};
+
+ScriptPromise BluetoothRemoteGATTService::getCharacteristicsImpl(ScriptState* scriptState, String characteristicUUID)
+{
+    ScriptPromiseResolver* resolver = ScriptPromiseResolver::create(scriptState);
+    ScriptPromise promise = resolver->promise();
+
+    WebBluetooth* webbluetooth = BluetoothSupplement::fromScriptState(scriptState);
+    webbluetooth->getCharacteristics(m_webService->serviceInstanceID, characteristicUUID, new CallbackPromiseAdapter<RemoteCharacteristicArray, BluetoothError>(resolver));
+
+    return promise;
+}
+
 } // namespace blink