Closed
Bug 1172830
Opened 9 years ago
Closed 9 years ago
Move buffering check out of MediaDecoderStateMachine::UpdateRenderedVideoFrames()
Categories
(Core :: Audio/Video, defect)
Core
Audio/Video
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.
Assignee | ||
Updated•9 years ago
|
Summary: Move buffering check out of MediaDecoderStateMachine::AdvanceFrame() → Move buffering check out of MediaDecoderStateMachine::UpdateRenderedVideoFrames()
Assignee | ||
Comment 1•9 years ago
|
||
The bug would make it easier to extract the video rendering code to VideoSink.
Assignee | ||
Comment 2•9 years ago
|
||
Try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=6885e074dd98
Assignee | ||
Comment 3•9 years ago
|
||
Bug 1172830 - Move buffering check out of MediaDecoderStateMachine::UpdateRenderedVideoFrames().
Attachment #8660563 -
Flags: review?(cpearce)
Comment 4•9 years ago
|
||
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+
Assignee | ||
Comment 5•9 years ago
|
||
Thanks for the review.
https://hg.mozilla.org/mozilla-central/rev/9ad6f8144c5b
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox43:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla43
You need to log in
before you can comment on or make changes to this bug.
Description
•