Closed Bug 1002294 Opened 11 years ago Closed 11 years ago

SourceBuffer::ContainsTime is too optimistic

Categories

(Core :: Audio/Video, defect)

Product:
Core
Component:
Audio/Video
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla31

People

(Reporter: cajbir, Assigned: cajbir)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

fix
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: nobody → cajbir.bugzilla
Blocks: MSE
Status: NEW → ASSIGNED
Attached patch fix — — Splinter 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: 11 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.

Attachment

General

Creator:
Created:
Updated:
Size: