Every MediaDecoderReader backend that has to estimate the buffered ranges using GetEstimatedBufferedTimeRanges() ends up reimplementing the same stub to call GetEstimatedBufferedTimeRanges(). We can instead implement this stub in MediaDecoderReader::GetBuffered(). Backends that can make a more accurate buffered ranges calculation override GetBuffered() as usual, and those that can't don't need to re-implement the code that calls GetEstimatedBufferedTimeRanges().
Patch. The only behaviour change here is in MediaPluginReader::GetBuffered(), where we use the duration as per the MediaDecoder rather than as per the plugin, but I think that that's not actually a behaviour change, since the decoder should have received the duration from the plugin anyway?
Attachment #819509 - Flags: review?(chris.double)
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla27
You need to log in before you can comment on or make changes to this bug.