2009-03-11 Simon Fraser <simon.fraser@apple.com>
authorsimon.fraser@apple.com <simon.fraser@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 11 Mar 2009 19:45:18 +0000 (19:45 +0000)
committersimon.fraser@apple.com <simon.fraser@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 11 Mar 2009 19:45:18 +0000 (19:45 +0000)
        Reviewed by Dave Hyatt

        https://bugs.webkit.org/show_bug.cgi?id=24366

        Make this test more reliable by only calling runAnimationTest() once
        the animation starts, not before some setTimeout().

        * animations/change-keyframes-name-expected.txt:
        * animations/change-keyframes-name.html:

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

LayoutTests/ChangeLog
LayoutTests/animations/change-keyframes-name-expected.txt
LayoutTests/animations/change-keyframes-name.html

index d2c6e37..c3b6cb0 100644 (file)
@@ -2,6 +2,18 @@
 
         Reviewed by Dave Hyatt
         
+        https://bugs.webkit.org/show_bug.cgi?id=24366
+        
+        Make this test more reliable by only calling runAnimationTest() once
+        the animation starts, not before some setTimeout().
+
+        * animations/change-keyframes-name-expected.txt:
+        * animations/change-keyframes-name.html:
+
+2009-03-11  Simon Fraser  <simon.fraser@apple.com>
+
+        Reviewed by Dave Hyatt
+        
         https://bugs.webkit.org/show_bug.cgi?id=24266
         
         Land new expected results for transform-absolute-in-positioned-container.html,
index 2a0bbfd..dc5db47 100644 (file)
@@ -1,4 +1,4 @@
 This test starts by making sure the animation is not running, because the animation-name and the name of they @keyframes rule do not match. Then it changes the name of the @keyframes rule so they match and makes sure the animation is now running.
 PASS: animation is not running
-PASS - "left" property for "box" element at 0.45s saw something close to: 200
+PASS - "left" property for "box" element at 0.25s saw something close to: 200
 
index 45426ce..6a7b2de 100644 (file)
@@ -29,7 +29,7 @@
     
     const expectedValues = [
       // [animation-name, time, element-id, property, expected-value, tolerance]
-      [null, 0.2 + 0.25, "box", "left", 200, 10],
+      [null, 0.25, "box", "left", 200, 10],
     ];
 
     function findKeyframesRule(rule)
         var keyframes = findKeyframesRule("foo");
         keyframes.name = "anim";
         document.getElementById('box').style.webkitAnimationName = "anim";
+
+        runAnimationTest(expectedValues);
     }
     
     function setup()
     {
-        if (layoutTestController.pauseAnimationAtTimeOnElementWithId("bar", 0.5, "box"))
-            document.getElementById("pre-result").innerHTML = "FAIL: animation is running";
-        else
-            document.getElementById("pre-result").innerHTML = "PASS: animation is not running";
+        if (window.layoutTestController) {
+          layoutTestController.dumpAsText();
+          layoutTestController.waitUntilDone();
+          if (layoutTestController.pauseAnimationAtTimeOnElementWithId("bar", 0.5, "box"))
+              document.getElementById("pre-result").innerHTML = "FAIL: animation is running";
+          else
+              document.getElementById("pre-result").innerHTML = "PASS: animation is not running";
+        }
         
-        setTimeout("change()", 200);
+        setTimeout(change, 200);
     }
     
-    runAnimationTest(expectedValues, setup, "load");
-    
   </script>
 </head>
-<body>
+<body onload="setup()">
 This test starts by making sure the animation is not running, because the animation-name and the
 name of they @keyframes rule do not match. Then it changes the name of the @keyframes rule so they 
 match and makes sure the animation is now running.