Closed Bug 1693114 Opened 3 years ago Closed 3 years ago

[wpt-sync] Sync PR 27648 - [CompositeAfterPaint] Handle main thread scrolling change during scroll animation

Categories

(Core :: DOM: CSS Object Model, task, P4)

Product:
Core
Component:
DOM: CSS Object Model
Type:
task

Tracking

()

Status:
RESOLVED FIXED
Milestone:
88 Branch
Tracking Flags:
Tracking Status
firefox88 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

(Blocks 1 open bug,
URL
)

Details

(Whiteboard: [wptsync downstream])

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

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

b'Xianzhu Wang <wangxianzhu@chromium.org>' wrote:

[CompositeAfterPaint] Handle main thread scrolling change during scroll animation

In pre-CAP the situation is handled when composited scrolling layer
changes in CompositedLayerMapping during compositing assignments update.
Not sure how a mere main thread scrolling status change (e.g. adding
background-attachment:fixed to a composited scrolling scroller) is
handled, but a test for the situation passes.

For CAP, now check change of main thread scrolling status in
PaintPropertyTreeBuilder.

The original test for the situation has not been working since we
removed rounded corner as a blocker of composited scrolling. Fix and
modernize it and add wpt tests.

Bug: 1176543
Change-Id: I00595635a7b1a729ddd05173e3452e40c34e52b3
Reviewed-on: https://chromium-review.googlesource.com/2688140
WPT-Export-Revision: 423ab9564b74d0a431ff610f9992ed35de057f3c

Component: web-platform-tests → DOM: CSS Object Model
Product: Testing → Core

CI Results

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

Total 2 tests

Status Summary

Firefox

OK : 1
PASS: 1
FAIL: 1

Chrome

OK : 1
PASS: 1
FAIL: 1

Safari

OK : 1
PASS: 2

Links

Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base

Details

New Tests That Don't Pass

/css/cssom-view/add-background-attachment-fixed-during-smooth-scroll.html: FAIL (Chrome: FAIL, Safari: PASS)

Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/0380a910d868
[wpt PR 27648] - [CompositeAfterPaint] Handle main thread scrolling change during scroll animation, a=testonly
https://hg.mozilla.org/integration/autoland/rev/efdc0bea4b2c
[wpt PR 27648] - Update wpt metadata, a=testonly
Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/9014d174f4fd
[wpt PR 27648] - [CompositeAfterPaint] Handle main thread scrolling change during scroll animation, a=testonly
https://hg.mozilla.org/integration/autoland/rev/6ec6725023f0
[wpt PR 27648] - Update wpt metadata, a=testonly
Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/a41284bdc5b2
[wpt PR 27648] - [CompositeAfterPaint] Handle main thread scrolling change during scroll animation, a=testonly
https://hg.mozilla.org/integration/autoland/rev/52669c2361f9
[wpt PR 27648] - Update wpt metadata, a=testonly

https://hg.mozilla.org/mozilla-central/rev/a41284bdc5b2
https://hg.mozilla.org/mozilla-central/rev/52669c2361f9

Status: NEW → RESOLVED
Closed: 3 years ago
status-firefox88: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 88 Branch
You need to log in before you can comment on or make changes to this bug.