Feed the render data to MediaStreamAudioProcessor and used AudioBus in render callback

content/renderer/media/media_stream_audio_processor_unittest.cc

Index: content/renderer/media/media_stream_audio_processor_unittest.cc
diff --git a/content/renderer/media/media_stream_audio_processor_unittest.cc b/content/renderer/media/media_stream_audio_processor_unittest.cc
index a890d1d43bfdf5f8b9ef9498610c7c50617f6ec6..6b3f427231da5a3e69a6511417427d6b39ba103f 100644
--- a/content/renderer/media/media_stream_audio_processor_unittest.cc
+++ b/content/renderer/media/media_stream_audio_processor_unittest.cc
@@ -80,10 +80,10 @@ class MediaStreamAudioProcessorTest : public ::testing::Test {
 
       // |audio_processor| does nothing when the audio processing is off in
       // the processor.
-      audio_processor->PushRenderData(
-          data_ptr,
-          params_.sample_rate(), params_.channels(),
-          params_.frames_per_buffer(), base::TimeDelta::FromMilliseconds(10));
+      if (audio_processor->has_audio_processing()) {
+        static_cast<WebRtcAudioRendererSource*>(audio_processor)->RenderData(

[miu, 2014/01/24 22:27:19]

This threw me off a bit.  I see that you're trying to call a private method of
MediaStreamAudioProcessor here.  I'll leave it up to you, but IMHO it's more
readable to make MediaStreamAudioProcessorTest a friend class of
MediaStreamAudioProcessor, which would allow you to invoke RenderData() without
the static_cast<> here.

[no longer working on chromium, 2014/01/27 17:09:33]

Actually MediaStreamAudioProcessorTest had been made a friend class of
MediaStreamAudioProcessor in my another CL. So I removed the static_cast after
rebasing.

+            data_bus.get(), params_.sample_rate(), 10);
+      }
 
       int16* output = NULL;
       while(audio_processor->ProcessAndConsumeData(
@@ -108,7 +108,7 @@ TEST_F(MediaStreamAudioProcessorTest, WithoutAudioProcessing) {
   // Setup the audio processor without enabling the flag.
   blink::WebMediaConstraints constraints;
   scoped_refptr<MediaStreamAudioProcessor> audio_processor(
-      new MediaStreamAudioProcessor(params_, constraints, 0));
+      new MediaStreamAudioProcessor(params_, constraints, 0, NULL));
   EXPECT_FALSE(audio_processor->has_audio_processing());
 
   ProcessDataAndVerifyFormat(audio_processor,
@@ -123,7 +123,7 @@ TEST_F(MediaStreamAudioProcessorTest, WithAudioProcessing) {
       switches::kEnableAudioTrackProcessing);
   blink::WebMediaConstraints constraints;
   scoped_refptr<MediaStreamAudioProcessor> audio_processor(
-      new MediaStreamAudioProcessor(params_, constraints, 0));
+      new MediaStreamAudioProcessor(params_, constraints, 0, NULL));
   EXPECT_TRUE(audio_processor->has_audio_processing());
 
   ProcessDataAndVerifyFormat(audio_processor,