1595618 - Optimization: Don't iterate all windows just to find SourceListeners on DeviceChange

Status: RESOLVED FIXED

(Core :: WebRTC: Audio/Video, enhancement, P3)

Description

[Jan-Ivar Bruaroey [:jib] (needinfo? me)]

From https://phabricator.services.mozilla.com/D52427#inline-319297:

I'm not sure whether you want to follow up with more work on this O(n^2) path here or in another patch, but I just had a thought about it and there is (plenty of) room for improvement.

When we reach here we have concluded that a device has been unplugged, and we want to stop the corresponding tracks for all SourceListeners that use them. So we iterate all windows because that would also cover all SourceListeners. (????!)

But but. We already know all SourceListeners -- they're in the GetUserMediaWindowListeners which are in mActiveWindows. Why don't we just iterate them instead? No checking of windows or docshells needed!

Comment 1

[Jan-Ivar Bruaroey [:jib] (needinfo? me)]

Attachment: Bug 1595618 - Optimize SourceListener iteration on devicechange.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/3b21c6bc1e1c
Optimize SourceListener iteration on devicechange. r=pehrsons

Comment 3

[Alexandru Michis [:malexandru]]

https://hg.mozilla.org/mozilla-central/rev/3b21c6bc1e1c