1205351 - Replace nsBaseHashtable::Enumerate() calls in dom/media/ with iterators

Status: RESOLVED FIXED

(Core :: Audio/Video: Playback, defect, P2)

Description

[Sotaro Ikeda [:sotaro]]

+++ This bug was initially created as a clone of Bug #1186813 +++

Because iterators are so much nicer than enumerate functions.

There are nine occurrences of Enumerate() in this directory.

A note to the assignee: to preserve existing behaviour, you should probably use
nsBaseHashtable::Iterator::Data() rather than nsBaseHashtable::Iterator::UserData(). (The latter should be used when replacing nsBaseHashtable::EnumerateRead()).

Comment 1

[Sotaro Ikeda [:sotaro]]

Attachment: patch - Replace nsBaseHashtable::Enumerate() calls in dom/media/ with iterators

Comment 2

[Sotaro Ikeda [:sotaro]]

Attachment: patch - Replace nsBaseHashtable::Enumerate() calls in dom/media/ with iterators

Update nits.

Comment 3

[Sotaro Ikeda [:sotaro]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=5df5b6e62cc4

Comment 4

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/c64634725cac

Comment 5

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/c64634725cac