Finding the duration of a .ogg media resource requires seeking to the end of the file and working backwards to find the timestamp of the last packet. This is slow on HTTP resources since it requires multiple byte range requests. liboggplay requests the duration whenever a seek is done. This requires multiple byte range requests, then the seek is performed which requires more. See bug 469408 comment 4 for details. The latest liboggplay svn caches the duration so it doesn't make multiple calls. We should update to get this fix. We should also add a 'duration' function to the our channel reader implementation that returns the duration if we know it as the result of an X-Content-Duration header (see bug 469923). This would mean that liboggplay never needs to seek to find the duration during a seek.

This updates liboggz to svn r3835 and liboggplay to svn r3848. The liboggplay update is to get changes that involve caching duration. The liboggz update is required for recent changes to liboggplay.

Implements the duration function in the channel reader to tell liboggplay what the duration is if we already know it through X-Content-Duration header or similar means. This enables avoiding the need to seek at all when liboggplay queries for the duration.

These patches were pushed http://hg.mozilla.org/mozilla-central/rev/45eb7626bca0 http://hg.mozilla.org/mozilla-central/rev/2b7e176d9e73 and backed out due to e:/builds/moz2_slave/mozilla-central-win32-debug/build/media/liboggplay/src/liboggplay/oggplay_callback.c(233) : error C2054: expected '(' to follow 'inline' http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1235097746.1235098651.12858.gz#err0

Updates liboggz to svn r3867 to pull in fixes. Fixes issues with inline that caused previous build failure. I'm testing with tryserver and will checkin if that goes fine.

liboggz config fixes to build on win32 and wince. No changes to the reviewed code by roc, carring forward s+r. Tested on tryserver.

Pushed to mozilla-central: http://hg.mozilla.org/mozilla-central/rev/0722a0e98124

Backed out due to: http://hg.mozilla.org/mozilla-central/rev/07682d87fcdf http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1236069997.1236070382.19802.gz#err0

liboggz on windows includes code compiled with DEBUG defined that causes compile errors. Force #undef DEBUG.

Comment on attachment 365155 [details] [diff] [review] Don't include broken liboggz debug code on windows carry forward r/sr

Pushed to mozilla-central: http://hg.mozilla.org/mozilla-central/rev/5d72424078d4

Pushed to mozilla-1.9.1: http://hg.mozilla.org/releases/mozilla-1.9.1/rev/b5f896adf581 I had to tweak the patch a bit to fix merge errors due to bug 461844 not landing on 1.9.1.