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

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

Issue 2534703002: [Sensors] Layouts test to check frequency hint (Closed)
Patch Set: fixed MagnetometerReading::isReadingUpdated Created 4 years, 1 month 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/ambient-light-sensor.html
diff --git a/third_party/WebKit/LayoutTests/sensor/ambient-light-sensor.html b/third_party/WebKit/LayoutTests/sensor/ambient-light-sensor.html
index 1fd4e1074c1e40f17ddd131f60d8be2ca78e5e37..cde39bd7d0853a08da203cb86aac73e22bc2d070 100644
--- a/third_party/WebKit/LayoutTests/sensor/ambient-light-sensor.html
+++ b/third_party/WebKit/LayoutTests/sensor/ambient-light-sensor.html
@@ -12,10 +12,15 @@ if (!window.testRunner)
debug('This test cannot be run without the TestRunner');
const kDefaultReadingValue = 3.1415;
+var readsCount = 0;
-function update_sensor_reading(buffer) {
+function update_sensor_reading(buffer, expects_modified_reading) {
buffer[1] = window.performance.now();
buffer[2] = kDefaultReadingValue;
+ if (expects_modified_reading) {
+ buffer[2] += readsCount;
+ }
Reilly Grant (use Gerrit) 2016/11/28 19:22:53 nit: Unnecessary braces.
Mikhail 2016/11/30 10:31:30 Done.
+ readsCount++;
}
function verify_sensor_reading(reading) {

Powered by Google App Engine
This is Rietveld 408576698