Merge AssociateStreamWithProducer message into CreateStream message for both audio output and input.

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/renderer/media/audio_input_message_filter.h"
 #include "content/renderer/media/audio_message_filter.h"
-#include "content/renderer/media/renderer_audio_output_device.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
-scoped_refptr<RendererAudioOutputDevice> AudioDeviceFactory::NewOutputDevice() {
-  RendererAudioOutputDevice* device = NULL;
-  if (factory_)
-    device = factory_->CreateOutputDevice();
+scoped_refptr<media::AudioOutputDevice> AudioDeviceFactory::NewOutputDevice(
+    int render_view_id) {
+  if (factory_) {
+    media::AudioOutputDevice* const device =
+        factory_->CreateOutputDevice(render_view_id);
+    if (device)
+      return device;
+  }
 
-  return device ? device : new RendererAudioOutputDevice(
-      AudioMessageFilter::Get(), AudioMessageFilter::Get()->io_message_loop());
+  AudioMessageFilter* const filter =
+      RenderThreadImpl::current()->audio_message_filter();

[DaleCurtis, 2013/03/07 21:02:58]

I don't think this works? AudioRendererMixerManager will call this method from
the MediaPipeline thread and current() is only usable on the render thread.

[miu, 2013/04/16 00:07:36]

I dedcided to put the AudioMessageFilter::Get() static method back in.  This
change is big enough that I don't want to also start refactoring to solve this
rather complex "create from any thread" problem.  ;-)

+  return new media::AudioOutputDevice(
+      filter->CreateAudioOutputIPC(render_view_id), filter->io_message_loop());
 }
 
 // static
-scoped_refptr<media::AudioInputDevice> AudioDeviceFactory::NewInputDevice() {
-  media::AudioInputDevice* device = NULL;
-  if (factory_)
-    device = factory_->CreateInputDevice();
+scoped_refptr<media::AudioInputDevice> AudioDeviceFactory::NewInputDevice(
+    int render_view_id) {
+  if (factory_) {
+    media::AudioInputDevice* const device =
+        factory_->CreateInputDevice(render_view_id);
+    if (device)
+      return device;
+  }
 
-  return device ? device : new media::AudioInputDevice(
-      AudioInputMessageFilter::Get(),
-      AudioInputMessageFilter::Get()->io_message_loop());
+  AudioInputMessageFilter* const filter =
+      RenderThreadImpl::current()->audio_input_message_filter();
+  return new media::AudioInputDevice(
+      filter->CreateAudioInputIPC(render_view_id), filter->io_message_loop());
 }
 
 AudioDeviceFactory::AudioDeviceFactory() {
   DCHECK(!factory_) << "Can't register two factories at once.";
   factory_ = this;
 }
 
 AudioDeviceFactory::~AudioDeviceFactory() {
   factory_ = NULL;
 }
 
 }  // namespace content