Unreviewed followup for r232736
authorsaid@apple.com <said@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 12 Jun 2018 22:28:25 +0000 (22:28 +0000)
committersaid@apple.com <said@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 12 Jun 2018 22:28:25 +0000 (22:28 +0000)
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:

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

LayoutTests/ChangeLog
LayoutTests/svg/dom/animated-tearoff-list-remove-target.html

index 5426b59..667f067 100644 (file)
@@ -1,3 +1,14 @@
+2018-06-12  Said Abou-Hallawa  <sabouhallawa@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  <ryanhaddad@apple.com>
 
         Mark inspector/console/webcore-logging.html as flaky on mac-wk2 debug.
index b80513c..7ec717f 100644 (file)
@@ -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();
                     }