Closed Bug 1888424 Opened 2 years ago Closed 1 year ago

[wpt-sync] Sync PR 45409 - Make Composited Animations Invalidate Paint Directly

Categories

(Core :: Web Painting, task, P4)

task

Tracking

()

RESOLVED FIXED
127 Branch
Tracking Status
firefox127 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

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

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

Claire Chambers <clchambers@microsoft.com> wrote:

Make Composited Animations Invalidate Paint Directly

The previous system of relying on synthetic style diffs to produce paint invalidation proved too unwieldly, and created issues where animation changes in the pending animation update would cause crashes in certain situations.

This CL completely removes CompositablePaintAnimationChanged and replaces it with an extra hook in post style update scope. Any animations that are compositor pending or have been canceled have their corresponding layout objects invalidated.

Help requested:

  • initial code review
  • I can verify that my clip-path-animation-stress.html test works in other directories, but when I put it in css3/masking/ external/wpt/css/css-masking/clip-path/animation the promise test API I have used doesn't run. My knowledge of the test infrastructure is lacking in this respect so advice is welcome.

Bug: 1246826

Change-Id: Ib071d09b337ec786188aa5323dd4e6a95cfb962f
Reviewed-on: https://chromium-review.googlesource.com/5384914
WPT-Export-Revision: 2da11981bc45d6fd0f624d10366429ebb17aed32

Component: web-platform-tests → Web Painting
Product: Testing → Core
Whiteboard: [wptsync downstream] → [wptsync downstream error]
Whiteboard: [wptsync downstream error] → [wptsync downstream]

CI Results

Ran 0 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

GitHub PR Head
GitHub PR Base

Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/ebf0214c1a94 [wpt PR 45409] - Make Composited Animations Invalidate Paint Directly, a=testonly
Status: NEW → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 127 Branch
You need to log in before you can comment on or make changes to this bug.