2010-01-14 Ojan Vafai <ojan@chromium.org>
authorojan@chromium.org <ojan@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 15 Jan 2010 17:46:01 +0000 (17:46 +0000)
committerojan@chromium.org <ojan@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 15 Jan 2010 17:46:01 +0000 (17:46 +0000)
        Reviewed by Eric Seidel.

        animations/combo-transform-translate+scale.html failed on Snow Leopard Debug Bot
        https://bugs.webkit.org/show_bug.cgi?id=33461

        Make test half as long. Remove a setTimeout and tweak tolerances to
        reduce flakiness.

        * animations/combo-transform-translate+scale.html:

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

LayoutTests/ChangeLog
LayoutTests/animations/combo-transform-translate+scale-expected.txt
LayoutTests/animations/combo-transform-translate+scale.html

index a62b03c..8125c4f 100644 (file)
@@ -1,3 +1,15 @@
+2010-01-14  Ojan Vafai  <ojan@chromium.org>
+
+        Reviewed by Eric Seidel.
+
+        animations/combo-transform-translate+scale.html failed on Snow Leopard Debug Bot
+        https://bugs.webkit.org/show_bug.cgi?id=33461
+
+        Make test half as long. Remove a setTimeout and tweak tolerances to
+        reduce flakiness.
+
+        * animations/combo-transform-translate+scale.html:
+
 2010-01-14  Darin Fisher  <darin@chromium.org>
 
         Reviewed by Brady Eidson.
index 91c9d4f..65ec680 100644 (file)
@@ -1,2 +1,2 @@
-This test performs an animation of the translate() and scale() operators. It animates over 2 seconds. At 0.5 and 1 second it takes a snapshot and expects the result to be within range.
+This test performs an animation of the translate() and scale() operators. It animates over 1 seconds. At 0.25 and 0.5 seconds it takes a snapshot and expects the result to be within range.
 PASS
index 362b6e8..d330c14 100644 (file)
@@ -10,7 +10,7 @@
       height: 100px;
       width: 100px;
       background-color: blue;
-      -webkit-animation-duration: 2s;
+      -webkit-animation-duration: 1s;
       -webkit-animation-timing-function: linear;
       -webkit-animation-name: "anim";
     }
       layoutTestController.waitUntilDone();
     }
     
-    result = "PASS";
+    result = "";
 
     const expected =  [ [ 1.25,0,0,1.75,25,50 ],
                         [ 1.5,0,0,2.5,50,100 ] ];
-    const tolerance = [ 0.2,0.2,0.2,0.2,5,10 ];
+    const tolerance = [ 0.2,0.2,0.2,0.3,5,15 ];
     const prop = [ "a", "b", "c", "d", "e", "f" ];
     
     function isEqual(actual, desired, tolerance)
         return diff <= tolerance;
     }
     
+    var numSnapshots = 0;
+    
     function snapshot(which)
     {
-        if (result != "PASS")
-            return;
-        
         var t = new WebKitCSSMatrix(window.getComputedStyle(document.getElementById('box')).webkitTransform);
-        for (i = 0; i < 6; ++i)
-            if (!isEqual(t[prop[i]], expected[which][i], tolerance[i])) {
-                result = "FAIL('"+prop[i]+"' was:"+t[prop[i]]+", expected:"+expected[which][i]+")";
-                return;
-            }
+        for (i = 0; i < 6; ++i) {
+            if (!isEqual(t[prop[i]], expected[which][i], tolerance[i]))
+                result += "FAIL('"+prop[i]+"' was:"+t[prop[i]]+", expected:"+expected[which][i]+")";
+        }
+        
+        numSnapshots++;
+        if (numSnapshots == 2) {
+            document.getElementById('result').innerHTML = result || "PASS";
+            if (window.layoutTestController)
+                layoutTestController.notifyDone();
+        }
     }
 
     function start()
     {
-        setTimeout("snapshot(0)", 500);
-        setTimeout("snapshot(1)", 1000);
-        
-        window.setTimeout(function() {
-            document.getElementById('result').innerHTML = result;
-            if (window.layoutTestController)
-                layoutTestController.notifyDone();
-        }, 1100);
+        setTimeout("snapshot(0)", 250);
+        setTimeout("snapshot(1)", 500);
     }
     
     document.addEventListener('webkitAnimationStart', start, false);
@@ -68,8 +67,8 @@
   </script>
 </head>
 <body>
-This test performs an animation of the translate() and scale() operators. It animates over 2 seconds. 
-At 0.5 and 1 second it takes a snapshot and expects the result to be within range.
+This test performs an animation of the translate() and scale() operators. It animates over 1 seconds. 
+At 0.25 and 0.5 seconds it takes a snapshot and expects the result to be within range.
 <div id="box">
 </div>
 <div id="result">