SourceBuffer::ContainsTime is too optimistic

RESOLVED FIXED in mozilla31

Status

()

defect
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: cajbir, Assigned: cajbir)

Tracking

(Blocks 1 bug)

Trunk
mozilla31
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
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)

Updated

5 years ago
Assignee: nobody → cajbir.bugzilla
Blocks: MSE
Status: NEW → ASSIGNED
(Assignee)

Comment 1

5 years ago
Posted 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
Last Resolved: 5 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.