Let DecodedStream have a worker thread

RESOLVED FIXED in Firefox 43

Status

()

Core
Audio/Video: Playback
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: jwwang, Assigned: jwwang)

Tracking

unspecified
mozilla43
Points:
---

Firefox Tracking Flags

(firefox43 fixed)

Details

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(2 attachments)

(Assignee)

Description

3 years ago
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.
(Assignee)

Updated

3 years ago
Blocks: 1196112
(Assignee)

Comment 2

3 years ago
Created attachment 8650817 [details]
MozReview Request: Bug 1195632. Part 1 - Let DecodedStream have a worker thread and asset some funtions on the worker thread.

Bug 1195632. Part 1 - Let DecodedStream have a worker thread and asset some funtions on the worker thread.
Attachment #8650817 - Flags: review?(roc)
(Assignee)

Comment 3

3 years ago
Created 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.

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+
(Assignee)

Comment 6

3 years ago
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
Last Resolved: 3 years ago
status-firefox43: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla43
You need to log in before you can comment on or make changes to this bug.