Chromium Code Reviews Chromium Code Reviews
Issue 1666003002:
bluetooth: Add Web Bluetooth blacklist checks to requestDevice. (Closed)
Base URL: https://chromium.googlesource.com/chromium/src.git@bt-blacklist-| OLD | NEW |
|---|---|
| (Empty) | |
| 1 <!DOCTYPE html> | |
| 2 <script src="../resources/testharness.js"></script> | |
| 3 <script src="../resources/testharness-helpers.js"></script> | |
| 4 <script src="../resources/testharnessreport.js"></script> | |
| 5 <script src="resources/bluetooth-helpers.js"></script> | |
| 6 <script> | |
| 7 'use strict'; | |
| 8 | |
| 9 test(t => { assert_true(window.testRunner instanceof Object); t.done(); }, | |
| 10 'window.testRunner is required for the following tests.'); | |
| 11 | |
| 12 promise_test(() => { | |
| 13 testRunner.setBluetoothMockDataSet('HeartRateAdapter'); | |
| 14 return assert_promise_rejects_with_message( | |
| 15 requestDeviceWithKeyDown( | |
| 16 {filters: [{services: ['human_interface_device']}]}), | |
| 17 new DOMException( | |
| 18 'requestDevice() called with a filter containing a blacklisted UUID.', | |
| 19 'SecurityError'), | |
| 20 'Requesting blacklisted service rejects.'); | |
| 21 }, 'Reject with SecurityError if requesting a blacklisted service.'); | |
| 22 | |
| 23 promise_test(() => { | |
| 24 testRunner.setBluetoothMockDataSet('HeartRateAndHIDAdapter'); | |
| 25 return requestDeviceWithKeyDown( | |
| 26 { | |
| 27 filters: [{services: ['heart_rate']}], | |
| 28 optionalServices: ['human_interface_device'] | |
| 29 }) | |
| 30 .then(device => device.gatt.connect()) | |
| 31 .then(gattServer => { | |
| 32 return assert_promise_rejects_with_message( | |
| 33 gattServer.getPrimaryService('human_interface_device'), | |
| 34 new DOMException( | |
| 35 'Origin is not allowed to access the service. Remember to add the ' + | |
| 36 'service to a filter or to optionalServices in requestDevice().', | |
| 37 'SecurityError'), | |
| 38 'Blacklisted service not accessible.'); | |
| 39 }); | |
| 40 }, 'Blacklisted UUID in optionalServices is removed and access not granted.'); | |
| 41 | |
| 42 | |
|
ortuno
2016/02/12 16:41:47
Remove new lines.
scheib
2016/02/12 18:25:32
Done.
| |
| 43 </script> | |
| OLD | NEW |
