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

        A few more flaky transitions/animations tests
        https://bugs.webkit.org/show_bug.cgi?id=33642

        Attempt to deflake a few more tests. Some of these are second
        attempts. The first attempt made it less flaky, but not 100%.
        Also, for a couple tests, make them shorter. They seemed
        unnecessarily long.

        * animations/change-one-anim-expected.txt:
        * animations/change-one-anim.html:
        * transitions/cancel-transition.html:
        * transitions/mask-transitions.html:
        * transitions/shorthand-border-transitions-expected.txt:
        * transitions/shorthand-border-transitions.html:

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

LayoutTests/ChangeLog
LayoutTests/animations/change-one-anim-expected.txt
LayoutTests/animations/change-one-anim.html
LayoutTests/transitions/cancel-transition.html
LayoutTests/transitions/mask-transitions.html
LayoutTests/transitions/shorthand-border-transitions-expected.txt
LayoutTests/transitions/shorthand-border-transitions.html

index b97f253..16e9074 100644 (file)
@@ -1,3 +1,22 @@
+2010-01-13  Ojan Vafai  <ojan@chromium.org>
+
+        Reviewed by Eric Seidel.
+
+        A few more flaky transitions/animations tests
+        https://bugs.webkit.org/show_bug.cgi?id=33642
+
+        Attempt to deflake a few more tests. Some of these are second
+        attempts. The first attempt made it less flaky, but not 100%.
+        Also, for a couple tests, make them shorter. They seemed
+        unnecessarily long.
+
+        * animations/change-one-anim-expected.txt:
+        * animations/change-one-anim.html:
+        * transitions/cancel-transition.html:
+        * transitions/mask-transitions.html:
+        * transitions/shorthand-border-transitions-expected.txt:
+        * transitions/shorthand-border-transitions.html:
+
 2010-01-14  Beth Dakin  <bdakin@apple.com>
 
         Reviewed by Sam Weinig.
index 1d47d94..e014d41 100644 (file)
@@ -1,4 +1,4 @@
-This test performs two animations, left and top. It animates over 1 second. At 0.5 second it removes the left animation and the top animation should continue from where it left off.
-PASS - "left" property for "box" element at 0.75s saw something close to: 150
-PASS - "top" property for "box" element at 0.75s saw something close to: 225
+This test performs two animations, left and top. It animates over 0.4 second. At 0.2 second it removes the left animation and the top animation should continue from where it left off.
+PASS - "left" property for "box" element at 0.3s saw something close to: 200
+PASS - "top" property for "box" element at 0.3s saw something close to: 300
 
index bdfe6d8..1b5588d 100644 (file)
@@ -8,11 +8,11 @@
     <style type="text/css" media="screen">
         #box {
             position: relative;
-            height: 200px;
-            width: 200px;
+            height: 20px;
+            width: 20px;
             background-color: #9bb;
             -webkit-animation-name: horiz, vert;
-            -webkit-animation-duration: 1s;
+            -webkit-animation-duration: 0.4s;
             -webkit-animation-iteration-count: infinite;
             -webkit-animation-direction: alternate;
             -webkit-animation-timing-function: linear;
          
          @-webkit-keyframes horiz {
             from { left: 0px; }
-            to { left: 300px; }
+            to { left: 400px; }
          }
          @-webkit-keyframes vert {
             from { top: 0px; }
-            to { top: 300px; }
+            to { top: 400px; }
          }
     </style>
     <script src="animation-test-helpers.js" type="text/javascript" charset="utf-8"></script>
@@ -32,8 +32,8 @@
         
         const expectedValues = [
           // [animation-name, time, element-id, property, expected-value, tolerance]
-          [null, 0.75, "box", "left", 150, 30],
-          [null, 0.75, "box", "top", 225, 30],
+          [null, 0.3, "box", "left", 200, 40],
+          [null, 0.3, "box", "top", 300, 40],
         ];
 
         function removeAnim()
         
         function setup()
         {
-            setTimeout("removeAnim()", 500);
+            setTimeout("removeAnim()", 200);
         }
 
         runAnimationTest(expectedValues, setup);
     </script>
 </head>
 <body>
-This test performs two animations, left and top. It animates over 1 second.
-At 0.5 second it removes the left animation and the top animation should continue
+This test performs two animations, left and top. It animates over 0.4 second.
+At 0.2 second it removes the left animation and the top animation should continue
 from where it left off.
 <div id="box">
 </div>
index a938076..83375ec 100644 (file)
@@ -50,8 +50,6 @@
 
         function isEqual(actual, desired, tolerance)
         {
-            if (tolerance == undefined || tolerance == 0)
-                tolerance = defaultTolerance;
             var diff = Math.abs(actual - desired);
             return diff < tolerance;
         }
@@ -64,7 +62,6 @@
         function restartTransition()
         {
             document.getElementById("container").className = "run";
-            setTimeout("check()", 250);
         }
 
         function check()
@@ -96,6 +93,7 @@
             document.getElementById("container").className = "run";
             setTimeout("cancelTransition()", 100);
             setTimeout("restartTransition()", 200);
+            setTimeout("check()", 500);
         }
     </script>
 <body onload="start()">
