1676784 - Implement animation-timeline property

Status: RESOLVED FIXED

(Core :: CSS Parsing and Computation, enhancement)

Description

[Hiroyuki Ikezoe (:hiro)]

animation-timeline-computed.html would be automated tests.

Comment 1

[Boris Chiou [:boris]]

Attachment: Bug 1676784 - Part 1: Add animation-timeline longhand property in style system.

This patch adds the animation-timeline longhand property. For
shorthand, we will do that in the next patch.

Comment 2

[Boris Chiou [:boris]]

Attachment: Bug 1676784 - Part 2: Add animation-timeline into animation shorthand.

We use the same rule as blink: serialize the animation-timeline only if
it is not the initial value.

Comment 3

[Boris Chiou [:boris]]

Attachment: Bug 1676784 - Part 3: Hook scroll-timeline rule into Cascade data.

We add the placeholder to keep the scroll-timeline rule, so layout code
can access it in the future (i.e. Bug 1676791).

Comment 4

[Boris Chiou [:boris]]

Reference of the proposal to update the syntax of animation-timeline: https://github.com/w3c/csswg-drafts/issues/6674

Comment 5

[Phabricator Automation]

Comment on attachment 9242631 [details]
Bug 1676784 - Part 3: Hook scroll-timeline rule into Cascade data.

Revision D126452 was moved to bug 1676791. Setting attachment 9242631 [details] to obsolete.

Comment 6

[Boris Chiou [:boris]]

I'd like to land these patches because these shouldn't have an impact on Nightly (the pref is off).

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/943f56defa61
Part 1: Add animation-timeline longhand property in style system. r=emilio
https://hg.mozilla.org/integration/autoland/rev/32bbcf7dd8be
Part 2: Add animation-timeline into animation shorthand. r=emilio

Comment 8

[Cristian Tuns]

https://hg.mozilla.org/mozilla-central/rev/943f56defa61
https://hg.mozilla.org/mozilla-central/rev/32bbcf7dd8be