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

content/renderer/media/audio_device_factory.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 "content/renderer/media/audio_device_factory.h"
 
 #include "base/logging.h"
 #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"
 
 namespace content {
 
 // static
 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());
 }
 
 // static
 media::AudioInputDevice* AudioDeviceFactory::NewInputDevice() {
   media::AudioInputDevice* device = NULL;
   if (factory_)
     device = factory_->CreateInputDevice();
 
   return device ? device : new media::AudioInputDevice(
       AudioInputMessageFilter::Get(),
       ChildProcess::current()->io_message_loop()->message_loop_proxy());
 }
 
 AudioDeviceFactory::AudioDeviceFactory() {
   DCHECK(!factory_) << "Can't register two factories at once.";
   factory_ = this;
 }
 
 AudioDeviceFactory::~AudioDeviceFactory() {
   factory_ = NULL;
 }
 
 }  // namespace content