Closed Bug 1182737 Opened 9 years ago Closed 9 years ago

Align DecodedStream with MDSM::StartAudioThread/StopAudioThread

Categories

(Core :: Audio/Video, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla42
Tracking Status
firefox42 --- fixed

People

(Reporter: jwwang, Assigned: jwwang)

References

Details

Attachments

(3 files)

Another step forward to unify DecodedStream and AudioSink which are both consumers of media data of MDSM.
Blocks: 1182738
Try is green. https://treeherder.mozilla.org/#/jobs?repo=try&revision=2e0de5b03815

Android 2.3 API9 opt M9 oranges are not capture-stream related.
Bug 1182737. Part 1 - have DecodedStream keep reference to media queues of MDSM so we don't have to pass them everytime.
Attachment #8632444 - Flags: review?(roc)
Bug 1182737. Part 2 - remove unnecessary code.
Attachment #8632445 - Flags: review?(roc)
Bug 1182737. Part 3 - make start/stop playback of DecodedStream more consistent with that of AudioSink.
Attachment #8632446 - Flags: review?(roc)
Comment on attachment 8632444 [details]
MozReview Request: Bug 1182737. Part 1 - have DecodedStream keep reference to media queues of MDSM so we don't have to pass them everytime.

https://reviewboard.mozilla.org/r/13091/#review11703

Ship It!
Comment on attachment 8632445 [details]
MozReview Request: Bug 1182737. Part 2 - remove unnecessary code.

https://reviewboard.mozilla.org/r/13093/#review11705

Ship It!
Attachment #8632445 - Flags: review?(roc) → review+
Comment on attachment 8632446 [details]
MozReview Request: Bug 1182737. Part 3 - make start/stop playback of DecodedStream more consistent with that of AudioSink.

https://reviewboard.mozilla.org/r/13095/#review11707

Ship It!
Attachment #8632446 - Flags: review?(roc) → review+
Thanks for the review.
https://hg.mozilla.org/mozilla-central/rev/70e9ca580858
https://hg.mozilla.org/mozilla-central/rev/24352a7d9b73
https://hg.mozilla.org/mozilla-central/rev/ed6f4b96c7fe
Assignee: nobody → jwwang
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla42
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: