[macOS WK1] Layout Test webanimations/accelerated-transition-interrupted-on-composite...
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 4 Apr 2019 17:39:32 +0000 (17:39 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 4 Apr 2019 17:39:32 +0000 (17:39 +0000)
https://bugs.webkit.org/show_bug.cgi?id=189678
<rdar://problem/44621674>

Patch by Antoine Quint <graouts@apple.com> on 2019-04-04
Reviewed by Dean Jackson.

We make the test sturdier by waiting for two rAFs instead of just one to wait until animations have been updated (one rAF)
and accelerated animations have been committed (two rAFs).

We also use the "transitioncancel" event to determine the transition has been interrupted rather than using a rAF callback.

* webanimations/accelerated-transition-interrupted-on-composited-element.html:

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

LayoutTests/ChangeLog
LayoutTests/webanimations/accelerated-transition-interrupted-on-composited-element.html

index 62f8cf0..425155f 100644 (file)
@@ -1,3 +1,18 @@
+2019-04-04  Antoine Quint  <graouts@apple.com>
+
+        [macOS WK1] Layout Test webanimations/accelerated-transition-interrupted-on-composited-element.html is a flaky image failure
+        https://bugs.webkit.org/show_bug.cgi?id=189678
+        <rdar://problem/44621674>
+
+        Reviewed by Dean Jackson.
+
+        We make the test sturdier by waiting for two rAFs instead of just one to wait until animations have been updated (one rAF)
+        and accelerated animations have been committed (two rAFs).
+
+        We also use the "transitioncancel" event to determine the transition has been interrupted rather than using a rAF callback.
+
+        * webanimations/accelerated-transition-interrupted-on-composited-element.html:
+
 2019-04-04  Diego Pino Garcia  <dpino@igalia.com>
 
         [WPE] Unreviewed gardening
index cc05afd..51a1815 100644 (file)
@@ -1,4 +1,4 @@
-<!DOCTYPE html><!-- webkit-test-runner [ experimental:WebAnimationsCSSIntegrationEnabled=true ] -->
+<!DOCTYPE html>
 <body>
 <style>
 div {
@@ -18,10 +18,19 @@ const div = document.body.appendChild(document.createElement("div"));
 // Wait for a transition to start and abort it.
 div.addEventListener("transitionstart", event => {
     div.style.transition = "none";
-    if (window.testRunner)
-        requestAnimationFrame(() => testRunner.notifyDone());
 });
 
+// When notified the transition was aborted, finish the test once we've ensured that animations
+// have been updated (one rAF) and that accelerated animations will have been committed (two rAFs).
+div.addEventListener("transitioncancel", event => {
+    if (window.testRunner) {
+        requestAnimationFrame(() => {
+            requestAnimationFrame(() => testRunner.notifyDone());
+        });
+    }
+});
+
+
 // Initiate a transform transition.
 setTimeout(() => {
     div.style.transition = "transform 2s";