[SCM] WebKit Debian packaging branch, webkit-1.2, updated. upstream/1.1.90-6072-g9a69373
ojan at chromium.org
ojan at chromium.org
Thu Apr 8 01:09:06 UTC 2010
The following commit has been merged in the webkit-1.2 branch:
commit d8347eba08df2787e6f537d775a1601159495632
Author: ojan at chromium.org <ojan at chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Date: Fri Jan 15 01:22:54 2010 +0000
2010-01-13 Ojan Vafai <ojan at 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:
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@53302 268f45cc-cd09-0410-ab3c-d52691b4dbfc
diff --git a/LayoutTests/ChangeLog b/LayoutTests/ChangeLog
index b97f253..16e9074 100644
--- a/LayoutTests/ChangeLog
+++ b/LayoutTests/ChangeLog
@@ -1,3 +1,22 @@
+2010-01-13 Ojan Vafai <ojan at 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 at apple.com>
Reviewed by Sam Weinig.
diff --git a/LayoutTests/animations/change-one-anim-expected.txt b/LayoutTests/animations/change-one-anim-expected.txt
index 1d47d94..e014d41 100644
--- a/LayoutTests/animations/change-one-anim-expected.txt
+++ b/LayoutTests/animations/change-one-anim-expected.txt
@@ -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
diff --git a/LayoutTests/animations/change-one-anim.html b/LayoutTests/animations/change-one-anim.html
index bdfe6d8..1b5588d 100644
--- a/LayoutTests/animations/change-one-anim.html
+++ b/LayoutTests/animations/change-one-anim.html
@@ -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;
@@ -20,11 +20,11 @@
@-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()
@@ -46,15 +46,15 @@
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>
diff --git a/LayoutTests/transitions/cancel-transition.html b/LayoutTests/transitions/cancel-transition.html
index a938076..83375ec 100644
--- a/LayoutTests/transitions/cancel-transition.html
+++ b/LayoutTests/transitions/cancel-transition.html
@@ -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()">
diff --git a/LayoutTests/transitions/mask-transitions.html b/LayoutTests/transitions/mask-transitions.html
index ba9347f..1d088a0 100644
--- a/LayoutTests/transitions/mask-transitions.html
+++ b/LayoutTests/transitions/mask-transitions.html
@@ -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()
diff --git a/LayoutTests/transitions/shorthand-border-transitions-expected.txt b/LayoutTests/transitions/shorthand-border-transitions-expected.txt
index 58d7062..7c35d92 100644
--- a/LayoutTests/transitions/shorthand-border-transitions-expected.txt
+++ b/LayoutTests/transitions/shorthand-border-transitions-expected.txt
@@ -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
diff --git a/LayoutTests/transitions/shorthand-border-transitions.html b/LayoutTests/transitions/shorthand-border-transitions.html
index d71c4a7..ef4b9c3 100644
--- a/LayoutTests/transitions/shorthand-border-transitions.html
+++ b/LayoutTests/transitions/shorthand-border-transitions.html
@@ -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 {
@@ -52,22 +52,22 @@
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()
--
WebKit Debian packaging
More information about the Pkg-webkit-commits
mailing list