Closed Bug 1198100 Opened 6 years ago Closed 6 years ago
Notification Main Thread Runnable
All callers of AddMainThreadListener make the call before the stream is even in the graph, well before mFinishedNotificationSent is set. If AddMainThreadListener() were called multiple times after mFinishedNotificationSent is set then we'd get some extra NotifyRunnables but NotifyMainThreadListeners() clears mMainThreadListeners anyway so we still get only one notification per listener. mNotificationMainThreadRunnable is an unnecessary optimization, so better not to add storage to every MediaStream. patch reviewed in https://bugzilla.mozilla.org/show_bug.cgi?id=1196632#c9
You need to log in before you can comment on or make changes to this bug.