Closed Bug 1384836 Opened 2 years ago Closed 2 years ago

Rename MediaDecoderReaderWrapper

Categories

(Core :: Audio/Video: Playback, enhancement, P5)

enhancement

Tracking

()

RESOLVED FIXED
mozilla56
Tracking Status
firefox56 --- fixed

People

(Reporter: jwwang, Assigned: jwwang)

Details

Attachments

(1 file)

Since we don't have MediaDecoderReader anymore, MediaDecoderReaderWrapper should be renamed accordingly.
Assignee: nobody → jwwang
Priority: -- → P5
Attachment #8890721 - Flags: review?(kaku)
Comment on attachment 8890721 [details]
Bug 1384836 - rename MediaDecoderReaderWrapper to ReaderProxy.

https://reviewboard.mozilla.org/r/161922/#review167256

::: dom/media/MediaDecoderStateMachine.h:111
(Diff revision 1)
> -class MediaDecoderReaderWrapper;
> +class ReaderProxy;
>  class OutputStreamManager;

Maybe follow the alphabet order here.
Attachment #8890721 - Flags: review?(kaku) → review+
Thanks!
Pushed by jwwang@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/a9cb88990607
rename MediaDecoderReaderWrapper to ReaderProxy. r=kaku
https://hg.mozilla.org/mozilla-central/rev/a9cb88990607
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla56
You need to log in before you can comment on or make changes to this bug.