Update corresponding tests for CSS animations to make them consistent

This is a collection of fix-ups to the corresponding tests for CSS animations to apply the same sort of naming etc. that we have now applied to CSS transitions tests. bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1467344 gecko-commit: 877cdf25b483aaccf75c0ef6821194523dcce259 gecko-integration-branch: mozilla-inbound gecko-reviewers: hiro
ewilligers · Aug 18, 2018 · 6aef667 · 6aef667
1 parent 8ff1614
commit 6aef667
Show file tree

Hide file tree

Showing 18 changed files with 7 additions and 52 deletions.
diff --git a/...nimation-getComputedTiming.tentative.html → ...onEffect-getComputedTiming.tentative.html b/...nimation-getComputedTiming.tentative.html → ...onEffect-getComputedTiming.tentative.html
@@ -1,6 +1,6 @@
 <!doctype html>
 <meta charset=utf-8>
-<title>CSSAnimation.getComputedTiming()</title>
+<title>AnimationEffect.getComputedTiming() for CSS animations</title>
 <!-- TODO: Add a more specific link for this once it is specified. -->
 <link rel="help" href="https://drafts.csswg.org/css-animations-2/#cssanimation">
 <script src="/resources/testharness.js"></script>

diff --git a/css/css-animations/CSSAnimation-animationName.tentative.html b/css/css-animations/CSSAnimation-animationName.tentative.html
@@ -11,7 +11,6 @@
   to { left: 100px }
 }
 </style>
-<body>
 <div id="log"></div>
 <script>
 'use strict';
@@ -39,4 +38,3 @@
 }, 'Animation name with hex-escape');
 
 </script>
-</body>
diff --git a/css/css-animations/CSSAnimation-canceling.tentative.html b/css/css-animations/CSSAnimation-canceling.tentative.html
@@ -18,7 +18,6 @@
   to { margin-left: 200px }
 }
 </style>
-<body>
 <div id="log"></div>
 <script>
 'use strict';
@@ -195,5 +194,3 @@
    'descendants');
 
 </script>
-</body>
-</html>
diff --git a/css/css-animations/CSSAnimation-effect.tentative.html b/css/css-animations/CSSAnimation-effect.tentative.html
@@ -16,7 +16,6 @@
   }
 }
 </style>
-<body>
 <div id="log"></div>
 <script>
 'use strict';
@@ -129,4 +128,3 @@
    'it fires an animationstart event');
 
 </script>
-</body>
diff --git a/css/css-animations/CSSAnimation-finished.tentative.html b/css/css-animations/CSSAnimation-finished.tentative.html
@@ -12,7 +12,6 @@
 }
 @keyframes def {}
 </style>
-<body>
 <div id="log"></div>
 <script>
 'use strict';
@@ -84,4 +83,3 @@
 }, 'finished promise is not reset when animationPlayState is set to running');
 
 </script>
-</body>
diff --git a/css/css-animations/CSSAnimation-getCurrentTime.tentative.html b/css/css-animations/CSSAnimation-getCurrentTime.tentative.html
@@ -1,10 +1,11 @@
 <!doctype html>
-<html>
-<head>
 <meta charset=utf-8>
 <title>CSSAnimation.currentTime</title>
 <!-- TODO: Add a more specific link for this once it is specified. -->
 <link rel="help" href="https://drafts.csswg.org/css-animations-2/#cssanimation">
+<script src="/resources/testharness.js"></script>
+<script src="/resources/testharnessreport.js"></script>
+<script src="support/testcommon.js"></script>
 <style>
 
 .animated-div {
@@ -19,11 +20,6 @@
 }
 
 </style>
-<script src="/resources/testharness.js"></script>
-<script src="/resources/testharnessreport.js"></script>
-<script src="support/testcommon.js"></script>
-</head>
-<body>
 <div id="log"></div>
 <script type="text/javascript">
 
@@ -70,5 +66,3 @@
 }, 'Setting currentTime to null on a CSS animation throws');
 
 </script>
-</body>
-</html>
diff --git a/css/css-animations/CSSAnimation-id.tentative.html b/css/css-animations/CSSAnimation-id.tentative.html
@@ -9,7 +9,6 @@
 <style>
 @keyframes abc { }
 </style>
-<body>
 <div id="log"></div>
 <script>
 'use strict';
@@ -25,5 +24,4 @@
 }, 'Animation.id for CSS Animations');
 
 </script>
-</body>
 </html>
diff --git a/css/css-animations/CSSAnimation-pausing.tentative.html b/css/css-animations/CSSAnimation-pausing.tentative.html
@@ -12,7 +12,6 @@
   100% { margin-left: 10000px }
 }
 </style>
-<body>
 <div id="log"></div>
 <script>
 'use strict';
@@ -169,4 +168,3 @@
 }, 'Setting the current time completes a pending pause');
 
 </script>
