Audio pause and seek better

BUG=39885

Pause for audio is now in effect in less than 200ms.

Seek for audio is also much better as we play also very short amount
after seek.

Review URL: http://codereview.chromium.org/2931014

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@52560 0039d316-1c4b-4281-b951-d872f2087c98

2 files changed, 25 insertions, 55 deletions

diff --git a/chrome/renderer/media/audio_renderer_impl.cc b/chrome/renderer/media/audio_renderer_impl.cc
index a1cae17..15a98a5 100644
--- a/chrome/renderer/media/audio_renderer_impl.cc
+++ b/chrome/renderer/media/audio_renderer_impl.cc
@@ -137,16 +137,36 @@ void AudioRendererImpl::SetPlaybackRate(float rate) {
   }
 }
 
+void AudioRendererImpl::Pause(media::FilterCallback* callback) {
+  AudioRendererBase::Pause(callback);
+  AutoLock auto_lock(lock_);
+  if (stopped_)
+    return;
+
+  io_loop_->PostTask(FROM_HERE,
+      NewRunnableMethod(this, &AudioRendererImpl::PauseTask));
+}
+
 void AudioRendererImpl::Seek(base::TimeDelta time,
                              media::FilterCallback* callback) {
   AudioRendererBase::Seek(time, callback);
+  AutoLock auto_lock(lock_);
+  if (stopped_)
+    return;
 
+  io_loop_->PostTask(FROM_HERE,
+      NewRunnableMethod(this, &AudioRendererImpl::SeekTask));
+}
+
+
+void AudioRendererImpl::Play(media::FilterCallback* callback) {
+  AudioRendererBase::Play(callback);
   AutoLock auto_lock(lock_);
   if (stopped_)
     return;
 
   io_loop_->PostTask(FROM_HERE,
-    NewRunnableMethod(this, &AudioRendererImpl::SeekTask));
+      NewRunnableMethod(this, &AudioRendererImpl::PlayTask));
 }
 
 void AudioRendererImpl::SetVolume(float volume) {
@@ -268,6 +288,8 @@ void AudioRendererImpl::PauseTask() {
 void AudioRendererImpl::SeekTask() {
   DCHECK(MessageLoop::current() == io_loop_);
 
+  // We have to pause the audio stream before we can flush.
+  filter_->Send(new ViewHostMsg_PauseAudioStream(0, stream_id_));
   filter_->Send(new ViewHostMsg_FlushAudioStream(0, stream_id_));
 }
 
diff --git a/chrome/renderer/media/audio_renderer_impl.h b/chrome/renderer/media/audio_renderer_impl.h
index a0e99d8..7c3aa58 100644
--- a/chrome/renderer/media/audio_renderer_impl.h
+++ b/chrome/renderer/media/audio_renderer_impl.h
@@ -33,60 +33,6 @@
 //    Properties of this filter is also set in this thread.
 // 3. Audio decoder thread (If there's one.)
 //    Responsible for decoding audio data and gives raw PCM data to this object.
-//
-// Methods categorized according to the thread(s) they are running on.
-//
-// Render thread
-// +-- CreateFactory()
-// |     Helper method for construction this class.
-// \-- IsMediaFormatSupported()
-//       Helper method to identify media formats accepted by this class for
-//       construction.
-//
-// IO thread (Main thread in render process)
-// +-- OnCreateStream()
-// |     Sends an IPC message to browser to create audio output stream and
-// |     register this object with AudioMessageFilter.
-// |-- OnSetVolume()
-// |     Sends an IPC message to browser to set volume.
-// |-- OnNotifyPacketReady
-// |     Try to fill the shared memory with decoded audio packet and sends IPC
-// |     messages to browser if packet is ready.
-// |-- OnRequestPacket()
-// |     Called from AudioMessageFilter when an audio packet requested is
-// |     received from browser process.
-// |-- OnStateChanged()
-// |     Called from AudioMessageFilter upon state change of the audio output
-// |     stream in the browser process. Error of the stream is reported here.
-// |-- OnCreated()
-// |     Called from AudioMessageFilter upon successful creation of audio output
-// |     stream in the browser process, called along with a SharedMemoryHandle.
-// |-- OnVolume()
-// |     Called from AudioMessageFilter about the volume of the audio output
-// |     stream.
-// \-- OnDestroy()
-//       Release resources that live inside io thread.
-//
-// Pipeline thread
-// +-- AudioRendererImpl()
-// |     Constructor method.
-// |-- ~AudioRendererImpl()
-// |     Destructor method.
-// |-- SetPlaybackRate()
-// |     Given the playback rate information.
-// |-- GetMediaFormat()
-// |     Obtain the current media format of this unit.
-// |-- SetVolume()
-// |     Given the volume information.
-// |-- OnInitialize()
-// |     Called from AudioRendererBase for initialization event.
-// \-- OnStop()
-//       Called from AudioRendererBase for stop event.
-//
-// Audio decoder thread (If there's one.)
-// \-- OnReadComplete()
-//       A raw PCM audio packet buffer is received here, this method is called
-//       from pipeline thread if audio decoder thread does not exist.
 
 #ifndef CHROME_RENDERER_MEDIA_AUDIO_RENDERER_IMPL_H_
 #define CHROME_RENDERER_MEDIA_AUDIO_RENDERER_IMPL_H_
@@ -130,7 +76,9 @@ class AudioRendererImpl : public media::AudioRendererBase,
   // Methods called on pipeline thread ----------------------------------------
   // media::MediaFilter implementation.
   virtual void SetPlaybackRate(float rate);
+  virtual void Pause(media::FilterCallback* callback);
   virtual void Seek(base::TimeDelta time, media::FilterCallback* callback);
+  virtual void Play(media::FilterCallback* callback);
 
   // media::AudioRenderer implementation.
   virtual void SetVolume(float volume);