bluetooth: Add warning JS message when feature is not implemented |
|||
Issue description
It would be nice to add warning messages to the JS console when a method is not implemented yet.
For instance, on Mac OS, we know server.getPrimaryService, service.getCharacteristic, characteristic.readValue, characteristic.writeValue, characteristic.startNotifications and characteristic.stopNotifications are not working yet. Sadly, they are available and might confuse developers.
Same applies to characteristic.stopNotifications on Android.
I believe a simple `console.warn('[Feature] is not fully implemented yet. Check out Web BLuetooth [implementation status]')` could help developers while they're experimenting with the API.
,
May 4 2016
,
May 4 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/4ae4c0730e8984a7e18a0b2e170deec765b1a600 commit 4ae4c0730e8984a7e18a0b2e170deec765b1a600 Author: beaufort.francois <beaufort.francois@gmail.com> Date: Wed May 04 13:06:11 2016 bluetooth: Add JS error messages for unimplemented features BUG= 607822 Review-Url: https://codereview.chromium.org/1940923002 Cr-Commit-Position: refs/heads/master@{#391487} [modify] https://crrev.com/4ae4c0730e8984a7e18a0b2e170deec765b1a600/third_party/WebKit/LayoutTests/TestExpectations [modify] https://crrev.com/4ae4c0730e8984a7e18a0b2e170deec765b1a600/third_party/WebKit/Source/modules/bluetooth/BluetoothRemoteGATTCharacteristic.cpp [modify] https://crrev.com/4ae4c0730e8984a7e18a0b2e170deec765b1a600/third_party/WebKit/Source/modules/bluetooth/BluetoothRemoteGATTServer.cpp [modify] https://crrev.com/4ae4c0730e8984a7e18a0b2e170deec765b1a600/third_party/WebKit/Source/modules/bluetooth/BluetoothRemoteGATTService.cpp
,
May 4 2016
Verified in Chromium for Mac 52.0.2725.0 |
|||
►
Sign in to add a comment |
|||
Comment 1 by fbeaufort@chromium.org
, Apr 29 2016