Implement accurate seeking by decoding audio/video until we reach our desired timestamp.

Implement accurate seeking by decoding audio/video until we reach our desired timestamp.

This fixes a host of seeking, synchronization and layout test issues but does incur a CPU performance hit while seeking.  In the worst case a seek could take quite some time if the content was high resolution H.264 and we attempted to seek to right before a keyframe.

We'll see how people like the new behaviour and back this change out if needed.

BUG=48984
TEST=layout tests, media_unittests

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=53872

[Alpha Left Google, 2010-07-27 21:21:49]

LGTM after a small nit.

http://codereview.chromium.org/3032030/diff/1/3
File media/filters/audio_renderer_base.cc (right):

http://codereview.chromium.org/3032030/diff/1/3#newcode156
media/filters/audio_renderer_base.cc:156: (buffer_in->GetTimestamp() +
buffer_in->GetDuration()) < seek_timestamp_) {
nit: 80 chars

[scherkus (not reviewing), 2010-07-27 21:57:47]

Will check in after try servers etc....

http://codereview.chromium.org/3032030/diff/1/3
File media/filters/audio_renderer_base.cc (right):

http://codereview.chromium.org/3032030/diff/1/3#newcode156
media/filters/audio_renderer_base.cc:156: (buffer_in->GetTimestamp() +
buffer_in->GetDuration()) < seek_timestamp_) {
On 2010/07/27 21:21:49, Alpha wrote:
> nit: 80 chars
> 

Done.