diff options
| author | hclam@chromium.org <hclam@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-04-23 01:09:10 +0000 |
|---|---|---|
| committer | hclam@chromium.org <hclam@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-04-23 01:09:10 +0000 |
| commit | e056807011e51e390f690591e65f48d1f037f081 (patch) | |
| tree | 758a940c0479e207b73e65c00839cac92a05240a /chrome/renderer | |
| parent | ce2c185eca0d6db644b765a5ba3254f5b06fd6b1 (diff) | |
| download | chromium_src-e056807011e51e390f690591e65f48d1f037f081.zip chromium_src-e056807011e51e390f690591e65f48d1f037f081.tar.gz chromium_src-e056807011e51e390f690591e65f48d1f037f081.tar.bz2 | |
Merge 44502 - Looping for audio / video
Fixed the looping problem with audio / video.
BUG=39478
TEST=audio plays with looping
Review URL: http://codereview.chromium.org/1620010
TBR=hclam@chromium.org
Review URL: http://codereview.chromium.org/1737005
git-svn-id: svn://svn.chromium.org/chrome/branches/375/src@45396 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/renderer')
| -rw-r--r-- | chrome/renderer/media/audio_renderer_impl.cc | 15 |
1 files changed, 6 insertions, 9 deletions
diff --git a/chrome/renderer/media/audio_renderer_impl.cc b/chrome/renderer/media/audio_renderer_impl.cc index 4325798..396ceeb 100644 --- a/chrome/renderer/media/audio_renderer_impl.cc +++ b/chrome/renderer/media/audio_renderer_impl.cc @@ -330,15 +330,12 @@ void AudioRendererImpl::NotifyPacketReadyTask() { uint32 filled = FillBuffer(static_cast<uint8*>(shared_memory_->memory()), shared_memory_size_, request_delay); - // TODO(hclam): we should try to fill in the buffer as much as possible. - if (filled > 0) { - pending_request_ = false; - request_delay_ = base::TimeDelta(); - request_timestamp_ = base::Time(); - // Then tell browser process we are done filling into the buffer. - filter_->Send( - new ViewHostMsg_NotifyAudioPacketReady(0, stream_id_, filled)); - } + pending_request_ = false; + request_delay_ = base::TimeDelta(); + request_timestamp_ = base::Time(); + // Then tell browser process we are done filling into the buffer. + filter_->Send( + new ViewHostMsg_NotifyAudioPacketReady(0, stream_id_, filled)); } } |