Closed Bug 1274192 Opened 8 years ago Closed 8 years ago

Make the MediaDecoderReaderWrapper as a proxy of waiting media data form MediaDecoderReader and be able to change callbacks at runtime.

Categories

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

defect

Tracking

()

RESOLVED FIXED
mozilla49
Tracking Status
firefox49 --- fixed

People

(Reporter: kaku, Assigned: kaku)

References

Details

Attachments

(2 files)

This is a counter part of Bug 1266027. Make the WaitForData() operation into MediaDecoderReaderWrapper so that the MDSM and SeekTask could know if each other is "waiting" for data.
Assignee: nobody → kaku
Blocks: 1235301
See Also: → 1266027
Comment on attachment 8754647 [details]
MozReview Request: Bug 1274192 part 1 - make the MediaDecoderReaderWrapper as a proxy of WaitForData(); r=jwwang

https://reviewboard.mozilla.org/r/54110/#review50866
Attachment #8754647 - Flags: review?(jwwang) → review+
Comment on attachment 8754648 [details]
MozReview Request: Bug 1274192 part 2 - make MDSM and SeekTask to adopt new MediaDecoderReaderWrapper API; r=jwwang

https://reviewboard.mozilla.org/r/54112/#review50868
Attachment #8754648 - Flags: review?(jwwang) → review+
Comment on attachment 8754647 [details]
MozReview Request: Bug 1274192 part 1 - make the MediaDecoderReaderWrapper as a proxy of WaitForData(); r=jwwang

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/54110/diff/1-2/
Attachment #8754647 - Attachment description: MozReview Request: Bug 1274192 part 1 - make the MediaDecoderReaderWrapper as a proxy of WaitForData(); r?jwwang → MozReview Request: Bug 1274192 part 1 - make the MediaDecoderReaderWrapper as a proxy of WaitForData(); r=jwwang
Attachment #8754648 - Attachment description: MozReview Request: Bug 1274192 part 2 - make MDSM and SeekTask to adopt new MediaDecoderReaderWrapper API; r?jwwang → MozReview Request: Bug 1274192 part 2 - make MDSM and SeekTask to adopt new MediaDecoderReaderWrapper API; r=jwwang
Comment on attachment 8754648 [details]
MozReview Request: Bug 1274192 part 2 - make MDSM and SeekTask to adopt new MediaDecoderReaderWrapper API; r=jwwang

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/54112/diff/1-2/
Try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=271052df8fec
So far looks good, waiting for the Windows8.
Window8 is also passed. Thanks for the review!
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/62990f9e91aa
https://hg.mozilla.org/mozilla-central/rev/4df69b84d9e5
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla49
You need to log in before you can comment on or make changes to this bug.