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

RESOLVED FIXED in mozilla31

Status

()

defect
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: roc, Assigned: roc)

Tracking

Trunk
mozilla31
x86_64
Linux
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

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.
Attachment #8408922 - Flags: review?(cpearce) → review+
https://hg.mozilla.org/mozilla-central/rev/8c7e0c36f4ea
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla31
You need to log in before you can comment on or make changes to this bug.