1645450 - Do not search for current element in nsTObserverArray loops to remove the current element

Status: RESOLVED FIXED

(Core :: XPCOM, task)

Description

[Simon Giesecke [:sg] [he/him]]

... rather, the iterator's Remove member function can be used.

Comment 1

[Simon Giesecke [:sg] [he/him]]

Attachment: Bug 1645450 - Remove elements by index rather than searching in docshell. r=nika

Depends on D79512

Comment 2

[Simon Giesecke [:sg] [he/him]]

Attachment: Bug 1645450 - Remove elements by index rather than searching in dom/audiochannel. r=baku

Depends on D79513

Comment 3

[Simon Giesecke [:sg] [he/him]]

Attachment: Bug 1645450 - Remove elements by index rather than searching in dom/base. r=mccr8

Depends on D79514

Comment 4

[Simon Giesecke [:sg] [he/him]]

Attachment: Bug 1645450 - Use ForwardIterator::Remove in dom/debugger. r=jimb

Depends on D79515

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/20d28014ebc6
Remove elements by index rather than searching in docshell. r=nika
https://hg.mozilla.org/integration/autoland/rev/5790089fdc1d
Remove elements by index rather than searching in dom/audiochannel. r=baku
https://hg.mozilla.org/integration/autoland/rev/409f7c1ab3f3
Remove elements by index rather than searching in dom/base. r=mccr8

Comment 6

[Atila Butkovits]

https://hg.mozilla.org/mozilla-central/rev/20d28014ebc6
https://hg.mozilla.org/mozilla-central/rev/5790089fdc1d
https://hg.mozilla.org/mozilla-central/rev/409f7c1ab3f3

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/b40733a67099
Use ForwardIterator::Remove in dom/debugger. r=loganfsmyth

Comment 8

[Natalia Csoregi [:nataliaCs]]

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