Closed Bug 1161946 Opened 4 years ago Closed 4 years ago

MainThreadMediaStreamListener should be notified just when the stream is finished

Categories

(Core :: Web Audio, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla41
Tracking Status
firefox40 --- affected
firefox41 --- fixed

People

(Reporter: baku, Assigned: baku)

Details

(Keywords: perf)

Attachments

(2 files)

Attached patch a.patchSplinter Review
No description provided.
Attachment #8601978 - Flags: review?(padenot)
Attached patch b.patchSplinter Review
This patch is useful in case we have some registration of listeners after that the MediaStream is finished. In this case, we dispatch a runnable. It doesn't seem this is needed for our mochitest but I don't know if this could be needed in the future or in some race condition.
Attachment #8602680 - Flags: review?(padenot)
Keywords: perf
Attachment #8601978 - Flags: review?(padenot) → review+
Attachment #8602680 - Flags: review?(padenot) → review+
baku - should this be uplifted to 40, or WONTFIX?
Flags: needinfo?(amarchesini)
I would say WONTFIX for 40.
Flags: needinfo?(amarchesini)
You need to log in before you can comment on or make changes to this bug.