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

Unified Diff: third_party/WebKit/LayoutTests/sensor/resources/generic-sensor-tests.js

Issue 2839963004: [Sensors] Error handler in Senor class should stop the platform sensor (Closed)
Patch Set: fix test failures 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 side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/LayoutTests/sensor/resources/generic-sensor-tests.js
diff --git a/third_party/WebKit/LayoutTests/sensor/resources/generic-sensor-tests.js b/third_party/WebKit/LayoutTests/sensor/resources/generic-sensor-tests.js
index 52783b278800745431f7a657e08da6202422b9da..ff03398bf410ff7c1f58676d45255c00072cc746 100644
--- a/third_party/WebKit/LayoutTests/sensor/resources/generic-sensor-tests.js
+++ b/third_party/WebKit/LayoutTests/sensor/resources/generic-sensor-tests.js
@@ -68,6 +68,30 @@ function runGenericSensorTests(sensorType, updateReading, verifyReading) {
}, 'Test that frequency is capped to 60.0 Hz.');
sensor_test(sensor => {
+ let sensorObject = new sensorType();
+ sensorObject.start();
+ return sensor.mockSensorProvider.getCreatedSensor()
+ .then(mockSensor => mockSensor.addConfigurationCalled())
+ .then(mockSensor => {
+ return new Promise((resolve, reject) => {
+ sensorObject.onactivate = () => {
+ // Now sensor proxy is initialized.
+ let anotherSensor = new sensorType({frequency: 21});
+ anotherSensor.start();
+ anotherSensor.stop();
+ resolve(mockSensor);
+ }
+ });
+ })
+ .then(mockSensor => mockSensor.removeConfigurationCalled())
+ .then(mockSensor => {
+ sensorObject.stop();
+ return mockSensor;
+ })
+ .then(mockSensor => mockSensor.removeConfigurationCalled());
+ }, 'Test that configuration is removed for a stopped sensor.');
+
+ sensor_test(sensor => {
let maxSupportedFrequency = 15;
sensor.mockSensorProvider.setMaximumSupportedFrequency(maxSupportedFrequency);
let sensorObject = new sensorType({frequency: 50});

Powered by Google App Engine
This is Rietveld 408576698