Closed Bug 1573500 Opened 5 years ago Closed 5 years ago

[wpt-sync] Sync PR 18399 - Code health refactoring of animation.cc.

Categories

(Core :: DOM: Animation, task, P4)

Product:
Core
Component:
DOM: Animation
Type:
task

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla70
Tracking Flags:
Tracking Status
firefox70 --- fixed

People

(Reporter: mozilla.org, Unassigned)

References

(
URL
)

Details

(Whiteboard: [wptsync downstream])

Sync web-platform-tests PR 18399 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/18399
Details from upstream follow.

Kevin Ellis <kevers@chromium.org> wrote:

Code health refactoring of animation.cc.

This patch aligns the implementation of web-animations with the spec.
Previously updates were performed via a PlayStateUpdateScope. This
class depended on an extra state (kPending) that is not to spec.
Support for microtasks is also introduced in this patch, which
addresses most of the remaining timing issues with the finished state
of an animation.

Bug: 960944
Change-Id: I62aff2254d2b5265f869415e0f763e140b460c37
Reviewed-on: https://chromium-review.googlesource.com/1739386
WPT-Export-Revision: 5a0be4acf6f7927cb07628f35baffa664750f066

Component: web-platform-tests → DOM: Animation
Product: Testing → Core
Ran 1 tests and 5 subtests
OK     : 1
PASS   : 5
Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/299f19714e7c
[wpt PR 18399] - Code health refactoring of animation.cc., a=testonly

https://hg.mozilla.org/mozilla-central/rev/299f19714e7c

Status: NEW → RESOLVED
Closed: 5 years ago
status-firefox70: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla70
You need to log in before you can comment on or make changes to this bug.