Closed Bug 998284 Opened 10 years ago Closed 10 years ago

MediaDecoderStateMachine::AdvanceFrame should not immediately reschedule the state machine when buffering

Categories

(Core :: Audio/Video, defect)

x86_64
Linux
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla31

People

(Reporter: roc, Assigned: roc)

Details

Attachments

(1 file)

After MediaDecoderStateMachine::AdvanceFrame enters buffering, it immediately reschedules the state machine. Unfortunately this can cause thrashing where the buffering logic immediately switches back to DECODING, which runs AdvanceFrame, which flips us back to buffering, etc.
Attached patch fixSplinter Review
Attachment #8408922 - Flags: review?(cpearce)
Attachment #8408922 - Flags: review?(cpearce) → review+
https://hg.mozilla.org/mozilla-central/rev/8c7e0c36f4ea
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla31
You need to log in before you can comment on or make changes to this bug.