diff options
| author | rsleevi@chromium.org <rsleevi@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-08-23 02:30:08 +0000 |
|---|---|---|
| committer | rsleevi@chromium.org <rsleevi@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-08-23 02:30:08 +0000 |
| commit | ccd50d5c6ce8362c86882caba06d3ac7aaaeee92 (patch) | |
| tree | 7602a5fe8702639ff3ce5e4cd2578dab614b631a /media/base/pipeline.cc | |
| parent | fe1f67a971a68ef59e2e521397bec0df27cdcdf5 (diff) | |
| download | chromium_src-ccd50d5c6ce8362c86882caba06d3ac7aaaeee92.zip chromium_src-ccd50d5c6ce8362c86882caba06d3ac7aaaeee92.tar.gz chromium_src-ccd50d5c6ce8362c86882caba06d3ac7aaaeee92.tar.bz2 | |
Revert 152919 - Guard against ALSA returning insane frame counts for current delay.
One video observed generated an alleged delay of 9222246136947932171 frames near
the end of stream, which when converted to bytes and passed through a chain of
uint32/int32 conversions became -4052 bytes, and then -46ms, triggering the
DCHECK in the bug below.
BUG=144281
Review URL: https://chromiumcodereview.appspot.com/10869019
TBR=fischman@chromium.org
Review URL: https://chromiumcodereview.appspot.com/10879027
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@152923 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'media/base/pipeline.cc')
| -rw-r--r-- | media/base/pipeline.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/media/base/pipeline.cc b/media/base/pipeline.cc index 4650e6e..a27bfc3 100644 --- a/media/base/pipeline.cc +++ b/media/base/pipeline.cc @@ -352,7 +352,7 @@ void Pipeline::OnAudioDisabled() { } void Pipeline::OnAudioTimeUpdate(TimeDelta time, TimeDelta max_time) { - DCHECK_LE(time.InMicroseconds(), max_time.InMicroseconds()); + DCHECK(time <= max_time); DCHECK(IsRunning()); base::AutoLock auto_lock(lock_); |