1836089 - [wpt-sync] Sync PR 40327 - [scroll-animations] Respect fill behavior on compositor

Status: RESOLVED FIXED

(Testing :: web-platform-tests, task, P4)

Description

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

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

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

Robert Flack <[e-mail]> wrote:

[scroll-animations] Respect fill behavior on compositor

Previously once an animation produced a value we would apply that
value unconditionally to all pending trees. However, scroll
driven animations should only apply their values when active.

When a cc animation goes inactive we currently don't know what the underlying properties used to be before they were animated and
need a commit from the main thread. We should track the original
property values in the future but for now this fixes the visual
bug with occasional commits.

To ensure that time based animations work as before they are
modified to implicitly fill in the playback direction so that
they hold their final value until the next commit.

Bug:1445137
Change-Id: I32ccc18f78a167a9e89d2e6289471b24c5659c63
Reviewed-on: https://chromium-review.googlesource.com/4563314
WPT-Export-Revision: 9d3a10ac32d5e319062d7eb5fca20807277bee4f

Comment 1

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=fc6b53c31b9d27bfcc7e03d1928fdd6c70f0b7a4

Comment 2

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

CI Results

Ran 5 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI

Total 1 tests

Status Summary

Firefox

PASS: 1

Chrome

PASS: 1

Safari

PASS: 1

Links

Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/531576530d41
[wpt PR 40327] - [scroll-animations] Respect fill behavior on compositor, a=testonly

Comment 4

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/531576530d41