Closed
Bug 1062670
Opened 10 years ago
Closed 10 years ago
Deal with timecode error episilon when handling time ranges
Categories
(Core :: Audio/Video, defect, P5)
Core
Audio/Video
Tracking
()
RESOLVED
WORKSFORME
People
(Reporter: kinetik, Unassigned)
References
(Blocks 1 open bug)
Details
This is related to bug 1062657. We need to consider the same issue when dealing with time ranges in other instances, specifically:
- Any handling related to computing GetBuffered
- Switch{Audio,Video}Readers to avoid becoming stuck on an exhausted decoder
(currently this is somewhat hacked around by checking the decoder's {Audio,Video}Queue for EOS)
- Other places I've forgotten?
Reporter | ||
Comment 1•10 years ago
|
||
Converting MSE's internal timestamp handling to use int64_t/microseconds (as the rest of the media code does) will also make this less painful. It's currently using doubles/seconds because that's what flows in via the API and what's available in the existing TimeRanges, so it was the path of least resistance until there was sufficient time to make the necessary changes elsewhere in the codebase.
Updated•10 years ago
|
Priority: -- → P5
Updated•10 years ago
|
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → WORKSFORME
You need to log in
before you can comment on or make changes to this bug.
Description
•