Closed
Bug 1504064
Opened 6 years ago
Closed 6 years ago
[wpt-sync] Sync PR 13843 - [LayoutNG] Use cached fragment if size won't change.
Categories
(Core :: CSS Parsing and Computation, enhancement, P4)
Core
CSS Parsing and Computation
Tracking
()
RESOLVED
FIXED
mozilla65
Tracking | Status | |
---|---|---|
firefox65 | --- | fixed |
People
(Reporter: mozilla.org, Unassigned)
References
()
Details
(Whiteboard: [wptsync downstream])
Sync web-platform-tests PR 13843 into mozilla-central (this bug is closed when the sync is complete). PR: https://github.com/web-platform-tests/wpt/pull/13843 Details from upstream follow. Morten Stenshorne <mstensho@chromium.org> wrote: > [LayoutNG] Use cached fragment if size won't change. > > Available size may change from one layout pass to another, but that > doesn't have to mean that we need to re-lay out a child. If we can be > sure that its size will remain the same, and that if it will keep the > same size, that won't affect any descendants, we can skip layout and > return the cached fragment. > > We cannot ignore initial containing block size changes if there are > orthogonal flow root children, so keep track of that. > > Makes perf_tests/layout/many-block-children-fixed-inline-size.html > about 3 times faster. > > Update expectations for one invalidation test. The test has some > fixed-width descendants, so when only the window width changes, > there's less work to do, than before. > > Update one unit test, and expand it a bit, to provide a case where we > actually still do need layout. > > Bug: 897450 > Change-Id: I063f650110d5a488de430b938b3d46e713a1433e > > Reviewed-on: https://chromium-review.googlesource.com/1297367 > WPT-Export-Revision: 0096b5dd7085f5375e4a042155c772359236df08
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=6bf3369594c61be94257be8c1cca145f94087e49
Assignee | ||
Comment 2•6 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=445931de48991911c24c9c38dfe256d7dbec3393
Assignee | ||
Comment 3•6 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=6c03635d6b7875ffebd71c428efb327662e43a25
Assignee | ||
Comment 4•6 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=196753fc68253aad5579e5e7c20a731727a81082
Assignee | ||
Comment 5•6 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=e20ea2e98aa054333075fc3be029fab245cc3187
Assignee | ||
Comment 6•6 years ago
|
||
Ran 8 tests and 8 subtests OK : 8 PASS : 8
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/1b54a404506c [wpt PR 13843] - [LayoutNG] Use cached fragment if size won't change., a=testonly
Comment 8•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/1b54a404506c
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox65:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla65
You need to log in
before you can comment on or make changes to this bug.
Description
•