Closed Bug 1868549 Opened 1 years ago Closed 1 years ago

[wpt-sync] Sync PR 43535 - Avoid bad relayout roots inside pagination.

Categories

(Core :: Layout, task, P4)

task

Tracking

()

RESOLVED FIXED
122 Branch
Tracking Status
firefox122 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

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

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

Morten Stenshorne <mstensho@chromium.org> wrote:

Avoid bad relayout roots inside pagination.

We were already preventing this from happening when inside multicol, but
it turns out that we need it for printing as well.

It is actually (at least currently) possible for the print layout to be
updated after initial layout for pagination but before painting
(printing) it. Accessibility may trigger this via Document::
AddAXContext(), which may be called at such an inconvenient time, and it
may trigger a style recalc, and thus layout. Due to crbug.com/452672, it
may also happen inside a window.matchMedia("print") listener.

Bug: 1507840
Change-Id: I213efc610c2b57fc3beceef99e5c52c8655bc483
Reviewed-on: https://chromium-review.googlesource.com/5093875
WPT-Export-Revision: e54e5aa9cf79f880e808a4711278afb5cc4a7f8a

Component: web-platform-tests → Layout
Product: Testing → Core
Status: NEW → RESOLVED
Closed: 1 years ago
Resolution: --- → INVALID
Status: RESOLVED → REOPENED
Resolution: INVALID → ---

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/92094b790da6 [wpt PR 43535] - Avoid bad relayout roots inside pagination., a=testonly
Status: REOPENED → RESOLVED
Closed: 1 years ago1 years ago
Resolution: --- → FIXED
Target Milestone: --- → 122 Branch
You need to log in before you can comment on or make changes to this bug.