Implements AudioMessageFilter as member in RenderThread

content/renderer/renderer_webaudiodevice_impl.cc

Index: content/renderer/renderer_webaudiodevice_impl.cc
===================================================================
--- content/renderer/renderer_webaudiodevice_impl.cc	(revision 88939)
+++ content/renderer/renderer_webaudiodevice_impl.cc	(working copy)
@@ -4,13 +4,20 @@
 
 #include "content/renderer/renderer_webaudiodevice_impl.h"
 
+#include "content/renderer/media/audio_message_filter.h"
+#include "content/renderer/media/audio_message_filter_creator.h"
+
 using WebKit::WebAudioDevice;
 using WebKit::WebVector;
 
 RendererWebAudioDeviceImpl::RendererWebAudioDeviceImpl(size_t buffer_size,
     int channels, double sample_rate, WebAudioDevice::RenderCallback* callback)
     : client_callback_(callback) {
-  audio_device_ = new AudioDevice(buffer_size, channels, sample_rate, this);
+  VLOG(1) << "RendererWebAudioDeviceImpl::RendererWebAudioDeviceImpl()";
+  AudioMessageFilter* audio_message_filter =
+      AudioMessageFilterCreator::SharedFilter();

[Chris Rogers, 2011/06/14 22:57:10]

Is there any reason why AudioDevice can't call
AudioMessageFilterCreator::SharedFilter() itself to avoid having to pass this in
as a constructor argument?  AudioDevice was supposed to be an abstraction to
hide the underlying implementation details of the IPC.  This seems to be
requiring clients of AudioDevice to know more than they should have to about
IPC.

[henrika_dont_use, 2011/06/15 15:16:39]

Done.

+  audio_device_ = new AudioDevice(
+      buffer_size, channels, sample_rate, this, audio_message_filter);
 }
 
 RendererWebAudioDeviceImpl::~RendererWebAudioDeviceImpl() {