Closed
Bug 1511484
Opened 6 years ago
Closed 5 years ago
[wpt-sync] Sync PR 14311 - Don't estimate the top of bottom-aligned OOFs as bottom.
Categories
(Core :: CSS Parsing and Computation, enhancement, P4)
Core
CSS Parsing and Computation
Tracking
()
RESOLVED
FIXED
mozilla66
Tracking | Status | |
---|---|---|
firefox66 | --- | fixed |
People
(Reporter: mozilla.org, Unassigned)
References
()
Details
(Whiteboard: [wptsync downstream])
Sync web-platform-tests PR 14311 into mozilla-central (this bug is closed when the sync is complete). PR: https://github.com/web-platform-tests/wpt/pull/14311 Details from upstream follow. Morten Stenshorne <mstensho@chromium.org> wrote: > Don't estimate the top of bottom-aligned OOFs as bottom. > > We'll incorrectly think that we get fragmented in that case, which will > lead to incorrect height, which will lead to incorrect top offset, > which will lead to incorrect fragmentation. > > This has probably been the cause for a flaky web test, that behaved > even worse in LayoutNG, because in NG we force legacy layout as soon as > fragmentation is involved (so when entering printing, we typically need > to rebuild the entire layout object tree, with no former height to base > the bottom estimate on (which is what saved us when NG is disabled - > ehm, at least most of the time)). > > Bug: 910300 > Change-Id: I8e152f77e9a94bb05e51f02a24be6ef5fe03c6f9 > Cq-Include-Trybots: luci.chromium.try:linux_layout_tests_layout_ng > > Reviewed-on: https://chromium-review.googlesource.com/1355920 > WPT-Export-Revision: e2e38838c32cc97786dcc9e5695874ab48ab25d8
Assignee | ||
Updated•6 years ago
|
Component: web-platform-tests → CSS Parsing and Computation
Product: Testing → Core
Assignee | ||
Comment 1•6 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=380ab984a49ae875fc5f20e640753cbc20870b27
Assignee | ||
Comment 2•6 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=acc68c221c408005474d3eda8bf5ee9bba09b95f
Assignee | ||
Comment 3•6 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=059166ead761e0a0f467e893658c9adc18fa0bee
Assignee | ||
Comment 4•6 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=3b933d5063066b9a45febaa006ccdba6e99af561
Assignee | ||
Comment 5•5 years ago
|
||
Ran 2 tests and 3 subtests OK : 1 PASS : 3 FAIL : 1 New tests that have failures or other problems: /css/css-break/block-end-aligned-abspos-with-overflow.html: FAIL
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/d8062a5a3bd5 [wpt PR 14311] - Don't estimate the top of bottom-aligned OOFs as bottom., a=testonly https://hg.mozilla.org/integration/mozilla-inbound/rev/9d7bbdbb5355 [wpt PR 14311] - Update wpt metadata, a=testonly
Comment 7•5 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/d8062a5a3bd5 https://hg.mozilla.org/mozilla-central/rev/9d7bbdbb5355
Status: NEW → RESOLVED
Closed: 5 years ago
status-firefox66:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla66
You need to log in
before you can comment on or make changes to this bug.
Description
•