Tab Audio Mirroring: WebContentsAudioInputStream is a new implementation which represents the lifet…

media/audio/virtual_audio_output_stream.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "media/audio/virtual_audio_output_stream.h"
 
-#include "base/message_loop.h"
-#include "media/audio/audio_manager_base.h"
+#include "base/message_loop_proxy.h"
 #include "media/audio/virtual_audio_input_stream.h"
 
 namespace media {
 
-// static
-VirtualAudioOutputStream* VirtualAudioOutputStream::MakeStream(
-    AudioManagerBase* manager, const AudioParameters& params,
-    base::MessageLoopProxy* message_loop, VirtualAudioInputStream* target) {
-  return new VirtualAudioOutputStream(manager, params, message_loop, target);
-}
-
 VirtualAudioOutputStream::VirtualAudioOutputStream(
-    AudioManagerBase* manager, const AudioParameters& params,
-    base::MessageLoopProxy* message_loop, VirtualAudioInputStream* target)
-    : audio_manager_(manager), message_loop_(message_loop), callback_(NULL),
-      params_(params), target_input_stream_(target), volume_(1.0f),
-      attached_(false) {
+    const AudioParameters& params, base::MessageLoopProxy* message_loop,
+    VirtualAudioInputStream* target, const AfterCloseCallback& after_close_cb)
+    : params_(params), message_loop_(message_loop),
+      target_input_stream_(target), after_close_cb_(after_close_cb),
+      callback_(NULL), volume_(1.0f) {
+  DCHECK(params_.IsValid());
+  DCHECK(message_loop_);
+  DCHECK(target);
 }
 
 VirtualAudioOutputStream::~VirtualAudioOutputStream() {
   DCHECK(!callback_);
-  DCHECK(!attached_);
 }
 
 bool VirtualAudioOutputStream::Open() {
   DCHECK(message_loop_->BelongsToCurrentThread());
   return true;
 }
 
 void VirtualAudioOutputStream::Start(AudioSourceCallback* callback)  {
   DCHECK(message_loop_->BelongsToCurrentThread());
-  DCHECK(!attached_);
+  DCHECK(!callback_);
   callback_ = callback;
   target_input_stream_->AddOutputStream(this, params_);
-  attached_ = true;
 }
 
 void VirtualAudioOutputStream::Stop() {
   DCHECK(message_loop_->BelongsToCurrentThread());
-  DCHECK(attached_);
-  callback_ = NULL;
-  target_input_stream_->RemoveOutputStream(this, params_);
-  attached_ = false;
+  if (callback_) {
+    callback_ = NULL;
+    target_input_stream_->RemoveOutputStream(this, params_);
+  }
 }
 
 void VirtualAudioOutputStream::Close() {
   DCHECK(message_loop_->BelongsToCurrentThread());
-  audio_manager_->ReleaseOutputStream(this);
+  Stop();
+  if (!after_close_cb_.is_null()) {

[DaleCurtis, 2013/01/17 01:15:52]

Ditto.

[miu, 2013/01/17 05:33:55]

Added the comments here as well.

+    const AfterCloseCallback cb = after_close_cb_;
+    after_close_cb_.Reset();
+    cb.Run(this);
+  }
 }
 
 void VirtualAudioOutputStream::SetVolume(double volume) {
+  DCHECK(message_loop_->BelongsToCurrentThread());
   volume_ = volume;
 }
 
 void VirtualAudioOutputStream::GetVolume(double* volume) {
+  DCHECK(message_loop_->BelongsToCurrentThread());
   *volume = volume_;
 }
 
 double VirtualAudioOutputStream::ProvideInput(AudioBus* audio_bus,
                                               base::TimeDelta buffer_delay) {
   DCHECK(message_loop_->BelongsToCurrentThread());
   DCHECK(callback_);
 
   int frames = callback_->OnMoreData(audio_bus, AudioBuffersState());
   if (frames < audio_bus->frames())
     audio_bus->ZeroFramesPartial(frames, audio_bus->frames() - frames);
 
   return frames > 0 ? volume_ : 0;
 }
 
 }  // namespace media