Closed Bug 1712237 Opened 3 years ago Closed 3 years ago

[wpt-sync] Sync PR 29069 - [LayoutNG] OOF contained by inner multicol

Categories

(Core :: Layout, task, P4)

task

Tracking

()

RESOLVED FIXED
90 Branch
Tracking Status
firefox90 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

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

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

b'Alison Maher <almaher@microsoft.com>' wrote:

[LayoutNG] OOF contained by inner multicol

An OOF contained by an inner multicol was not getting laid out because
we skipped propagating up out-of-flow descendants when we reached
an inner multicol (because if the containing block was somewhere inside
the inner multicol, we would lay out those OOFs once we reached the
outer multicol). However, we should not skip propagating the OOF
descendant if the inner multicol is the containing block.

Bug: 1211522
Change-Id: I00876c44eeaa415edd6a90d1d29712077a2ce24c
Reviewed-on: https://chromium-review.googlesource.com/2911733
WPT-Export-Revision: 4bd121be2eb0e3c1a465c1e29d2c3dd67cf50a73

Component: web-platform-tests → Layout
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

PASS: 2

Chrome

FAIL: 2

Safari

PASS: 2

Links

Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base

Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/e58cca48d9c7 [wpt PR 29069] - [LayoutNG] OOF contained by inner multicol, a=testonly
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/13d7f561200c [wpt PR 29069] - [LayoutNG] OOF contained by inner multicol, a=testonly
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/9bf05a67ece5 [wpt PR 29069] - [LayoutNG] OOF contained by inner multicol, a=testonly
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 90 Branch
You need to log in before you can comment on or make changes to this bug.