index ba9347f..1d088a0 100644 (file)
@@ -37,8 +37,8 @@
 
     const expectedValues = [
       // [time, element-id, property, expected-value, tolerance]
-      [0.5, 'box', '-webkit-mask-position', [10, 10], 2],
-      [0.5, 'box2', '-webkit-mask-size', [150, 150], 10],
+      [0.5, 'box', '-webkit-mask-position', [10, 10], 4],
+      [0.5, 'box2', '-webkit-mask-size', [150, 150], 20],
     ];
   
     function setupTest()
index 58d7062..7c35d92 100644 (file)
@@ -1,15 +1,15 @@
 Tests transitions of the border shorthand properties.
 
-PASS - "border-top-color" property for "box" element at 0.5s saw something close to: 128,0,128
-PASS - "border-right-color" property for "box" element at 0.5s saw something close to: 128,0,128
-PASS - "border-bottom-color" property for "box" element at 0.5s saw something close to: 128,0,128
-PASS - "border-left-color" property for "box" element at 0.5s saw something close to: 128,0,128
-PASS - "border-top-width" property for "box" element at 0.5s saw something close to: 10
-PASS - "border-right-width" property for "box" element at 0.5s saw something close to: 10
-PASS - "border-bottom-width" property for "box" element at 0.5s saw something close to: 10
-PASS - "border-left-width" property for "box" element at 0.5s saw something close to: 10
-PASS - "border-top-width" property for "box1" element at 0.5s saw something close to: 10
-PASS - "border-top-color" property for "box1" element at 0.5s saw something close to: 255,0,255
-PASS - "border-top-width" property for "box2" element at 0.5s saw something close to: 10
-PASS - "border-bottom-width" property for "box2" element at 0.5s saw something close to: 20
+PASS - "border-top-color" property for "box" element at 0.25s saw something close to: 128,0,128
+PASS - "border-right-color" property for "box" element at 0.25s saw something close to: 128,0,128
+PASS - "border-bottom-color" property for "box" element at 0.25s saw something close to: 128,0,128
+PASS - "border-left-color" property for "box" element at 0.25s saw something close to: 128,0,128
+PASS - "border-top-width" property for "box" element at 0.25s saw something close to: 10
+PASS - "border-right-width" property for "box" element at 0.25s saw something close to: 10
+PASS - "border-bottom-width" property for "box" element at 0.25s saw something close to: 10
+PASS - "border-left-width" property for "box" element at 0.25s saw something close to: 10
+PASS - "border-top-width" property for "box1" element at 0.25s saw something close to: 10
+PASS - "border-top-color" property for "box1" element at 0.25s saw something close to: 255,0,255
+PASS - "border-top-width" property for "box2" element at 0.25s saw something close to: 10
+PASS - "border-bottom-width" property for "box2" element at 0.25s saw something close to: 20
 
index d71c4a7..ef4b9c3 100644 (file)
@@ -12,7 +12,7 @@
       margin: 10px;
       background-color: gray;
       border: 0px solid rgb(0, 0, 0);
-      -webkit-transition: border 1s linear;
+      -webkit-transition: border 0.5s linear;
     }
     
     #box.final {
@@ -25,7 +25,7 @@
       margin: 10px;
       background-color: gray;
       border: 0px solid rgb(0, 0, 0);
-      -webkit-transition: border-width 1s linear;
+      -webkit-transition: border-width 0.5s linear;
     }
     
     #box1.final {
@@ -38,7 +38,7 @@
       margin: 10px;
       background-color: gray;
       border: 0px solid rgb(0, 0, 0);
-      -webkit-transition: border 1s linear;
+      -webkit-transition: border 0.5s linear;
     }
     
     #box2.final {
     const expectedValues = [
       // [time, element-id, property, expected-value, tolerance]
       // color and width of each side should be animating
-      [0.5, 'box', 'border-top-color', [128, 0, 128], 15],
-      [0.5, 'box', 'border-right-color', [128, 0, 128], 15],
-      [0.5, 'box', 'border-bottom-color', [128, 0, 128], 15],
-      [0.5, 'box', 'border-left-color', [128, 0, 128], 15],
-      [0.5, 'box', 'border-top-width', 10, 1],
-      [0.5, 'box', 'border-right-width', 10, 1],
-      [0.5, 'box', 'border-bottom-width', 10, 1],
-      [0.5, 'box', 'border-left-width', 10, 1],
+      [0.25, 'box', 'border-top-color', [128, 0, 128], 20],
+      [0.25, 'box', 'border-right-color', [128, 0, 128], 20],
+      [0.25, 'box', 'border-bottom-color', [128, 0, 128], 20],
+      [0.25, 'box', 'border-left-color', [128, 0, 128], 20],
+      [0.25, 'box', 'border-top-width', 10, 2],
+      [0.25, 'box', 'border-right-width', 10, 2],
+      [0.25, 'box', 'border-bottom-width', 10, 2],
+      [0.25, 'box', 'border-left-width', 10, 2],
 
       // only border-width should be animating
-      [0.5, 'box1', 'border-top-width', 10, 1],
-      [0.5, 'box1', 'border-top-color', [255, 0, 255], 0],  // initial value
+      [0.25, 'box1', 'border-top-width', 10, 2],
+      [0.25, 'box1', 'border-top-color', [255, 0, 255], 0],  // initial value
 
       // border-width should be animating
-      [0.5, 'box2', 'border-top-width', 10, 1],
-      [0.5, 'box2', 'border-bottom-width', 20, 2],
+      [0.25, 'box2', 'border-top-width', 10, 2],
+      [0.25, 'box2', 'border-bottom-width', 20, 4],
     ];
     
     function setupTest()