Title: [232780] trunk/LayoutTests
Revision
232780
Author
s...@apple.com
Date
2018-06-12 15:28:25 -0700 (Tue, 12 Jun 2018)

Log Message

Unreviewed followup for r232736
https://bugs.webkit.org/show_bug.cgi?id=185698

Consider the case when the SVG animation starts before requestAnimationFrame()
is called. Store the value of animItem and compare it later with the value
of animItem after detaching it from the animated animList.

* svg/dom/animated-tearoff-list-remove-target.html:

Modified Paths

Diff

Modified: trunk/LayoutTests/ChangeLog (232779 => 232780)


--- trunk/LayoutTests/ChangeLog	2018-06-12 22:10:01 UTC (rev 232779)
+++ trunk/LayoutTests/ChangeLog	2018-06-12 22:28:25 UTC (rev 232780)
@@ -1,3 +1,14 @@
+2018-06-12  Said Abou-Hallawa  <sabouhall...@apple.com>
+
+        Unreviewed followup for r232736
+        https://bugs.webkit.org/show_bug.cgi?id=185698
+
+        Consider the case when the SVG animation starts before requestAnimationFrame()
+        is called. Store the value of animItem and compare it later with the value
+        of animItem after detaching it from the animated animList.
+
+        * svg/dom/animated-tearoff-list-remove-target.html:
+
 2018-06-12  Ryan Haddad  <ryanhad...@apple.com>
 
         Mark inspector/console/webcore-logging.html as flaky on mac-wk2 debug.

Modified: trunk/LayoutTests/svg/dom/animated-tearoff-list-remove-target.html (232779 => 232780)


--- trunk/LayoutTests/svg/dom/animated-tearoff-list-remove-target.html	2018-06-12 22:10:01 UTC (rev 232779)
+++ trunk/LayoutTests/svg/dom/animated-tearoff-list-remove-target.html	2018-06-12 22:28:25 UTC (rev 232780)
@@ -33,6 +33,7 @@
                 // animItem is a reference (not a copy) to the first item in the x animated list.
                 var animList = tspanElement.x.animVal;
                 var animItem = animList.getItem(0);
+                var originalValue = animItem.valueAsString;
 
                 // Now change the target of the <animate> element to be the "second-tspan".
                 var animateElement = document.getElementById("animate");
@@ -42,7 +43,7 @@
                 // of an SVGLength. animItem should not be affected if garbage collection is forced.
                 gc();
                 window.requestAnimationFrame(() => {
-                    if (animItem.valueAsString == "0") {
+                    if (animItem.valueAsString == originalValue) {
                         document.querySelector("div").innerHTML += "<br><br>PASS."
                         document.querySelector("svg").remove();
                     }
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to