1182408 - Use nsTHashTable::Iterator in extensions/

Status: RESOLVED FIXED

(Core :: Permission Manager, defect)

Description

[Nicholas Nethercote [inactive]]

Because iterators are so much nicer than enumerate functions.

There are seven occurrences of EnumerateEntries() in extensions/ to be dealt with.

Comment 1

[Birunthan Mohanathas [:poiru]]

Attachment: Part 1: Move permission key checks out of loop

Comment 2

[Birunthan Mohanathas [:poiru]]

Attachment: Part 1: Move permission key checks out of loop

Comment 3

[Birunthan Mohanathas [:poiru]]

Attachment: Part 2: Use nsTHashTable::Iterator in nsPermissionManager

Comment 4

[Birunthan Mohanathas [:poiru]]

Attachment: Part 3: Use nsTHashTable::Iterator in mozPersonalDictionary

Comment 5

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/f7eb7bc6a99c
https://hg.mozilla.org/integration/mozilla-inbound/rev/2c709a5c7aff
https://hg.mozilla.org/integration/mozilla-inbound/rev/5ebdd39f8789

Comment 6

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/f7eb7bc6a99c
https://hg.mozilla.org/mozilla-central/rev/2c709a5c7aff
https://hg.mozilla.org/mozilla-central/rev/5ebdd39f8789