-</body>
diff --git a/css/css-animations/CSSAnimation-playState.tentative.html b/css/css-animations/CSSAnimation-playState.tentative.html
@@ -9,7 +9,6 @@
 <style>
 @keyframes anim { }
 </style>
-<body>
 <div id="log"></div>
 <script>
 'use strict';
@@ -54,4 +53,3 @@
 }, 'Animation returns correct playState when canceled');
 
 </script>
-</body>
diff --git a/css/css-animations/CSSAnimation-ready.tentative.html b/css/css-animations/CSSAnimation-ready.tentative.html
@@ -11,7 +11,6 @@
   to { transform: translate(10px) }
 }
 </style>
-<body>
 <div id="log"></div>
 <script>
 'use strict';
@@ -97,4 +96,3 @@
 }, 'Pausing twice re-uses the same Promise');
 
 </script>
-</body>
diff --git a/css/css-animations/CSSAnimation-startTime.tentative.html b/css/css-animations/CSSAnimation-startTime.tentative.html
@@ -1,10 +1,11 @@
 <!doctype html>
-<html>
-<head>
 <meta charset=utf-8>
 <title>CSSAnimation.startTime</title>
 <!-- TODO: Add a more specific link for this once it is specified. -->
 <link rel="help" href="https://drafts.csswg.org/css-animations-2/#cssanimation">
+<script src="/resources/testharness.js"></script>
+<script src="/resources/testharnessreport.js"></script>
+<script src="support/testcommon.js"></script>
 <style>
 
 .animated-div {
@@ -19,11 +20,6 @@
 }
 
 </style>
-<script src="/resources/testharness.js"></script>
-<script src="/resources/testharnessreport.js"></script>
-<script src="support/testcommon.js"></script>
-</head>
-<body>
 <div id="log"></div>
 <script type="text/javascript">
 
@@ -71,5 +67,3 @@
 }, 'Seeking a CSS animation using the start time dispatches animation events');
 
 </script>
-</body>
-</html>
diff --git a/css/css-animations/CSSPseudoElement-getAnimations.tentative.html b/css/css-animations/CSSPseudoElement-getAnimations.tentative.html
@@ -26,7 +26,6 @@
   content: '';
 }
 </style>
-<body>
 <div id="log"></div>
 <script>
 'use strict';
@@ -74,4 +73,3 @@
    'animations in the expected order');
 
 </script>
-</body>
diff --git a/css/css-animations/Document-getAnimations.tentative.html b/css/css-animations/Document-getAnimations.tentative.html
@@ -25,7 +25,6 @@
   content: ''
 }
 </style>
-<body>
 <div id="log"></div>
 <script>
 'use strict';
@@ -283,4 +282,3 @@
    'after sorting');
 
 </script>
-</body>
diff --git a/css/css-animations/Element-getAnimations-dynamic-changes.tentative.html b/css/css-animations/Element-getAnimations-dynamic-changes.tentative.html
@@ -14,7 +14,6 @@
 }
 @keyframes anim2 { }
 </style>
-<body>
 <div id="log"></div>
 <script>
 'use strict';
@@ -162,4 +161,3 @@
 }, 'Animation state is preserved when interleaving animations in list');
 
 </script>
-</body>
diff --git a/css/css-animations/Element-getAnimations.tentative.html b/css/css-animations/Element-getAnimations.tentative.html
@@ -24,7 +24,6 @@
 }
 @keyframes empty { }
 </style>
-<body>
 <div id="log"></div>
 <script>
 'use strict';
@@ -446,4 +445,3 @@
    + ' from all the descendants');
 
 </script>
-</body>
diff --git a/css/css-animations/KeyframeEffect-target.tentative.html b/css/css-animations/KeyframeEffect-target.tentative.html
@@ -15,7 +15,6 @@
   content: ''
 }
 </style>
-<body>
 <div id="log"></div>
 <script>
 'use strict';
@@ -62,4 +61,3 @@
    'CSSPseudoElement object as that from the CSS generated animation');
 
 </script>
-</body>
diff --git a/css/css-animations/event-dispatch.tentative.html b/css/css-animations/event-dispatch.tentative.html
@@ -11,7 +11,6 @@
   to { margin-left: 100px; }
 }
 </style>
-<body>
 <div id="log"></div>
 <script>
 'use strict';
@@ -419,5 +418,3 @@
 }, 'Cancel the animation after clearing the target effect.');
 
 </script>
-</body>
-</html>
diff --git a/css/css-animations/event-order.tentative.html b/css/css-animations/event-order.tentative.html
@@ -11,7 +11,6 @@
     to { margin-left: 100px; }
   }
 </style>
-<body>
 <div id="log"></div>
 <script type='text/javascript'>
 'use strict';
@@ -160,5 +159,3 @@
 }, 'Test start and end events are sorted correctly when fired simultaneously');
 
 </script>
-</body>
-</html>