commit 038a0ab59cfd6cde67e3c917eaa869310cd339e7
author Anders Hartvoll Ruud <andruud@chromium.org> Fri Dec 02 00:35:21 2022
committer Blink WPT Bot <blink-w3c-test-autoroller@chromium.org> Fri Dec 02 00:51:27 2022
tree 184dc7a0e44b1d90887597e16b06c7bdb7db91ba
parent b2c7909ad889b2dda22a452e530043aef0ea98bc

[scroll-animations] Make animation-delay a shorthand behind a flag

Building on the new "alternative property" system, this CL makes
animation-delay a shorthand of animation-delay-start/end based on
the runtime flag CSSScrollTimeline.

Since animation-delay is no longer a longhand, the animation shorthand
must expand to animation-delay-start instead. In addition, the
animation shorthand needs to explicit set the animation-delay-end
property to its initial value, in order to maintain as much web compat
as possible. (Otherwise, had the animation shorthand *not* set
the animation-delay-end property, then we would have produced an
expansion which the animation-delay property (shorthand) couldn't
reproduce. This would cause gCS().animationDelay to typically be
an empty value after setting the animation shorthand!).

Note on WPTs: Since scroll-animation-1 still isn't a very mature spec,
I have tried to not "destructively" modify tests which expect
animation-delay to be a longhand. Instead this CL takes some failing
expectations where appropriate. This is to avoid unnecessary churn
in case the spec changes again.

Fixed: 1375994
Change-Id: I66c3c62bfd42d7bdd8d2e5b4748fd9b8943101a3
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4047793
Reviewed-by: Kevin Ellis <kevers@chromium.org>
Commit-Queue: Anders Hartvoll Ruud <andruud@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1078368}