Closed Bug 1002294 Opened 10 years ago Closed 10 years ago

SourceBuffer::ContainsTime is too optimistic

Categories

(Core :: Audio/Video, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla31

People

(Reporter: cajbir, Assigned: cajbir)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

SourceBuffer::ContainsTime will return true for times that it doesn't have data for due to it using the start of the first range and end of the last range. It assumes all ranges are consecutive.
Assignee: nobody → cajbir.bugzilla
Blocks: MSE
Status: NEW → ASSIGNED
Attached patch fixSplinter Review
Check each range in the source buffer to see if the requested time lies within.
Attachment #8413478 - Flags: review?(kinetik)
Attachment #8413478 - Flags: review?(kinetik) → review+
https://hg.mozilla.org/mozilla-central/rev/061e60bc9abb
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla32
Target Milestone: mozilla32 → mozilla31
You need to log in before you can comment on or make changes to this bug.