[wpt-sync] Sync PR 28555 - [cleanup] Run ComputeMinMaxSizes / Layout on demand for OOF nodes.
Categories
(Core :: Layout, task, P4)
Tracking
()
Tracking | Status | |
---|---|---|
firefox90 | --- | fixed |
People
(Reporter: mozilla.org, Unassigned)
References
()
Details
(Whiteboard: [wptsync downstream])
Sync web-platform-tests PR 28555 into mozilla-central (this bug is closed when the sync is complete).
PR: https://github.com/web-platform-tests/wpt/pull/28555
Details from upstream follow.
b'Ian Kilpatrick <ikilpatrick@chromium.org>' wrote:
[cleanup] Run ComputeMinMaxSizes / Layout on demand for OOF nodes.
This performs a relatively large cleanup, but effectively this removes
a bunch of complicated logic for determining if we need to run
NGBlockNode::ComputeMinMaxSizes / Layout, and instead calculates
on-demand within ng_absolute_utils.ccThis is easier to read (IMO), and significantly reduces the API
surface.Change-Id: I674d4f15354097aebe8131e7f664b52a18ddf692
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2818969
Commit-Queue: Ian Kilpatrick \<ikilpatrick@chromium.org>
Reviewed-by: Morten Stenshorne \<mstensho@chromium.org>
Reviewed-by: Alison Maher \<almaher@microsoft.com>
Cr-Commit-Position: refs/heads/master@{#873363}
Assignee | ||
Comment 1•3 years ago
|
||
PR 28555 applied with additional changes from upstream: 312b5934720a19dd6209890f98f6a7894c0e09e5
Assignee | ||
Updated•3 years ago
|
Assignee | ||
Comment 2•3 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=10dc4c70e3b9b736f36d29e1eac971f4617b5081
Assignee | ||
Comment 3•3 years ago
|
||
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
PASS: 1
FAIL: 1
Safari
PASS: 2
Links
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/560d79b206ef [wpt PR 28555] - [cleanup] Run ComputeMinMaxSizes / Layout on demand for OOF nodes., a=testonly
Comment 5•3 years ago
|
||
bugherder |
Description
•