1199562 - Unify AudioSink and DecodedStream so MDSM needs to care about only one consumer

Status: RESOLVED FIXED

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

Description

[JW Wang [:jwwang] [:jw_wang]]

Time to have just one MediaSink to rule them all for MDSM.

Comment 1

[JW Wang [:jwwang] [:jw_wang]]

Try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=0edcbcad1524

Comment 2

[JW Wang [:jwwang] [:jw_wang]]

Attachment: MozReview Request: Bug 1199562. Part 1 - rename mAudioSink to mMediaSink as well as related member names.

Bug 1199562. Part 1 - rename mAudioSink to mMediaSink as well as related member names.

Comment 3

[JW Wang [:jwwang] [:jw_wang]]

Attachment: MozReview Request: Bug 1199562. Part 2 - replace usage of mStreamSink with mMediaSink in most cases.

Bug 1199562. Part 2 - replace usage of mStreamSink with mMediaSink in most cases.

Comment 4

[JW Wang [:jwwang] [:jw_wang]]

Attachment: MozReview Request: Bug 1199562. Part 3 - remove unused code.

Bug 1199562. Part 3 - remove unused code.

Comment 5

[Robert O'Callahan (:roc) (email my personal email if necessary)]

Comment on attachment 8658004 [details]
MozReview Request: Bug 1199562. Part 1 - rename mAudioSink to mMediaSink as well as related member names.

https://reviewboard.mozilla.org/r/18457/#review16611

Comment 6

[Robert O'Callahan (:roc) (email my personal email if necessary)]

Comment on attachment 8658005 [details]
MozReview Request: Bug 1199562. Part 2 - replace usage of mStreamSink with mMediaSink in most cases.

https://reviewboard.mozilla.org/r/18459/#review16613

Comment 7

[Robert O'Callahan (:roc) (email my personal email if necessary)]

Comment on attachment 8658006 [details]
MozReview Request: Bug 1199562. Part 3 - remove unused code.

https://reviewboard.mozilla.org/r/18461/#review16615

Comment 8

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/311f27477629
https://hg.mozilla.org/integration/mozilla-inbound/rev/fb545c690083
https://hg.mozilla.org/integration/mozilla-inbound/rev/07ff9769e202

Comment 9

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/311f27477629
https://hg.mozilla.org/mozilla-central/rev/fb545c690083
https://hg.mozilla.org/mozilla-central/rev/07ff9769e202