Closed Bug 1635817 Opened 4 years ago Closed 4 years ago

[wpt-sync] Sync PR 23424 - Revert "[FlexNG] Make ComputeMinMaxSizes cache more precise."


(Testing :: web-platform-tests, task, P4)



(firefox78 fixed)

Tracking Status
firefox78 --- fixed


(Reporter: wpt-sync, Unassigned)




(Whiteboard: [wptsync downstream])

Sync web-platform-tests PR 23424 into mozilla-central (this bug is closed when the sync is complete).

Details from upstream follow.

Balazs Engedy <> wrote:

Revert "[FlexNG] Make ComputeMinMaxSizes cache more precise."

This reverts commit 044aeb3ed3364905cc185d67eb947c7ca3c13741.

Reason for revert: Use-of-uninitialized-value.

Original change's description:

[FlexNG] Make ComputeMinMaxSizes cache more precise.

Previously our cache was imprecise, we recalculate our min/max sizes
when we didn't strictly need to, as we didn't know if we'd have a
%-block-size replaced element beneath us.

This patch changes ComputeMinMaxSizes to return a MinMaxSizesResult
(instead of just MinMaxSizes). This contains an additional bit of
information to indicate if this sub-tree depended on the %-block-size
given to it.

For most things, this will return false. For legacy layout roots we
always assume that they have children which depend on the %-block-size
(and why we initialize:
intrinsic_logical_widths_depends_on_percentage_block_size_ to true).

Along with this change, the cache changes in the following way:

  • We store an addition bit of information on LayoutBox to store the bit
    of information from ComputeMinMaxSizes.
  • We calculate the %-block-size for our children up-front, this means
    each of the algorithms don't have to do this.
  • With these changes, the cache can now hit, when the %-block-size
    given to us changes.


Bug: 845235
Change-Id: I93504721dbdd13bef58917b1fab2cabbcd1ce51e
Commit-Queue: Ian Kilpatrick \<>
Reviewed-by: Christian Biesinger \<>
Reviewed-by: Morten Stenshorne \<>
Reviewed-by: Koji Ishii \<>
Cr-Commit-Position: refs/heads/master@{#765797},,,,

Change-Id: I9e5312e4a919fedbdc2cfd08e50a5f8b188e7a25
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 845235
Reviewed-by: Balazs Engedy \<>
Commit-Queue: Balazs Engedy \<>
Cr-Commit-Position: refs/heads/master@{#765919}

PR 23424 applied with additional changes from upstream: 76e181c868d50d7f9a3901af03249102451d5de7
Pushed by
[wpt PR 23424] - Revert "[FlexNG] Make ComputeMinMaxSizes cache more precise.", a=testonly
Test result changes from PR not available.
Pushed by
[wpt PR 23424] - Revert "[FlexNG] Make ComputeMinMaxSizes cache more precise.", a=testonly
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla78
You need to log in before you can comment on or make changes to this bug.