WebUSB: Create a requestDevice method separate from getDevice.

Source/modules/webusb/USB.cpp

Index: Source/modules/webusb/USB.cpp
diff --git a/Source/modules/webusb/USB.cpp b/Source/modules/webusb/USB.cpp
index 859334df20658a6454354c370459f49528004e2a..5acd9acac742004e5bd733717f624432d60d89da 100644
--- a/Source/modules/webusb/USB.cpp
+++ b/Source/modules/webusb/USB.cpp
@@ -12,41 +12,44 @@
 #include "core/dom/Document.h"
 #include "core/dom/ExceptionCode.h"
 #include "modules/webusb/USBDevice.h"
-#include "modules/webusb/USBDeviceEnumerationOptions.h"
 #include "modules/webusb/USBDeviceFilter.h"
+#include "modules/webusb/USBDeviceRequestOptions.h"
 #include "modules/webusb/USBError.h"
 #include "public/platform/Platform.h"
 #include "public/platform/WebVector.h"
 #include "public/platform/modules/webusb/WebUSBClient.h"
-#include "public/platform/modules/webusb/WebUSBDeviceEnumerationOptions.h"
 #include "public/platform/modules/webusb/WebUSBDeviceFilter.h"
+#include "public/platform/modules/webusb/WebUSBDeviceRequestOptions.h"
 #include "public/platform/modules/webusb/WebUSBError.h"
 
 namespace blink {
 namespace {
 
-void convertDeviceEnumerationOptions(const USBDeviceEnumerationOptions& options, WebUSBDeviceEnumerationOptions* webOptions)
+void convertDeviceFilter(const USBDeviceFilter& filter, WebUSBDeviceFilter* webFilter)
+{
+    webFilter->hasVendorID = filter.hasVendorId();
+    if (filter.hasVendorId())
+        webFilter->vendorID = filter.vendorId();
+    webFilter->hasProductID = filter.hasProductId();
+    if (filter.hasProductId())
+        webFilter->productID = filter.productId();
+    webFilter->hasClassCode = filter.hasClassCode();
+    if (filter.hasClassCode())
+        webFilter->classCode = filter.classCode();
+    webFilter->hasSubclassCode = filter.hasSubclassCode();
+    if (filter.hasSubclassCode())
+        webFilter->subclassCode = filter.subclassCode();
+    webFilter->hasProtocolCode = filter.hasProtocolCode();
+    if (filter.hasProtocolCode())
+        webFilter->protocolCode = filter.protocolCode();
+}
+
+void convertDeviceRequestOptions(const USBDeviceRequestOptions& options, WebUSBDeviceRequestOptions* webOptions)
 {
     ASSERT(options.hasFilters());
     webOptions->filters = WebVector<WebUSBDeviceFilter>(options.filters().size());
     for (size_t i = 0; i < options.filters().size(); ++i) {
-        const USBDeviceFilter& filter = options.filters()[i];
-        WebUSBDeviceFilter& webFilter = webOptions->filters[i];
-        webFilter.hasVendorID = filter.hasVendorId();
-        if (filter.hasVendorId())
-            webFilter.vendorID = filter.vendorId();
-        webFilter.hasProductID = filter.hasProductId();
-        if (filter.hasProductId())
-            webFilter.productID = filter.productId();
-        webFilter.hasClassCode = filter.hasClassCode();
-        if (filter.hasClassCode())
-            webFilter.classCode = filter.classCode();
-        webFilter.hasSubclassCode = filter.hasSubclassCode();
-        if (filter.hasSubclassCode())
-            webFilter.subclassCode = filter.subclassCode();
-        webFilter.hasProtocolCode = filter.hasProtocolCode();
-        if (filter.hasProtocolCode())
-            webFilter.protocolCode = filter.protocolCode();
+        convertDeviceFilter(options.filters()[i], &webOptions->filters[i]);
     }
 }
 
@@ -76,7 +79,7 @@ USB::USB(LocalFrame& frame)
 {
 }
 
-ScriptPromise USB::getDevices(ScriptState* scriptState, const USBDeviceEnumerationOptions& options)
+ScriptPromise USB::getDevices(ScriptState* scriptState)
 {
     WebUSBClient* client = m_controller->client();
     if (!client)
@@ -84,10 +87,24 @@ ScriptPromise USB::getDevices(ScriptState* scriptState, const USBDeviceEnumerati
 
     ScriptPromiseResolver* resolver = ScriptPromiseResolver::create(scriptState);
     ScriptPromise promise = resolver->promise();
+    client->getDevices(new CallbackPromiseAdapter<DeviceArray, USBError>(resolver));
+
+    return promise;
+}
+
+ScriptPromise USB::requestDevice(ScriptState* scriptState, const USBDeviceRequestOptions& options)
+{
+    WebUSBClient* client = m_controller->client();
+    if (!client)
+        return ScriptPromise::rejectWithDOMException(scriptState, DOMException::create(NotSupportedError));
+
+    ScriptPromiseResolver* resolver = ScriptPromiseResolver::create(scriptState);
+    ScriptPromise promise = resolver->promise();
+
+    WebUSBDeviceRequestOptions webOptions;
+    convertDeviceRequestOptions(options, &webOptions);
+    client->requestDevice(webOptions, new CallbackPromiseAdapter<USBDevice, USBError>(resolver));
 
-    WebUSBDeviceEnumerationOptions webOptions;
-    convertDeviceEnumerationOptions(options, &webOptions);
-    client->getDevices(webOptions, new CallbackPromiseAdapter<DeviceArray, USBError>(resolver));
     return promise;
 }