Closed Bug 1172830 Opened 5 years ago Closed 5 years ago

Move buffering check out of MediaDecoderStateMachine::UpdateRenderedVideoFrames()

Categories

(Core :: Audio/Video, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla43
Tracking Status
firefox43 --- fixed

People

(Reporter: jwwang, Assigned: jwwang)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

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: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla43
You need to log in before you can comment on or make changes to this bug.