Closed Bug 1416067 Opened 2 years ago Closed 2 years ago

Skip doing event-regions processing for frames with empty border boxes

Categories

(Core :: Layout, enhancement, P3)

Other Branch
All
Unspecified
enhancement

Tracking

()

RESOLVED FIXED
mozilla58
Tracking Status
firefox58 --- fixed

People

(Reporter: kats, Assigned: kats)

References

Details

(Keywords: perf)

Attachments

(1 file)

There are frames whose border boxes are empty. For those frames, the bottom half of nsDisplayLayerEventRegions::AddFrame will never do anything useful, it will just do a bunch of work and then add the empty border box to one or more regions. We can add an early-exit to avoid doing that work.
Comment on attachment 8927142 [details]
Bug 1416067 - Avoid doing a bunch of useless event-regions work for frames with empty border boxes.

https://reviewboard.mozilla.org/r/198392/#review203584
Attachment #8927142 - Flags: review?(mstange) → review+
Pushed by kgupta@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/e7f0ad152631
Avoid doing a bunch of useless event-regions work for frames with empty border boxes. r=mstange
https://hg.mozilla.org/mozilla-central/rev/e7f0ad152631
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla58
You need to log in before you can comment on or make changes to this bug.