Closed Bug 1558442 Opened 5 years ago Closed 5 years ago

Use a weaker change hint for auto <-> non-auto offset changes.

Categories

(Core :: CSS Parsing and Computation, enhancement, P3)

enhancement

Tracking

()

RESOLVED FIXED
mozilla69
Tracking Status
firefox69 --- fixed

People

(Reporter: emilio, Assigned: emilio)

References

(Regressed 1 open bug)

Details

Attachments

(1 file)

If the change hints in bug 1558357 are right, they should hold as well for other positioned stuff.

Type: defect → enhancement

See the review comments in bug 1558442 to see why these should be correct.
Quoting:

I think it would be good to avoid passing ClearDescendantIntrinsics and
NeedDirtyReflow here, since those are the most expensive ones, and I think
they're pretty clearly not needed. I think ClearAncestorIntrinsics is also
clearly not needed, since I'm reasonably confident that absolutely positioned
kids don't contribute to their parent's intrinsic size.

I think the same thing applies to "slow" offset changes (when autoness changes),
since relative movement doesn't affect intrinsic sizes either.

Pushed by ealvarez@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/d82fddfaec91
Use a weaker change hint for auto -> non-auto offset changes. r=dbaron
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla69
Regressions: 1559570
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: