Plumb render view ID from audio-related code in renderer through IPCs to AudioRendererHost in brows…

content/renderer/media/audio_device_factory.cc

Index: content/renderer/media/audio_device_factory.cc
diff --git a/content/renderer/media/audio_device_factory.cc b/content/renderer/media/audio_device_factory.cc
index ad7968884e78c410b8e03094850e03da0efb7318..a6e62628926447db51bc83719833bdeb45b0d5fb 100644
--- a/content/renderer/media/audio_device_factory.cc
+++ b/content/renderer/media/audio_device_factory.cc
@@ -8,6 +8,7 @@
 #include "content/common/child_process.h"
 #include "content/renderer/media/audio_input_message_filter.h"
 #include "content/renderer/media/audio_message_filter.h"
+#include "content/renderer/render_thread_impl.h"
 #include "media/audio/audio_input_device.h"
 #include "media/audio/audio_output_device.h"
 
@@ -17,13 +18,15 @@ namespace content {
 AudioDeviceFactory* AudioDeviceFactory::factory_ = NULL;
 
 // static
-media::AudioRendererSink* AudioDeviceFactory::NewOutputDevice() {
+media::AudioRendererSink* AudioDeviceFactory::NewOutputDevice(
+    int render_view_id) {

[tommi (sloooow) - chröme, 2012/10/16 09:17:07]

Why don't we do the same for the input device?

We've historically had a problem with the output and input implementations
diverging, so if we need to do this for both, let's do it now.

[miu, 2012/10/16 23:43:49]

Done.

   media::AudioRendererSink* device = NULL;
   if (factory_)
-    device = factory_->CreateOutputDevice();
+    device = factory_->CreateOutputDevice(render_view_id);
 
   return device ? device : new media::AudioOutputDevice(
-      AudioMessageFilter::Get(),
+      RenderThreadImpl::current()->audio_message_filter()->
+          CreateAudioOutputIPC(render_view_id),
       ChildProcess::current()->io_message_loop()->message_loop_proxy());
 }