Implements AudioMessageFilter as member in RenderThread

content/renderer/renderer_webaudiodevice_impl.cc

 // Copyright (c) 2010 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/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() {
   stop();
 }
 
 void RendererWebAudioDeviceImpl::start() {
   audio_device_->Start();
 }
 
@@ skipping 12 matching lines @@
   DCHECK(client_callback_);
   if (client_callback_) {
     // Wrap the pointers using WebVector.
     WebVector<float*> web_audio_data(audio_data.size());
     for (size_t i = 0; i < audio_data.size(); ++i)
       web_audio_data[i] = audio_data[i];
 
     client_callback_->render(web_audio_data, number_of_frames);
   }
 }