Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(463)

Side by Side Diff: third_party/WebKit/LayoutTests/bluetooth/characteristic/notifications/add-multiple-event-listeners.html

Issue 2825393004: bluetooth: Check the order of event and promise resolution for readValue (Closed)
Patch Set: docs cleanup Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 <!DOCTYPE html> 1 <!DOCTYPE html>
2 <script src="../../../resources/testharness.js"></script> 2 <script src="../../../resources/testharness.js"></script>
3 <script src="../../../resources/testharnessreport.js"></script> 3 <script src="../../../resources/testharnessreport.js"></script>
4 <script src="../../../resources/bluetooth/bluetooth-helpers.js"></script> 4 <script src="../../../resources/bluetooth/bluetooth-helpers.js"></script>
5 <script> 5 <script>
6 'use strict'; 6 'use strict';
7 promise_test(() => { 7 promise_test(() => {
8 let char; 8 let char;
9 return setBluetoothFakeAdapter('HeartRateAdapter') 9 return setBluetoothFakeAdapter('HeartRateAdapter')
10 .then(() => requestDeviceWithKeyDown({ 10 .then(() => requestDeviceWithKeyDown({
11 filters: [{services: ['heart_rate']}]})) 11 filters: [{services: ['heart_rate']}]}))
12 .then(device => device.gatt.connect()) 12 .then(device => device.gatt.connect())
13 .then(gattServer => gattServer.getPrimaryService('heart_rate')) 13 .then(gattServer => gattServer.getPrimaryService('heart_rate'))
14 .then(service => service.getCharacteristic('heart_rate_measurement')) 14 .then(service => service.getCharacteristic('heart_rate_measurement'))
15 .then(characteristic => { 15 .then(characteristic => {
16 char = characteristic; 16 char = characteristic;
17 return assert_event_fires_after_promise(characteristic, 17 return assert_event_fires_after_promise(characteristic,
scheib 2017/04/21 06:32:12 legacy issue, but still: It's a bit odd that the
ortuno 2017/04/26 05:56:18 Totally agree. Done.
18 'startNotifications', 18 'startNotifications',
19 'characteristicvaluechanged', 19 'characteristicvaluechanged',
20 3 /* add 3 listeners */, 20 3 /* add 3 listeners */);
21 false /* ignore_event_promise_orde r */);
22 }) 21 })
23 .then(() => char.stopNotifications()) 22 .then(() => char.stopNotifications())
24 .then(() => assert_no_events(char, 'characteristicvaluechanged')); 23 .then(() => assert_no_events(char, 'characteristicvaluechanged'));
25 }, 'Add multiple event listeners then startNotifications().'); 24 }, 'Add multiple event listeners then startNotifications().');
26 </script> 25 </script>
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698