Closed Bug 1266304 Opened 4 years ago Closed 4 years ago

Merge MediaDecoderReader::AudioDataPromise and MediaDecoderReader::VideoDataPromise

Categories

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

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla48
Tracking Status
firefox48 --- fixed

People

(Reporter: jwwang, Assigned: jwwang)

Details

Attachments

(2 files)

They are typedeffed from the same thing. We can merge them into one type which would allow us to merge functions that only differ in {Audio,Video}DataPromise.
Assignee: nobody → jwwang
Comment on attachment 8744146 [details]
MozReview Request: Bug 1266304. Part 1 - Merge MediaDecoderReader::AudioDataPromise and MediaDecoderReader::VideoDataPromise. r=kaku.

https://reviewboard.mozilla.org/r/48313/#review45105
Attachment #8744146 - Flags: review?(tkuo) → review+
Attachment #8744147 - Flags: review?(tkuo) → review+
Comment on attachment 8744147 [details]
MozReview Request: Bug 1266304. Part 2 - remove unnecessary template parameters. r=kaku.

https://reviewboard.mozilla.org/r/48315/#review45107
Thanks!
https://hg.mozilla.org/mozilla-central/rev/1a8c3d77911d
https://hg.mozilla.org/mozilla-central/rev/b4d0ef7a0f1e
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla48
You need to log in before you can comment on or make changes to this bug.