Closed
Bug 1182737
Opened 9 years ago
Closed 9 years ago
Align DecodedStream with MDSM::StartAudioThread/StopAudioThread
Categories
(Core :: Audio/Video, defect)
Core
Audio/Video
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.
Assignee | ||
Comment 1•9 years ago
|
||
Try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=32d86b16a1ee
Assignee | ||
Comment 2•9 years ago
|
||
Try again: https://treeherder.mozilla.org/#/jobs?repo=try&revision=2e0de5b03815
Assignee | ||
Comment 3•9 years ago
|
||
Try is green. https://treeherder.mozilla.org/#/jobs?repo=try&revision=2e0de5b03815 Android 2.3 API9 opt M9 oranges are not capture-stream related.
Assignee | ||
Comment 4•9 years ago
|
||
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)
Assignee | ||
Comment 5•9 years ago
|
||
Bug 1182737. Part 2 - remove unnecessary code.
Attachment #8632445 -
Flags: review?(roc)
Assignee | ||
Comment 6•9 years ago
|
||
Bug 1182737. Part 3 - make start/stop playback of DecodedStream more consistent with that of AudioSink.
Attachment #8632446 -
Flags: review?(roc)
Attachment #8632444 -
Flags: review?(roc) → review+
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+
Assignee | ||
Comment 10•9 years ago
|
||
Thanks for the review.
Comment 11•9 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/70e9ca580858 https://hg.mozilla.org/integration/mozilla-inbound/rev/24352a7d9b73 https://hg.mozilla.org/integration/mozilla-inbound/rev/ed6f4b96c7fe
Comment 12•9 years ago
|
||
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
status-firefox42:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla42
You need to log in
before you can comment on or make changes to this bug.
Description
•