[wpt-sync] Sync PR 18316 - [LayoutNG] Fix invalidation issue with grids.
Categories
(Core :: CSS Parsing and Computation, task, P4)
Tracking
()
Tracking | Status | |
---|---|---|
firefox70 | --- | fixed |
People
(Reporter: mozilla.org, Unassigned)
References
()
Details
(Whiteboard: [wptsync downstream])
Sync web-platform-tests PR 18316 into mozilla-central (this bug is closed when the sync is complete).
PR: https://github.com/web-platform-tests/wpt/pull/18316
Details from upstream follow.
Ian Kilpatrick <ikilpatrick@chromium.org> wrote:
[LayoutNG] Fix invalidation issue with grids.
with: https://chromium-review.googlesource.com/c/chromium/src/+/1554113
We introduced a new flag for SelfNeedsLayoutForAvailableSpace. For any
alignment change this was set to true for grid-items.SelfNeedsLayout reports true if this flag is set, or if the normal
SelfNeedsLayoutForStyle is true.This caused an issue where we weren't performing layout on the whole tree
however.If a child of the grid-item (whose alignment changed) was marked for
needing style-layout, it would try and mark its container chain for
layout, hit the grid item, and abort, as it already "needed" layout.Within NG we'd come across the grid-item, hit the cache, or perform
simplified layout, never clearing layout, having a bad ptr, etc.This is a conservative fix so that this can be merged.
Bug: 991446
Change-Id: I75edebc864eb470ebfad757966ddc0e0248de153
Reviewed-on: https://chromium-review.googlesource.com/1741439
WPT-Export-Revision: 96fca1607df8d79448b9961073cdf0db8f58a695
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Comment 1•5 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=6677181937fde535a30d452cb502da28cf48362e
Assignee | ||
Comment 2•5 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=ddee75096b71bfc7c7b0da69a0e6c10217d25313
Assignee | ||
Comment 3•5 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=658940bb6b5944683f70875ca25baa4c34fe41b1
Assignee | ||
Comment 4•5 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=d6d9981dab57d462b0ae4f891fd809c794d319f1
Assignee | ||
Comment 5•5 years ago
|
||
Ran 1 tests and 1 subtests OK : 1 PASS : 1
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/a167060670c3 [wpt PR 18316] - [LayoutNG] Fix invalidation issue with grids., a=testonly
Comment 7•5 years ago
|
||
bugherder |
Description
•