Closed Bug 1887621 Opened 3 months ago Closed 3 months ago

Micro-optimize nsFocusManager::ContentRemoved

Categories

(Core :: DOM: Core & HTML, enhancement, P3)

enhancement

Tracking

()

RESOLVED FIXED
126 Branch
Tracking Status
firefox126 --- fixed

People

(Reporter: smaug, Assigned: smaug)

References

(Blocks 2 open bugs)

Details

(Whiteboard: [sp3])

Attachments

(1 file)

nsFocusManager::ContentRemoved does some refcounting even before the most common early return. It should be safe to not have those RefPtrs.
nsFocusManager::ContentRemoved is very hot code, so even small things matter. Bug 1815846 has some other ideas.

Pushed by opettay@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/1afb5ba97217
Micro-optimize nsFocusManager::ContentRemoved, r=edgar
Status: NEW → RESOLVED
Closed: 3 months ago
Resolution: --- → FIXED
Target Milestone: --- → 126 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: