Move buffering check out of MediaDecoderStateMachine::UpdateRenderedVideoFrames()

RESOLVED FIXED in Firefox 43

Status

()

defect
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: jwwang, Assigned: jwwang)

Tracking

(Blocks 1 bug)

unspecified
mozilla43
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox43 fixed)

Details

Attachments

(1 attachment)

We should do it in the pop listeners and make AdvanceFrame() more coherent which should deal with video frame rendering and playback position updating only.
Summary: Move buffering check out of MediaDecoderStateMachine::AdvanceFrame() → Move buffering check out of MediaDecoderStateMachine::UpdateRenderedVideoFrames()
The bug would make it easier to extract the video rendering code to VideoSink.
Assignee: nobody → jwwang
Blocks: 1199098
Depends on: 1204401
Bug 1172830 - Move buffering check out of MediaDecoderStateMachine::UpdateRenderedVideoFrames().
Attachment #8660563 - Flags: review?(cpearce)
Comment on attachment 8660563 [details]
MozReview Request: Bug 1172830 - Move buffering check out of MediaDecoderStateMachine::UpdateRenderedVideoFrames().

https://reviewboard.mozilla.org/r/19141/#review17119
Attachment #8660563 - Flags: review?(cpearce) → review+
Thanks for the review.
https://hg.mozilla.org/mozilla-central/rev/9ad6f8144c5b
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla43
You need to log in before you can comment on or make changes to this bug.