Closed Bug 1062670 Opened 6 years ago Closed 6 years ago

Deal with timecode error episilon when handling time ranges

Categories

(Core :: Audio/Video, defect, P5)

defect

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?
Blocks: MSE
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.
See Also: → 1065218
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.