Closed Bug 1753222 Opened 3 years ago Closed 3 years ago

[wpt-sync] Sync PR 32661 - [@container] Stop legacy propagation for CQ recalc container

Categories

(Core :: Layout, task, P4)

task

Tracking

()

RESOLVED FIXED
99 Branch
Tracking Status
firefox99 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

(Depends on 1 open bug, )

Details

(Whiteboard: [wptsync downstream])

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

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

Rune Lillesveen <futhark@chromium.org> wrote:

[@container] Stop legacy propagation for CQ recalc container

An element can be a container for size container queries even if size
containment does not apply, which meant the
IsContainerForContainerQueries check returns true for table type display
elements which meant we could end up with table layout trees which mixed
legacy and NG.

Instead check for the container being recalc'ed during layout
specifically to avoid detaching the layout boxes which had been laid
out.

Bug: 1291471
Change-Id: I6f7a65e77a90a09c320d75da5e836d7f8be9b86b
Reviewed-on: https://chromium-review.googlesource.com/3431427
WPT-Export-Revision: d1996b828f92ade19c187859dbb51f6b5d88c7f7

Component: web-platform-tests → Layout
Product: Testing → Core

CI Results

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

Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/ab7ed2a36b8b [wpt PR 32661] - [@container] Stop legacy propagation for CQ recalc container, a=testonly
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/c2285c249b06 [wpt PR 32661] - [@container] Stop legacy propagation for CQ recalc container, a=testonly
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 99 Branch
You need to log in before you can comment on or make changes to this bug.