| commit | 24da0bdd190adf22b9d240d364f0c757717a5f5a | |
| author | rockot@chromium.org <rockot@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | |
| Fri, 18 Apr 2014 16:13:19 +0000 (18 16:13 +0000) | ||
| committer | rockot@chromium.org <rockot@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | |
| Fri, 18 Apr 2014 16:13:19 +0000 (18 16:13 +0000) | ||
| tree | ae515c4bc5b2d5e53b5bb2515b491df1d5a546b5 | treesnapshot (tar.gz zip) |
| parent | dc5662c95d6bbf3532bd91aa1ee4e5cbd890dc35 | commitdiff |
Revert 264763 "Wire up codec_delay() to MP3StreamParser and FFmp..."
Possible candidate for broken media_unittests:
http://build.chromium.org/p/chromium.chromiumos/builders/Linux%20ChromiumOS%20Tests%20%28dbg%29%281%29/builds/30043/steps/media_unittests/logs/MediaSource_MP3_TimestampOffset_0
> Wire up codec_delay() to MP3StreamParser and FFmpegAudioDecoder.
>
> Allows us to drop silent samples from the decoder after initialization
> and resets.
>
> codec_delay() is always in frames, so I've changed all call sites to use
> frames instead of a TimeDelta.
>
> Coincidentally also fixes an issue where not enough data was dropped from
> Vorbis buffers with negative timestamps.
>
> BUG=360961
> TEST=new pipeline_integration_test
>
> Review URL: https://codereview.chromium.org/239423005
TBR=dalecurtis@chromium.org
Review URL: https://codereview.chromium.org/242203006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@264803 0039d316-1c4b-4281-b951-d872f2087c98
Possible candidate for broken media_unittests:
http://build.chromium.org/p/chromium.chromiumos/builders/Linux%20ChromiumOS%20Tests%20%28dbg%29%281%29/builds/30043/steps/media_unittests/logs/MediaSource_MP3_TimestampOffset_0
> Wire up codec_delay() to MP3StreamParser and FFmpegAudioDecoder.
>
> Allows us to drop silent samples from the decoder after initialization
> and resets.
>
> codec_delay() is always in frames, so I've changed all call sites to use
> frames instead of a TimeDelta.
>
> Coincidentally also fixes an issue where not enough data was dropped from
> Vorbis buffers with negative timestamps.
>
> BUG=360961
> TEST=new pipeline_integration_test
>
> Review URL: https://codereview.chromium.org/239423005
TBR=dalecurtis@chromium.org
Review URL: https://codereview.chromium.org/242203006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@264803 0039d316-1c4b-4281-b951-d872f2087c98
16 files changed: