ImportManager::Get does unnecessary hashtable lookups

RESOLVED WONTFIX

Status

()

Core
DOM: Core & HTML
P2
normal
RESOLVED WONTFIX
13 days ago
12 days ago

People

(Reporter: mats, Assigned: mats)

Tracking

({perf})

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Comment hidden (empty)
(Assignee)

Comment 1

13 days ago
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

13 days ago
Hmm, isn't this code going away?
Flags: needinfo?(bkelly)
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: 12 days ago
Flags: needinfo?(bkelly)
Resolution: --- → WONTFIX
See Also: → bug 1197401
(Assignee)

Updated

12 days ago
Attachment #8876527 - Flags: review?(nfroyd)
You need to log in before you can comment on or make changes to this bug.