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

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

Issue 2596063003: Revert of [Sensors] Reland: Align sensor reading updates and 'onchange' notification with rAF. (Closed)
Patch Set: Created 4 years 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 fc1fc8ed316a9d98204c4853717e2f68f9ba8f2a..ee5f312333cceab119156d79549531e72f380954 100644
--- a/third_party/WebKit/LayoutTests/sensor/resources/generic-sensor-tests.js
+++ b/third_party/WebKit/LayoutTests/sensor/resources/generic-sensor-tests.js
@@ -291,7 +291,6 @@
return new Promise((resolve, reject) => {
let fastSensorNotifiedCounter = 0;
let slowSensorNotifiedCounter = 0;
- let readingUpdatesCounter = 0;
let fastSensorWrapper = new CallbackWrapper(() => {
fastSensorNotifiedCounter++;
@@ -301,16 +300,11 @@
slowSensorNotifiedCounter++;
if (slowSensorNotifiedCounter == 1) {
fastSensor.start();
- readingUpdatesCounter = mockSensor.reading_updates_count();
} else if (slowSensorNotifiedCounter == 2) {
// By the moment slow sensor (9 Hz) is notified for the
// next time, the fast sensor (30 Hz) has been notified
// for int(30/9) = 3 times.
- // In actual implementation updates are bound to rAF,
- // (not to a timer) so fluctuations are possible, so we
- // reference to the actual elapsed updates count.
- let elapsedUpdates = mockSensor.reading_updates_count() - readingUpdatesCounter;
- assert_equals(fastSensorNotifiedCounter, elapsedUpdates);
+ assert_equals(fastSensorNotifiedCounter, 3);
fastSensor.stop();
slowSensor.stop();
resolve(mockSensor);

Powered by Google App Engine
This is Rietveld 408576698