Closed Bug 1607837 Opened 4 years ago Closed 4 years ago

[wpt-sync] Sync PR 21092 - Schedule frame for scroll linked animations only when scrolling changes.

Categories

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

task

Tracking

(firefox74 fixed)

RESOLVED FIXED
mozilla74
Tracking Status
firefox74 --- fixed

People

(Reporter: mozilla.org, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

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

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

Olga Gerchikov <gerchiko@microsoft.com> wrote:

Schedule frame for scroll linked animations only when scrolling changes.

Scheduling frame for scroll linked animations regardless of the scroller
updates makes scroll animations inefficient. This change schedules next
frame only when scroll timeline current time changes.

Bug: 1039750
Change-Id: Ib87a298e6d09cbeb85276cea4c165022b76f3732

Reviewed-on: https://chromium-review.googlesource.com/1990003
WPT-Export-Revision: 86f1a1d2f1b163bb6cd5a00dba9ad97f7c92c761

CI Results

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

Total 1 tests and 3 subtests

Status Summary

Firefox

OK : 1
FAIL: 3

Chrome

OK : 1
PASS: 3

Safari

OK : 1
FAIL: 3

Links

GitHub PR Head
GitHub PR Base

Details

New Tests That Don't Pass

/scroll-animations/scroll-animation.html
Animation start and current times are correct for each animation state when the animation starts playing with advanced scroller.: FAIL (Chrome: PASS, Safari: FAIL)
Animation start and current times are correct when multiple animations are attached to the same timeline.: FAIL (Chrome: PASS, Safari: FAIL)
Animation start and current times are correct for each animation state.: FAIL (Chrome: PASS, Safari: FAIL)

Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/9f0fe00eb92a
[wpt PR 21092] - Schedule frame for scroll linked animations only when scrolling changes, a=testonly
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla74
You need to log in before you can comment on or make changes to this bug.