Closed Bug 1265315 Opened 4 years ago Closed 4 years ago

Remove dependency on MediaDecoderReader from SeekTask

Categories

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

defect

Tracking

()

RESOLVED FIXED
mozilla48
Tracking Status
firefox48 --- fixed

People

(Reporter: jwwang, Assigned: jwwang)

References

Details

Attachments

(2 files)

After fixing bug 1265311, SeekTask will be able to depend solely on MediaDecoderReaderWrapper without referencing MediaDecoderReader.
Assignee: nobody → jwwang
Depends on: 1265311
Attachment #8742382 - Flags: review?(tkuo) → review+
Comment on attachment 8742382 [details]
MozReview Request: Bug 1265315. Part 2 - rename mReaderWrapper to mReader for less verbose coding. r=kaku.

https://reviewboard.mozilla.org/r/47179/#review44033
Comment on attachment 8742381 [details]
MozReview Request: Bug 1265315. Part 1 - remove use of MediaDecoderReader from SeekTask. r=kaku.

https://reviewboard.mozilla.org/r/47177/#review44031

::: dom/media/SeekTask.h
(Diff revision 1)
>  
>  #ifndef SEEK_TASK_H
>  #define SEEK_TASK_H
>  
>  #include "mozilla/MozPromise.h"
> -#include "MediaDecoderReader.h"

I think we still need to include this header for the definitions of:
MediaDecoderReader::SeekPromise
MediaDecoderReader::AudioDataPromise
MediaDecoderReader::VideoDataPromise
MediaDecoderReader::WaitForDataPromise
Attachment #8742381 - Flags: review?(tkuo) → review+
https://reviewboard.mozilla.org/r/47177/#review44031

> I think we still need to include this header for the definitions of:
> MediaDecoderReader::SeekPromise
> MediaDecoderReader::AudioDataPromise
> MediaDecoderReader::VideoDataPromise
> MediaDecoderReader::WaitForDataPromise

Somehow I didn't get compile errors in non-unified build. However, it makes sense to include "MediaDecoderReader.h".
https://hg.mozilla.org/mozilla-central/rev/1ba90ff27748
https://hg.mozilla.org/mozilla-central/rev/0fcee6752011
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.