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

Unified Diff: LayoutTests/web-animations-api/timeline-time.html

Issue 275863002: Revert of Web Animations: Timeline should not advance during task execution (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 7 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/timeline-time.html
diff --git a/LayoutTests/web-animations-api/timeline-time.html b/LayoutTests/web-animations-api/timeline-time.html
index 9c86843c31507f4ee5d51e3dd25c028f6b9d0fef..359e22662fd636f74fc37d8e8e7173f7386501d4 100644
--- a/LayoutTests/web-animations-api/timeline-time.html
+++ b/LayoutTests/web-animations-api/timeline-time.html
@@ -2,45 +2,25 @@
<script src="../resources/testharness.js"></script>
<script src="../resources/testharnessreport.js"></script>
+<body>
+ <div id='element'></div>
+</body>
+
<script>
+var element = document.getElementById('element');
+var duration = 1;
+
+var animation = new Animation(element,
+ [{opacity: '1', offset: 0},
+ {opacity: '0', offset: 1}],
+ duration);
+
test(function() {
- var startTime = document.timeline.currentTime;
- assert_greater_than_equal(startTime, 0);
- var start = performance.now();
- while (performance.now() < start + 250)
- /* wait */;
- assert_equals(document.timeline.currentTime, startTime);
+ var startTime = document.timeline.currentTime;
+ var player = document.timeline.play(animation);
+ player.finish();
+ var finishTime = document.timeline.currentTime;
+ assert_greater_than_equal(startTime, 0);
+ assert_equals(startTime, finishTime);
}, 'document.timeline.currentTime should not change within a script block.');
-
-(function() {
- var timeoutTest = async_test('document.timeline.currentTime time should change after a script timeout');
- var startTime = document.timeline.currentTime;
-
- setTimeout(function() {
- timeoutTest.step(function() {
- assert_greater_than(document.timeline.currentTime, startTime);
- });
- timeoutTest.done();
- }, 100);
-})();
-
-(function() {
- var rafTest = async_test('document.timeline.currentTime time should be the same for all RAF callbacks in a frame');
- var startTime = document.timeline.currentTime;
- var firstRafTime;
-
- requestAnimationFrame(function() {
- rafTest.step(function() {
- assert_greater_than_equal(document.timeline.currentTime, startTime);
- firstRafTime = document.timeline.currentTime;
- });
- });
-
- requestAnimationFrame(function() {
- rafTest.step(function() {
- assert_equals(document.timeline.currentTime, firstRafTime);
- });
- rafTest.done();
- });
-})();
</script>
« no previous file with comments | « LayoutTests/web-animations-api/element-animate-position-crash-expected.txt ('k') | Source/core/animation/AnimationClock.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698