Unreviewed, rolling out r239607.
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 5 Jan 2019 11:30:16 +0000 (11:30 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 5 Jan 2019 11:30:16 +0000 (11:30 +0000)
https://bugs.webkit.org/show_bug.cgi?id=193169

This change makes the test even more flaky (Requested by fredw
on #webkit).

Reverted changeset:

"Flaky Test: http/wpt/css/css-animations/start-
animation-001.html"
https://bugs.webkit.org/show_bug.cgi?id=190903
https://trac.webkit.org/changeset/239607

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

LayoutTests/ChangeLog
LayoutTests/http/wpt/css/css-animations/start-animation-001.html

index 14d08a0..1ce9b3a 100644 (file)
@@ -1,3 +1,18 @@
+2019-01-05  Commit Queue  <commit-queue@webkit.org>
+
+        Unreviewed, rolling out r239607.
+        https://bugs.webkit.org/show_bug.cgi?id=193169
+
+        This change makes the test even more flaky (Requested by fredw
+        on #webkit).
+
+        Reverted changeset:
+
+        "Flaky Test: http/wpt/css/css-animations/start-
+        animation-001.html"
+        https://bugs.webkit.org/show_bug.cgi?id=190903
+        https://trac.webkit.org/changeset/239607
+
 2019-01-04  Daniel Bates  <dabates@apple.com>
 
         REGRESSION (r238522): Erratic scrolling on Google flights search result page and vrbo.com
index fd058b9..116b4eb 100644 (file)
       }
     </style>
     <script>
-      function squareLeft(id) {
-          var element = document.getElementById(id);
-          if (id === "squareNonAccelerated")
-              return element.getBoundingClientRect().left;
-          var transform = window.getComputedStyle(element).getPropertyValue("transform");
-          var match = /matrix\(1, 0, 0, 1, (.*), 0\)/.exec(transform);
-          return match ? parseFloat(match[1]) : 0;
+      function runTest() {
+        if (!window.testRunner)
+          return;
+        // We wait a bit after the squares are moved behind the blue rectangle. For discontinuous
+        // transforms we have 5000px / 100 steps = 5px/step. Hence this happens after 20 steps i.e.
+        // 20/100*10s = 200ms.
+        testRunner.waitUntilDone();
+        setTimeout(() => { testRunner.notifyDone(); }, 300);
       }
-      function shouldBeHiddenByCoveringRect(id) {
-          var coveringRectLeft = document.getElementById("coveringRect").getBoundingClientRect().left;
-          return squareLeft(id) > coveringRectLeft;
-      }
-      // We wait until all the squares are moved behind the blue rectangle.
-      if (window.testRunner)
-          testRunner.waitUntilDone();
-      function step() {
-          if (["squareLinear", "squareSteps", "squareNonAccelerated"].every(shouldBeHiddenByCoveringRect)) {
-              if (window.testRunner)
-                  testRunner.notifyDone();
-              return;
-          }
-          window.requestAnimationFrame(step);
-      }
-      window.requestAnimationFrame(step);
     </script>
   </head>
-  <body>
+  <body onload="runTest()">
     <p>This test passes if green squares are moved behind the blue rectangle.</p>
     <div id="container">
       <div id="squareLinear"><tt>transform</tt> (linear)</div>