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)
Core
Audio/Video: Playback
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 | ||
Updated•8 years ago
|
Assignee | ||
Comment 1•8 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/54110/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/54110/
Attachment #8754647 -
Flags: review?(jwwang)
Attachment #8754648 -
Flags: review?(jwwang)
Assignee | ||
Comment 2•8 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/54112/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/54112/
Updated•8 years ago
|
Priority: -- → P2
Comment 3•8 years ago
|
||
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 4•8 years ago
|
||
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+
Assignee | ||
Comment 5•8 years ago
|
||
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
Assignee | ||
Comment 6•8 years ago
|
||
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/
Assignee | ||
Comment 7•8 years ago
|
||
Try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=271052df8fec So far looks good, waiting for the Windows8.
Assignee | ||
Comment 8•8 years ago
|
||
Window8 is also passed. Thanks for the review!
Assignee | ||
Updated•8 years ago
|
Keywords: checkin-needed
https://hg.mozilla.org/integration/mozilla-inbound/rev/62990f9e91aa https://hg.mozilla.org/integration/mozilla-inbound/rev/4df69b84d9e5
Keywords: checkin-needed
Comment 10•8 years ago
|
||
bugherder |
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.
Description
•