ImportManager::Get does unnecessary hashtable lookups

RESOLVED WONTFIX

Status

()

Core
DOM: Core & HTML
P2
normal
RESOLVED WONTFIX
2 months ago
2 months ago

People

(Reporter: Mats Palmgren (vacation - back in August), Assigned: Mats Palmgren (vacation - back in August))

Tracking

({perf})

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Comment hidden (empty)
Created attachment 8876527 [details] [diff] [review]
Use LookupForAdd instead of Get+Put to avoid an unnecessary hashtable lookup
Attachment #8876527 - Flags: review?(nfroyd)

Comment 2

2 months ago
Hmm, isn't this code going away?
Flags: needinfo?(bkelly)

Comment 3

2 months ago
Yes, I'm about to nuke it in bug 1197401.  I have r+ and am planning to push to inbound this morning.
Status: NEW → RESOLVED
Last Resolved: 2 months ago
Flags: needinfo?(bkelly)
Resolution: --- → WONTFIX
See Also: → bug 1197401
Attachment #8876527 - Flags: review?(nfroyd)
You need to log in before you can comment on or make changes to this bug.