1157053 - Test restarting of finished transitions

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect)

Comment 1

[Jonathan Watt [:jwatt]]

Attachment: patch

Comment 2

[Brian Birtles (:birtles)]

Comment on attachment 8595655 [details] [diff] [review]
patch

Review of attachment 8595655 [details] [diff] [review]:
-----------------------------------------------------------------

I wonder if this should be in test_animation-finishing.html since it's not really testing the finished promise? Perhaps it should even be in the mozilla folder since it's really testing a Gecko-specific bug? Up to you.

Also, I don't know if we need to test the reversing case but it probably doesn't hurt.

r=me with comments addressed assuming you've tested that the test fails without the patch from bug 1149990 applied.

::: dom/animation/test/css-transitions/test_animation-finished.html
@@ +48,5 @@
> +    animation.playbackRate = -1;
> +    return animation.finished;
> +  }).then(t.step_func(function() {
> +    animation.play();
> +    assert_equals(animation.currentTime, ANIM_DELAY_MS + ANIM_DUR_MS,

Put a FIXME comment here that when animation.effect.computedTiming.endTime is available (bug 1108055) we should use that here?

@@ +50,5 @@
> +  }).then(t.step_func(function() {
> +    animation.play();
> +    assert_equals(animation.currentTime, ANIM_DELAY_MS + ANIM_DUR_MS,
> +                  'Replaying a finished reversed transition should reset ' +
> +                  'its currentTime to the animation duration');

Nit: Strictly speaking it's more the "end of the effect" rather than the animation duration.

Comment 3

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/f57c4c6a9ad7

Comment 4

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/f57c4c6a9ad7