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

RESOLVED FIXED in Firefox 43

Status

()

Core
Audio/Video: Playback
P2
normal
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: jwwang, Assigned: jwwang)

Tracking

(Blocks: 1 bug)

unspecified
mozilla43
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox43 fixed)

Details

MozReview Requests

()

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

Attachments

(3 attachments)

(Assignee)

Description

2 years ago
Time to have just one MediaSink to rule them all for MDSM.
(Assignee)

Updated

2 years ago
Blocks: 1199098
Depends on: 1199155
Priority: -- → P2
(Assignee)

Updated

2 years ago
Blocks: 1202351
(Assignee)

Comment 2

2 years ago
Created attachment 8658004 [details]
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.
Attachment #8658004 - Flags: review?(roc)
(Assignee)

Comment 3

2 years ago
Created attachment 8658005 [details]
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.
Attachment #8658005 - Flags: review?(roc)
(Assignee)

Comment 4

2 years ago
Created attachment 8658006 [details]
MozReview Request: Bug 1199562. Part 3 - remove unused code.

Bug 1199562. Part 3 - remove unused code.
Attachment #8658006 - Flags: review?(roc)
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 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
Attachment #8658005 - Flags: review?(roc) → review+
https://hg.mozilla.org/mozilla-central/rev/311f27477629
https://hg.mozilla.org/mozilla-central/rev/fb545c690083
https://hg.mozilla.org/mozilla-central/rev/07ff9769e202
Assignee: nobody → jwwang
Status: NEW → RESOLVED
Last Resolved: 2 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.