Closed
Bug 1161946
Opened 9 years ago
Closed 9 years ago
MainThreadMediaStreamListener should be notified just when the stream is finished
Categories
(Core :: Web Audio, defect)
Core
Web Audio
Tracking
()
RESOLVED
FIXED
mozilla41
People
(Reporter: baku, Assigned: baku)
Details
(Keywords: perf)
Attachments
(2 files)
17.85 KB,
patch
|
padenot
:
review+
|
Details | Diff | Splinter Review |
6.15 KB,
patch
|
padenot
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Attachment #8601978 -
Flags: review?(padenot)
Assignee | ||
Comment 1•9 years ago
|
||
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)
Updated•9 years ago
|
Attachment #8601978 -
Flags: review?(padenot) → review+
Updated•9 years ago
|
Attachment #8602680 -
Flags: review?(padenot) → review+
Assignee | ||
Comment 2•9 years ago
|
||
remote: https://hg.mozilla.org/integration/mozilla-inbound/rev/bad7af6079e9 remote: https://hg.mozilla.org/integration/mozilla-inbound/rev/4c5f304c7451
Comment 4•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/bad7af6079e9 https://hg.mozilla.org/mozilla-central/rev/4c5f304c7451 https://hg.mozilla.org/mozilla-central/rev/68d13dd7ae73
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox41:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla41
Comment 5•9 years ago
|
||
baku - should this be uplifted to 40, or WONTFIX?
Flags: needinfo?(amarchesini)
You need to log in
before you can comment on or make changes to this bug.
Description
•