975933 - bug516323.indexed.ogv's index is too low precision

Status: RESOLVED FIXED

(Core :: Audio/Video, defect)

Description

[Chris Pearce [:cpearce (Not reading bugmail)]]

With attachment 8380443 [details] [diff] [review] (bug 975928) applied, I also get this test failure:

failed | bug516323.indexed.ogv: First range end should be media end - got 4.208333, expected 4.208

It turns out that the index in bug516323.indexed.ogv has a timestamp denominator of 1000; so its timestamps are in milliseconds. Whereas our media stack now has microsecond precision. So the index is reporting a duration of 4.208 seconds.

Whereas our Ogg GetBuffered() code doesn't use the index to calculate the buffered ranges, it examines the cached ranges and parses them to extract timestamps. So it gets a more precise duration, and our test fails due to the difference.

We can prevent this failure by increasing the precision of bug516323.indexed.ogv's index.

Comment 1

[Chris Pearce [:cpearce (Not reading bugmail)]]

Attachment: Patch: Update bug516323.indexed.ogv's index

I updated the index by patching OggIndex to write indexes with a timestamp denominator of 100000 instead of 1000. So I'll need to upstream that patch. I'm not even sure how to do that now.

Comment 2

[Ralph Giles (:rillian)]

Comment on attachment 8380445 [details] [diff] [review]
Patch: Update bug516323.indexed.ogv's index

Upstream for OggIndoex seems to be https://git.xiph.org/?p=OggIndex.git You should have commit access. So just push the patches to [e-mail]:OggIndex.git

If you don't still have access, send me a new key, or attach the patch here and I'll push it upstream.

Comment 3

[Chris Pearce [:cpearce (Not reading bugmail)]]

Pushed patch upstream. Thanks!

Comment 4

[Chris Pearce [:cpearce (Not reading bugmail)]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/72cfab2a3dcb

Comment 5

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/72cfab2a3dcb