Associate audio streams with their source/destination RenderView.

content/renderer/media/audio_renderer_mixer_manager.cc

Index: content/renderer/media/audio_renderer_mixer_manager.cc
diff --git a/content/renderer/media/audio_renderer_mixer_manager.cc b/content/renderer/media/audio_renderer_mixer_manager.cc
index 67daf2cbeb9a448f86e2329c295425c1d7f42934..8392d7712483ff26e325f9f5f5b6380d4d078977 100644
--- a/content/renderer/media/audio_renderer_mixer_manager.cc
+++ b/content/renderer/media/audio_renderer_mixer_manager.cc
@@ -7,6 +7,7 @@
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "content/renderer/media/audio_device_factory.h"
+#include "content/renderer/media/renderer_audio_output_device.h"
 #include "media/base/audio_renderer_mixer.h"
 #include "media/base/audio_renderer_mixer_input.h"
 
@@ -15,7 +16,8 @@ namespace content {
 AudioRendererMixerManager::AudioRendererMixerManager(int hardware_sample_rate,
                                                      int hardware_buffer_size)
     : hardware_sample_rate_(hardware_sample_rate),
-      hardware_buffer_size_(hardware_buffer_size) {
+      hardware_buffer_size_(hardware_buffer_size),
+      sink_for_testing_(NULL) {

[DaleCurtis, 2012/12/03 18:52:03]

Are these changes still necessary?

[miu, 2012/12/03 20:20:51]

Yes.  AudioDeviceFactory::NewOutputDevice() now returns a
RendererAudioOutputDevice.  In order to do testing the old way, I'd have to
write a mock that's a subclass of RendererAudioOutputDevice.  This seemed way
overkill, so I went with this simple solution.

 }
 
 AudioRendererMixerManager::~AudioRendererMixerManager() {
@@ -30,6 +32,11 @@ media::AudioRendererMixerInput* AudioRendererMixerManager::CreateInput() {
           &AudioRendererMixerManager::RemoveMixer, base::Unretained(this)));
 }
 
+void AudioRendererMixerManager::SetAudioRendererSinkForTesting(
+    media::AudioRendererSink* sink) {
+  sink_for_testing_ = sink;
+}
+
 media::AudioRendererMixer* AudioRendererMixerManager::GetMixer(
     const media::AudioParameters& params) {
   base::AutoLock auto_lock(mixers_lock_);
@@ -53,7 +60,10 @@ media::AudioRendererMixer* AudioRendererMixerManager::GetMixer(
     output_params = params;
 
   media::AudioRendererMixer* mixer = new media::AudioRendererMixer(
-      params, output_params, AudioDeviceFactory::NewOutputDevice());
+      params, output_params,
+      sink_for_testing_ ?
+          sink_for_testing_ :
+          AudioDeviceFactory::NewOutputDevice());
 
   AudioRendererMixerReference mixer_reference = { mixer, 1 };
   mixers_[params] = mixer_reference;