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

Unified Diff: third_party/WebKit/LayoutTests/bluetooth/stopNotifications/reconnect-during-success.html

Issue 2443063002: bluetooth: Require frame to be connected for stopNotifications to succeed. (Closed)
Patch Set: Fix test expectations Created 4 years, 2 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 side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/LayoutTests/bluetooth/stopNotifications/reconnect-during-success.html
diff --git a/third_party/WebKit/LayoutTests/bluetooth/writeValue/gen-gatt-op-reconnect-during-success.html b/third_party/WebKit/LayoutTests/bluetooth/stopNotifications/reconnect-during-success.html
similarity index 79%
copy from third_party/WebKit/LayoutTests/bluetooth/writeValue/gen-gatt-op-reconnect-during-success.html
copy to third_party/WebKit/LayoutTests/bluetooth/stopNotifications/reconnect-during-success.html
index c2f4f347b4510bcb85547a98fb41567939d24503..98809e6a9770415345866ce14905fcabae181060 100644
--- a/third_party/WebKit/LayoutTests/bluetooth/writeValue/gen-gatt-op-reconnect-during-success.html
+++ b/third_party/WebKit/LayoutTests/bluetooth/stopNotifications/reconnect-during-success.html
@@ -6,22 +6,22 @@
<script>
promise_test(() => {
let val = new Uint8Array([1]);
- return setBluetoothFakeAdapter('GATTOperationSucceedsAfterReconnectionAdapter')
+ return setBluetoothFakeAdapter('StopNotifySessionFinishesAfterReconnectionAdapter')
.then(() => requestDeviceWithKeyDown({
filters: [{services: ['health_thermometer']}]}))
.then(device => device.gatt.connect())
.then(gatt => gatt.getPrimaryService('health_thermometer'))
.then(service => service.getCharacteristic('measurement_interval'))
+ .then(characteristic => characteristic.startNotifications())
.then(characteristic => {
let promise = assert_promise_rejects_with_message(
- characteristic.writeValue(val),
+ characteristic.stopNotifications(),
new DOMException('GATT Server disconnected while performing a GATT operation.',
'NetworkError'));
let gatt = characteristic.service.device.gatt;
gatt.disconnect();
return gatt.connect().then(() => promise);
});
-}, 'disconnect() and connect() called during a writeValue call that ' +
+}, 'disconnect() and connect() called during a stopNotifications call that ' +
'succeeds. Reject with NetworkError.');
-
</script>

Powered by Google App Engine
This is Rietveld 408576698