Closed
Bug 1002294
Opened 10 years ago
Closed 10 years ago
SourceBuffer::ContainsTime is too optimistic
Categories
(Core :: Audio/Video, defect)
Core
Audio/Video
Tracking
()
RESOLVED
FIXED
mozilla31
People
(Reporter: cajbir, Assigned: cajbir)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
1.24 KB,
patch
|
kinetik
:
review+
|
Details | Diff | Splinter Review |
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•10 years ago
|
Assignee | ||
Comment 1•10 years ago
|
||
Check each range in the source buffer to see if the requested time lies within.
Attachment #8413478 -
Flags: review?(kinetik)
Updated•10 years ago
|
Attachment #8413478 -
Flags: review?(kinetik) → review+
Assignee | ||
Comment 2•10 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/061e60bc9abb
Comment 3•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/061e60bc9abb
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla32
Updated•10 years ago
|
Target Milestone: mozilla32 → mozilla31
You need to log in
before you can comment on or make changes to this bug.
Description
•