Fixed a test failure seen in web-timing-minimal-performance-now.html
authorggaren@apple.com <ggaren@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 7 Dec 2016 21:10:24 +0000 (21:10 +0000)
committerggaren@apple.com <ggaren@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 7 Dec 2016 21:10:24 +0000 (21:10 +0000)
https://bugs.webkit.org/show_bug.cgi?id=165542

Reviewed by Beth Dakin.

This test was always slightly wrong, but it is much more likely to fail
as of <https://trac.webkit.org/changeset/209462>.

* fast/dom/Window/web-timing-minimal-performance-now.html: < is not a
correct test for monotonicity. < tests for stricly increasing. <= is a
correct test for monotonicity. All computer clocks have some granularity,
so computer clocks are monotonic but not strictly increasing: you can
sample the clock twice in the same tick.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@209471 268f45cc-cd09-0410-ab3c-d52691b4dbfc

LayoutTests/ChangeLog
LayoutTests/fast/dom/Window/web-timing-minimal-performance-now.html

index 6e8a1fd..bc58d8d 100644 (file)
@@ -1,3 +1,19 @@
+2016-12-07  Geoffrey Garen  <ggaren@apple.com>
+
+        Fixed a test failure seen in web-timing-minimal-performance-now.html
+        https://bugs.webkit.org/show_bug.cgi?id=165542
+
+        Reviewed by Beth Dakin.
+
+        This test was always slightly wrong, but it is much more likely to fail
+        as of <https://trac.webkit.org/changeset/209462>.
+
+        * fast/dom/Window/web-timing-minimal-performance-now.html: < is not a
+        correct test for monotonicity. < tests for stricly increasing. <= is a
+        correct test for monotonicity. All computer clocks have some granularity,
+        so computer clocks are monotonic but not strictly increasing: you can
+        sample the clock twice in the same tick.
+
 2016-12-07  Joseph Pecoraro  <pecoraro@apple.com>
 
         Web Inspector: Remove unused and mostly untested Page domain commands and events
index c04f43d..65fdff0 100644 (file)
@@ -10,7 +10,7 @@ var isMonotonicClock = false;
 
 function checkTimestamp(previousTimestamp) { 
     currentTimestamp = performance.now();
-    isMonotonicClock = previousTimestamp < currentTimestamp;
+    isMonotonicClock = previousTimestamp <= currentTimestamp;
     if (isMonotonicClock)
         setTimeout('checkTimestamp(currentTimestamp)', 0);
 }