nsExternalResourceMap does unnecessary hashtable lookups

RESOLVED FIXED in Firefox 55

Status

()

enhancement
RESOLVED FIXED
2 years ago
3 months ago

People

(Reporter: mats, Assigned: mats)

Tracking

({perf})

Trunk
mozilla55
Points:
---

Firefox Tracking Flags

(firefox55 fixed)

Details

Attachments

(1 attachment)

Assignee

Description

2 years ago
No description provided.
Attachment #8875462 - Flags: review?(nfroyd) → review+

Comment 2

2 years ago
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

Comment 3

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/d1a431f894c2
Status: NEW → RESOLVED
Closed: 2 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.