Closed Bug 1259235 Opened 4 years ago Closed 4 years ago

nsLayoutUtils::IsScrollFrameWithSnapping() is slow

Categories

(Core :: Layout, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla48
Tracking Status
firefox48 --- fixed

People

(Reporter: BenWa, Assigned: BenWa)

References

Details

(Keywords: perf)

Attachments

(1 file)

Artificial profile (by running the code 5,000 times in a loop):
https://cleopatra.io/#report=967e511f12d9514ec580b3e5bdef77d352de128d
Comment on attachment 8734090 [details]
MozReview Request: Bug 1259235 - Add IsScrollFrameWithSnapping to speed up event regions. r=mstange

https://reviewboard.mozilla.org/r/42085/#review38593

Looks fine to me.
Attachment #8734090 - Flags: review?(mstange) → review+
ni? for try results
Flags: needinfo?(bgirard)
https://hg.mozilla.org/mozilla-central/rev/1c71992ab98c
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla48
Flags: needinfo?(bgirard)
Assignee: nobody → bgirard
Keywords: perf
You need to log in before you can comment on or make changes to this bug.