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

Unified Diff: LayoutTests/web-animations-api/w3c/eased-keyframes.html

Issue 253963002: Web Animations API: Add layout tests for element.animate() keyframes (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Added comment Created 6 years, 6 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
Index: LayoutTests/web-animations-api/w3c/eased-keyframes.html
diff --git a/LayoutTests/web-animations-api/eased-keyframes.html b/LayoutTests/web-animations-api/w3c/eased-keyframes.html
similarity index 81%
copy from LayoutTests/web-animations-api/eased-keyframes.html
copy to LayoutTests/web-animations-api/w3c/eased-keyframes.html
index 902dbc767f2a10c51c9c61d53fcc74a2e7b92c42..d278b09f126f1332087e0ac11509b2f335ab9e99 100644
--- a/LayoutTests/web-animations-api/eased-keyframes.html
+++ b/LayoutTests/web-animations-api/w3c/eased-keyframes.html
@@ -1,22 +1,9 @@
<!DOCTYPE html>
-<script src="../resources/testharness.js"></script>
-<script src="../resources/testharnessreport.js"></script>
-<script src="w3c/resources/keyframes-test.js"></script>
+<script src="../../resources/testharness.js"></script>
+<script src="../../resources/testharnessreport.js"></script>
+<script src="resources/keyframes-test.js"></script>
<script>
-// FIXME: Remove the need for and use of this function.
-// Currently our animation timeline is not stable during page load script execution.
-// By deferring the tests to requestAnimationFrame() we can get a stable timeline and
-// avoid flaky test results.
-function testInRAF(testFunction, description, properties) {
- async_test(function(testHandle) {
- requestAnimationFrame(function() {
- testHandle.step(testFunction);
- testHandle.done();
- })
- }, description, properties);
-}
-
-testInRAF(function() {
+test(function() {
assertAnimationStyles([
{opacity: '0', left: '0px', easing: 'steps(2, start)'},
{opacity: '0.25', left: '25px'},
@@ -60,7 +47,7 @@ testInRAF(function() {
author: 'Alan Cutter',
});
-testInRAF(function() {
+test(function() {
assertAnimationStyles([
{opacity: '0', offset: 0, easing: 'steps(2, start)'},
{left: '0px', offset: 0},
@@ -84,7 +71,7 @@ testInRAF(function() {
author: 'Alan Cutter',
});
-testInRAF(function() {
+test(function() {
assertAnimationStyles([
{opacity: '0', left: '0px'},
{opacity: '0.5', left: '50px', easing: 'steps(2, start)'},

Powered by Google App Engine
This is Rietveld 408576698