commit | 6fb8654984e276dc98b2df689e773a8636f227f4 | [log] [tgz] |
---|---|---|
author | dalecurtis@google.com <dalecurtis@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | Fri Apr 18 19:58:13 2014 |
committer | dalecurtis@google.com <dalecurtis@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | Fri Apr 18 19:58:13 2014 |
tree | 810bfa59fd46b3f14ffb44d8163133b1798d8910 | |
parent | b17906c16e1af9c7d94f9f9dbc5e8eb40c9a16f0 [diff] |
Revert 264803 "Revert 264763 "Wire up codec_delay() to MP3Stream..." > 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 TBR=rockot@chromium.org Review URL: https://codereview.chromium.org/243653002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@264834 0039d316-1c4b-4281-b951-d872f2087c98