Closed Bug 1195632 Opened 6 years ago Closed 6 years ago

Let DecodedStream have a worker thread

Categories

(Core :: Audio/Video: Playback, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla43
Tracking Status
firefox43 --- fixed

People

(Reporter: jwwang, Assigned: jwwang)

References

Details

Attachments

(2 files)

DecodedStream will be able to listen to push events of the media queues and call SendData() on its own without the help of MDSM. This change makes it more consistent with AudioSink.
Blocks: 1196112
Bug 1195632. Part 1 - Let DecodedStream have a worker thread and asset some funtions on the worker thread.
Attachment #8650817 - Flags: review?(roc)
Bug 1195632. Part 2 - Have DecodedStream listen to push events of the media queues and call SendData() on its own without the help of MDSM.
Attachment #8650818 - Flags: review?(roc)
Comment on attachment 8650817 [details]
MozReview Request: Bug 1195632. Part 1 - Let DecodedStream have a worker thread and asset some funtions on the worker thread.

https://reviewboard.mozilla.org/r/16713/#review15043

Ship It!
Attachment #8650817 - Flags: review?(roc) → review+
Comment on attachment 8650818 [details]
MozReview Request: Bug 1195632. Part 2 - Have DecodedStream listen to push events of the media queues and call SendData() on its own without the help of MDSM.

https://reviewboard.mozilla.org/r/16715/#review15045

Ship It!
Attachment #8650818 - Flags: review?(roc) → review+
Thanks for the review.
https://hg.mozilla.org/mozilla-central/rev/6521d38bea7a
https://hg.mozilla.org/mozilla-central/rev/d7f6ba119760
Assignee: nobody → jwwang
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla43
You need to log in before you can comment on or make changes to this bug.