Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(530)

Unified Diff: LayoutTests/animations/some-keyframes-empty.html

Issue 23875044: Web Animations: Correctly handle incomplete keyframes in CSS animations (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Handle end keyframes first Created 7 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | LayoutTests/animations/some-keyframes-empty-expected.txt » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: LayoutTests/animations/some-keyframes-empty.html
diff --git a/LayoutTests/animations/missing-keyframe-properties-repeating.html b/LayoutTests/animations/some-keyframes-empty.html
similarity index 57%
copy from LayoutTests/animations/missing-keyframe-properties-repeating.html
copy to LayoutTests/animations/some-keyframes-empty.html
index 5dc83d47decb802f92d01f2e5aa87d6a7e19ec82..ba84ee3e24d9334cc4b0406594361f2e7767e0e0 100644
--- a/LayoutTests/animations/missing-keyframe-properties-repeating.html
+++ b/LayoutTests/animations/some-keyframes-empty.html
@@ -2,50 +2,45 @@
<html>
<head>
<style type="text/css" media="screen">
- #box1 {
+ #test {
position: relative;
- height: 100px;
- width: 100px;
+ margin: 10px;
left: 0;
+ width: 100px;
+ height: 100px;
background-color: blue;
- margin: 0;
-webkit-animation-duration: 2s;
- -webkit-animation-timing-function: linear;
- -webkit-animation-iteration-count: 2;
- -webkit-animation-direction: alternate;
- -webkit-animation-name: move;
+ -webkit-animation-name: test;
}
- @-webkit-keyframes move {
+ @-webkit-keyframes test {
from {
- opacity: 1;
+ left: 100px;
+ }
+ 25% {
+ -webkit-animation-timing-function: linear;
}
50% {
- opacity: 1;
- left: 100px;
}
to {
- opacity: 1;
+ left: 200px;
}
}
-
- /* Test reversing animation with missing values */
-
</style>
<script src="resources/animation-test-helpers.js" type="text/javascript" charset="utf-8"></script>
<script type="text/javascript" charset="utf-8">
-
+
const expectedValues = [
// [animation-name, time, element-id, property, expected-value, tolerance]
- ["move", 1, "box1", "left", 100, 15],
- ["move", 2, "box1", "left", 0, 15],
- ["move", 3, "box1", "left", 100, 15],
+ ["test", 0.25, "test", "left", 113, 15],
+ ["test", 0.5, "test", "left", 132, 15],
];
-
+
runAnimationTest(expectedValues);
</script>
</head>
<body>
-<div id="box1"></div>
+<p>Tests that when an animation uses keyframes which contains no properties, those keyframes are ignored.</p>
+<div id="test"></div>
<div id="result">
</div>
</body>
« no previous file with comments | « no previous file | LayoutTests/animations/some-keyframes-empty-expected.txt » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698