Closed Bug 1370654 Opened 3 years ago Closed 3 years ago

nsExternalResourceMap does unnecessary hashtable lookups

Categories

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

enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla55
Tracking Status
firefox55 --- fixed

People

(Reporter: mats, Assigned: mats)

Details

(Keywords: perf)

Attachments

(1 file)

No description provided.
Attachment #8875462 - Flags: review?(nfroyd) → review+
Pushed by mpalmgren@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/d1a431f894c2
Use mPendingLoads.GetOrInsert() to avoid a second hashtable lookup for Put().  Remove a redundant Get() before Remove().  r=froydnj
https://hg.mozilla.org/mozilla-central/rev/d1a431f894c2
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.