bluetooth: show better error messages for services, characteristics and descriptors

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

 // Copyright 2014 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/BluetoothError.h"
 
 #include "core/dom/DOMException.h"
 #include "core/dom/ExceptionCode.h"
 
 namespace blink {
 
-DOMException* BluetoothError::take(ScriptPromiseResolver*,
-                                   mojom::blink::WebBluetoothResult error) {
+// static
+DOMException* BluetoothError::createDOMException(
+    BluetoothErrorCode error,
+    const String& detailedMessage) {
+  switch (error) {
+    case BluetoothErrorCode::InvalidService:
+    case BluetoothErrorCode::InvalidCharacteristic:
+    case BluetoothErrorCode::InvalidDescriptor:
+      return DOMException::create(InvalidStateError, detailedMessage);
+    case BluetoothErrorCode::ServiceNotFound:
+    case BluetoothErrorCode::CharacteristicNotFound:
+    case BluetoothErrorCode::DescriptorNotFound:
+      return DOMException::create(NotFoundError, detailedMessage);
+  }
+  NOTREACHED();
+  return DOMException::create(UnknownError);
+}
+
+// static
+DOMException* BluetoothError::createDOMException(
+    mojom::blink::WebBluetoothResult error) {
   switch (error) {
     case mojom::blink::WebBluetoothResult::SUCCESS:
-      ASSERT_NOT_REACHED();
+    case mojom::blink::WebBluetoothResult::SERVICE_NOT_FOUND:
+    case mojom::blink::WebBluetoothResult::CHARACTERISTIC_NOT_FOUND:
+    case mojom::blink::WebBluetoothResult::DESCRIPTOR_NOT_FOUND:
+      NOTREACHED();

[scheib, 2017/02/27 17:57:14]

Please add a comment here explaining why these three are not expected in this
switch, because they have detailed messages and are expected to be redirected to
the switch above.

[perja, 2017/02/28 11:45:18]

Done.

       return DOMException::create(UnknownError);
 #define MAP_ERROR(enumeration, name, message)         \
   case mojom::blink::WebBluetoothResult::enumeration: \
-    return DOMException::create(name, message)
+    return DOMException::create(name, message);
 
       // InvalidModificationErrors:
       MAP_ERROR(GATT_INVALID_ATTRIBUTE_LENGTH, InvalidModificationError,
                 "GATT Error: invalid attribute length.");
 
       // InvalidStateErrors:
       MAP_ERROR(SERVICE_NO_LONGER_EXISTS, InvalidStateError,
                 "GATT Service no longer exists.");
       MAP_ERROR(CHARACTERISTIC_NO_LONGER_EXISTS, InvalidStateError,
                 "GATT Characteristic no longer exists.");
@@ skipping 30 matching lines @@
                 "Unsupported device.");
       MAP_ERROR(CONNECT_WRITE_NOT_PERMITTED, NetworkError,
                 "GATT write operation is not permitted.");
       MAP_ERROR(DEVICE_NO_LONGER_IN_RANGE, NetworkError,
                 "Bluetooth Device is no longer in range.");
       MAP_ERROR(GATT_NOT_PAIRED, NetworkError, "GATT Error: Not paired.");
       MAP_ERROR(GATT_OPERATION_IN_PROGRESS, NetworkError,
                 "GATT operation already in progress.");
       MAP_ERROR(UNTRANSLATED_CONNECT_ERROR_CODE, NetworkError,
                 "Unknown ConnectErrorCode.");
+      MAP_ERROR(GATT_SERVER_NOT_CONNECTED, NetworkError,
+                "GATT Server is disconnected. Cannot perform GATT operations.");
+      MAP_ERROR(GATT_SERVER_DISCONNECTED, NetworkError,
+                "GATT Server disconnected while performing a GATT operation.");
+      MAP_ERROR(GATT_SERVER_DISCONNECTED_WHILE_RETRIEVING_CHARACTERISTICS,
+                NetworkError,
+                "GATT Server disconnected while retrieving characteristics.");
+      MAP_ERROR(
+          GATT_SERVER_NOT_CONNECTED_CANNOT_RETRIEVE_CHARACTERISTICS,
+          NetworkError,
+          "GATT Server is disconnected. Cannot retrieve characteristics.");
 
       // NotFoundErrors:
       MAP_ERROR(WEB_BLUETOOTH_NOT_SUPPORTED, NotFoundError,
                 "Web Bluetooth is not supported on this platform. For a list "
                 "of supported platforms see: https://goo.gl/J6ASzs");
       MAP_ERROR(NO_BLUETOOTH_ADAPTER, NotFoundError,
                 "Bluetooth adapter not available.");
       MAP_ERROR(CHOSEN_DEVICE_VANISHED, NotFoundError,
                 "User selected a device that doesn't exist anymore.");
       MAP_ERROR(CHOOSER_CANCELLED, NotFoundError,
                 "User cancelled the requestDevice() chooser.");
       MAP_ERROR(CHOOSER_NOT_SHOWN_API_GLOBALLY_DISABLED, NotFoundError,
                 "Web Bluetooth API globally disabled.");
       MAP_ERROR(CHOOSER_NOT_SHOWN_API_LOCALLY_DISABLED, NotFoundError,
                 "User or their enterprise policy has disabled Web Bluetooth.");
       MAP_ERROR(
           CHOOSER_NOT_SHOWN_USER_DENIED_PERMISSION_TO_SCAN, NotFoundError,
           "User denied the browser permission to scan for Bluetooth devices.");
-      MAP_ERROR(SERVICE_NOT_FOUND, NotFoundError,
-                "No Services with specified UUID found in Device.");
       MAP_ERROR(NO_SERVICES_FOUND, NotFoundError,
                 "No Services found in device.");
-      MAP_ERROR(CHARACTERISTIC_NOT_FOUND, NotFoundError,
-                "No Characteristics with specified UUID found in Service.");
       MAP_ERROR(NO_CHARACTERISTICS_FOUND, NotFoundError,
                 "No Characteristics found in service.");
-      MAP_ERROR(DESCRIPTOR_NOT_FOUND, NotFoundError,
-                "No Descriptors with specified UUID found in Characteristic.");
       MAP_ERROR(NO_DESCRIPTORS_FOUND, NotFoundError,
                 "No Descriptors found in Characteristic.");
       MAP_ERROR(BLUETOOTH_LOW_ENERGY_NOT_AVAILABLE, NotFoundError,
                 "Bluetooth Low Energy not available.");
 
       // NotSupportedErrors:
       MAP_ERROR(GATT_UNKNOWN_ERROR, NotSupportedError, "GATT Error Unknown.");
       MAP_ERROR(GATT_UNKNOWN_FAILURE, NotSupportedError,
                 "GATT operation failed for unknown reason.");
       MAP_ERROR(GATT_NOT_PERMITTED, NotSupportedError,
@@ skipping 24 matching lines @@
                 "options. https://goo.gl/HxfxSQ");
       MAP_ERROR(NOT_ALLOWED_TO_ACCESS_SERVICE, SecurityError,
                 "Origin is not allowed to access the service. Tip: Add the "
                 "service UUID to 'optionalServices' in requestDevice() "
                 "options. https://goo.gl/HxfxSQ");
       MAP_ERROR(REQUEST_DEVICE_WITH_BLOCKLISTED_UUID, SecurityError,
                 "requestDevice() called with a filter containing a blocklisted "
                 "UUID. https://goo.gl/4NeimX");
       MAP_ERROR(REQUEST_DEVICE_FROM_CROSS_ORIGIN_IFRAME, SecurityError,
                 "requestDevice() called from cross-origin iframe.");
-      MAP_ERROR(REQUEST_DEVICE_WITHOUT_FRAME, SecurityError,
-                "No window to show the requestDevice() dialog.");
 
 #undef MAP_ERROR
   }
 
-  ASSERT_NOT_REACHED();
+  NOTREACHED();
   return DOMException::create(UnknownError);
 }
 
 }  // namespace blink