[wpt-sync] Sync PR 16976 - [LayoutNG] Ensure that new formatting-context margins separates correctly.
Categories
(Core :: CSS Parsing and Computation, defect, P4)
Tracking
()
Tracking | Status | |
---|---|---|
firefox69 | --- | fixed |
People
(Reporter: mozilla.org, Unassigned)
References
()
Details
(Whiteboard: [wptsync upstream])
Sync web-platform-tests PR 16976 into mozilla-central (this bug is closed when the sync is complete).
PR: https://github.com/web-platform-tests/wpt/pull/16976
Details from upstream follow.
Ian Kilpatrick <ikilpatrick@chromium.org> wrote:
[LayoutNG] Ensure that new formatting-context margins separates correctly.
When positioning a new formatting-context in the presence of adjoining
floats sometimes a "forced" BFC block-offset will be set twice.When this new formatting-context was wrapped in a \<div> (a parent which
hadn't had its BFC block-offset resolved yet) we ran into a bug with our
state machine.Upon the second time resolving the BFC block-offset the parent would
resolve its BFC block-offset to the previous "forced" BFC
block-offset.Upon the final relayout the parent would set the first "forced" BFC
block-offset to its children causing an "infinite" loop (if we had a
loop driving this machinery.This ensures we use the childs new BFC block-offset in this case.
Bug: 966278, 966214
Change-Id: I5da2f3e90b9d2ff78f124247f6f68cd3a289ae42Reviewed-on: https://chromium-review.googlesource.com/1627692
WPT-Export-Revision: 5c7aa1a207b1eaf9be7873501a04475cd63a6873
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Comment 1•5 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=8486a7a2abe31c492cef1a5a59d9d721f0abbe6a
Assignee | ||
Comment 2•5 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=ae1c34e712754812ede36eb2d217183fbc3107fa
Assignee | ||
Comment 3•5 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=86d6c62ca6aab24e82a266e37a7e80625a5f050b
Assignee | ||
Comment 4•5 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=bb21a68c8e2145e96166c166fd5729adcb40ba37
Assignee | ||
Comment 5•5 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=58209a3294e988ce328483f3d0888a8bcbb0bb67
Assignee | ||
Comment 6•5 years ago
|
||
Failed to get results from try push
Assignee | ||
Comment 7•5 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=eeaad6c4e0cc7b594d11be8fdaa4aeccae976d30
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Updated•5 years ago
|
Pushed by james@hoppipolla.co.uk: https://hg.mozilla.org/integration/mozilla-inbound/rev/10dc40ccf61f [wpt PR 16976] - [LayoutNG] Ensure that new formatting-context margins separate correctly., a=testonly
Comment 9•5 years ago
|
||
bugherder |
Comment 10•5 years ago
|
||
bugherder |
Assignee | ||
Updated•5 years ago
|
Description
•