Chromium Code Reviews Chromium Code Reviews
Issue 2014473002:
bluetooth: Web Bluetooth can filter by empty device names, doesn't leak MACs. (Closed)
Base URL: https://chromium.googlesource.com/chromium/src.git@bt-GetNameOrEmpty-| Index: third_party/WebKit/LayoutTests/bluetooth/requestDevice/unnamed-device.html |
| diff --git a/third_party/WebKit/LayoutTests/bluetooth/requestDevice/unnamed-device.html b/third_party/WebKit/LayoutTests/bluetooth/requestDevice/unnamed-device.html |
| new file mode 100644 |
| index 0000000000000000000000000000000000000000..19b67a0909b74e5b04dc379a9884bad2d5e68396 |
| --- /dev/null |
| +++ b/third_party/WebKit/LayoutTests/bluetooth/requestDevice/unnamed-device.html |
| @@ -0,0 +1,44 @@ |
| +<!DOCTYPE html> |
| +<script src="../../resources/testharness.js"></script> |
| +<script src="../../resources/testharnessreport.js"></script> |
| +<script src="../../resources/bluetooth/bluetooth-helpers.js"></script> |
| +<script> |
| +'use strict'; |
| +promise_test(() => { |
| + return setBluetoothFakeAdapter('UnnamedHeartRateAdapter') |
| + .then(() => requestDeviceWithKeyDown({ |
| + filters: [{services: ['heart_rate']}], |
| + optionalServices: ['generic_access']})) |
| + .then(device => { |
| + assert_equals(device.name, ''); |
| + return device.gatt.connect(); |
| + }) |
| + .then(gattServer => gattServer.getPrimaryService('generic_access')) |
| + .then(service => service.getCharacteristic('gap.device_name')) |
| + .then(characteristic => characteristic.readValue()) |
| + .then(value => { |
| + let decoder = new TextDecoder('utf-8'); |
| + let value_str = decoder.decode(value); |
| + assert_equals(value_str, ''); |
| + }); |
| +}, 'An unnamed device can be obtained by advertised service UUID.'); |
| + |
| +promise_test(() => { |
| + return setBluetoothFakeAdapter('UnnamedHeartRateAdapter') |
| + .then(() => requestDeviceWithKeyDown({ |
| + filters: [{name: ['']}], |
| + optionalServices: ['generic_access']})) |
| + .then(device => { |
| + assert_equals(device.name, ''); |
| + return device.gatt.connect(); |
| + }) |
| + .then(gattServer => gattServer.getPrimaryService('generic_access')) |
| + .then(service => service.getCharacteristic('gap.device_name')) |
| + .then(characteristic => characteristic.readValue()) |
| + .then(value => { |
| + let decoder = new TextDecoder('utf-8'); |
| + let value_str = decoder.decode(value); |
| + assert_equals(value_str, ''); |
| + }); |
| +}, 'An unnamed device can be obtained by empty name.'); |
|
ortuno
2016/07/21 17:42:29
Hmm well the spec says that if the device is not p
scheib
2016/07/30 02:38:14
Renamed adapter to be 'EmptyNameHeartRateAdapter',
|
| +</script> |
