Closed Bug 1182966 Opened 4 years ago Closed 4 years ago

Use nsTHashTable::Iterator in caps/

Categories

(Core :: Security: CAPS, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla42
Tracking Status
firefox42 --- fixed

People

(Reporter: njn, Assigned: mccr8)

References

Details

Attachments

(3 files)

Because iterators are so much nicer than enumerate functions.

There is one occurrence of EnumerateEntries() in layout/{style,svg}/ to be dealt with.
> There is one occurrence of EnumerateEntries() in layout/{style,svg}/ to be
> dealt with.

Sorry, that should be "caps/", not "layout/{style,svg}/".
Assignee: nobody → continuation
While I was here, I threw in a few minor cleanups. For some reason, I needed the .get() when assigning from a refptr to a comptr.
Attachment #8633498 - Flags: review?(mrbkap)
I'm not entirely sure what these do, but it seems like a bad idea to extend them.
Attachment #8633499 - Flags: review?(mrbkap)
Attachment #8633497 - Flags: review?(mrbkap) → review+
Attachment #8633498 - Flags: review?(mrbkap) → review+
Attachment #8633499 - Flags: review?(mrbkap) → review+
Thanks for the reviews.
Keywords: checkin-needed
You need to log in before you can comment on or make changes to this bug.