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

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

Index: third_party/WebKit/Source/modules/bluetooth/BluetoothError.h
diff --git a/third_party/WebKit/Source/modules/bluetooth/BluetoothError.h b/third_party/WebKit/Source/modules/bluetooth/BluetoothError.h
index c9374283b4cf3b0de64c4c7daa985b68ac74b147..372eb11358ee496fcbc9342d33866b4a99fa4468 100644
--- a/third_party/WebKit/Source/modules/bluetooth/BluetoothError.h
+++ b/third_party/WebKit/Source/modules/bluetooth/BluetoothError.h
@@ -23,6 +23,11 @@ class BluetoothError {
   // Interface required by CallbackPromiseAdapter:
   static DOMException* take(ScriptPromiseResolver*,
                             mojom::blink::WebBluetoothResult);
+
+  static DOMException* take(

[ortuno, 2017/02/07 21:54:46]

I don't think it makes sense to keep using take. We needed it because of
CallbackPromiseAdapter which we no longer use. I think it would make more sense
to move CreateDOMException from BluetoothRemoteGATTUtils here and add enums for
the new types of errors.

[perja, 2017/02/08 15:05:47]

Done.

+      ScriptPromiseResolver*,
+      mojom::blink::WebBluetoothResult,
+      String detailedMessage /* Detailed error message - can be empty.*/);

[ortuno, 2017/02/07 21:54:46]

Also could this be an optional argument so we can use the same method?

[perja, 2017/02/08 15:05:47]

Done.

 };
 
 }  // namespace blink