tree b5ac49ea452dcd0c8f45003c881900fcb10574b6
parent 983fac69bdb2729ed018044df6a3bad80c6d6e81
author Jordan Taylor <jortaylo@microsoft.com> 1576529843 -0800
committer Blink WPT Bot <blink-w3c-test-autoroller@chromium.org> 1576530431 -0800

Removed exception for setting current time on a scroll linked animation

As far as I can tell, we are now ok to remove the exception that was
preventing current time from being set on animations using a
ScrollTimeline.

The change to hold_time_ in Animation::CommitPendingPlay is needed in
the case where the scroller has some non-zero offset and current time
has been set before play. If we set hold_time_ to 0 here, setting the
current_time on the animation has no effect, the current time from the
scroll timeline overrides it. This means that scroll-linked animations
are able to have a non-zero start_time.

I also had to add an additional condition to step 5 of the play
animation algorithm to prevent early abort when the above case occurs.

Added test for setting current time on scroll linked animation

Bug: 916117
Change-Id: I8017ce2f81496057cf5e4b8bb16b14707ee82484
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1945923
Commit-Queue: Jordan Taylor <jortaylo@microsoft.com>
Reviewed-by: Majid Valipour <majidvp@chromium.org>
Cr-Commit-Position: refs/heads/master@{#725247}