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

Side by Side Diff: third_party/WebKit/LayoutTests/sensor/ambient-light-sensor.html

Issue 2353493002: [Sensors] Allow Sensor API only on secure top-level browsing contexts and add frequency checks (Closed)
Patch Set: Created 4 years, 3 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/mojo-helpers.js"></script> 4 <script src="../resources/mojo-helpers.js"></script>
5 <script src="resources/sensor-helpers.js"></script> 5 <script src="resources/sensor-helpers.js"></script>
6 <script> 6 <script>
7 7
8 'use strict'; 8 'use strict';
9 9
10 if (!window.testRunner) 10 if (!window.testRunner)
11 debug('This test cannot be run without the TestRunner'); 11 debug('This test cannot be run without the TestRunner');
12 12
13 const kDefaultReadingValue = 3.1415; 13 const kDefaultReadingValue = 3.1415;
14 14
15 function update_sensor_reading(buffer) { 15 function update_sensor_reading(buffer) {
16 buffer[0] = window.performance.now(); 16 buffer[0] = window.performance.now();
17 buffer[1] = kDefaultReadingValue; 17 buffer[1] = kDefaultReadingValue;
18 } 18 }
19 19
20 test(() => assert_throws(new RangeError(), () => new AmbientLightSensor({frequen cy: -60})),
shalamov 2016/09/19 13:11:42 nit: Format to 80 columns?
Mikhail 2016/09/19 14:18:02 Done.
21 'Test that negative frequency argument causes exception from constructor.');
22
20 sensor_test(sensor => { 23 sensor_test(sensor => {
21 let ambientLightSensor = new AmbientLightSensor({frequency: 60}); 24 let ambientLightSensor = new AmbientLightSensor({frequency: 60});
22 ambientLightSensor.start(); 25 ambientLightSensor.start();
23 let testPromise = sensor.mockSensorProvider.getCreatedSensor() 26 let testPromise = sensor.mockSensorProvider.getCreatedSensor()
24 .then((mockSensor) => { 27 .then((mockSensor) => {
25 return new Promise((resolve, reject) => { 28 return new Promise((resolve, reject) => {
26 ambientLightSensor.onstatechange = event => { 29 ambientLightSensor.onstatechange = event => {
27 if (ambientLightSensor.state === 'idle') { 30 if (ambientLightSensor.state === 'idle') {
28 resolve(mockSensor); 31 resolve(mockSensor);
29 } 32 }
(...skipping 150 matching lines...) Expand 10 before | Expand all | Expand 10 after
180 ambientLightSensor.onerror = reject; 183 ambientLightSensor.onerror = reject;
181 }); 184 });
182 }) 185 })
183 .then(mockSensor => { return mockSensor.removeConfigurationCalled(); }); 186 .then(mockSensor => { return mockSensor.removeConfigurationCalled(); });
184 187
185 return testPromise; 188 return testPromise;
186 }, 'Test that sensor receives suspend / resume notifications when page' 189 }, 'Test that sensor receives suspend / resume notifications when page'
187 +' visibility changes.'); 190 +' visibility changes.');
188 191
189 </script> 192 </script>
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698