MediaStream audio: Refactor 3 separate "glue" implementations into one.

content/public/renderer/media_stream_utils.cc

 // Copyright 2015 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/public/renderer/media_stream_utils.h"
 
-#include <utility>
-
 #include "base/callback.h"
 #include "base/guid.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/rand_util.h"
 #include "base/strings/utf_string_conversions.h"
-#include "content/renderer/media/media_stream_audio_source.h"
+#include "content/renderer/media/external_media_stream_audio_source.h"
 #include "content/renderer/media/media_stream_video_capturer_source.h"
 #include "content/renderer/media/media_stream_video_source.h"
 #include "content/renderer/media/media_stream_video_track.h"
-#include "content/renderer/render_thread_impl.h"
 #include "media/base/audio_capturer_source.h"
 #include "media/base/video_capturer_source.h"
 #include "third_party/WebKit/public/platform/WebMediaStream.h"
 #include "third_party/WebKit/public/platform/WebMediaStreamSource.h"
 #include "third_party/WebKit/public/web/WebMediaStreamRegistry.h"
 
 namespace content {
 
 bool AddVideoTrackToMediaStream(
     scoped_ptr<media::VideoCapturerSource> video_source,
@@ skipping 47 matching lines @@
     DLOG(ERROR) << "Invalid audio parameters.";
     return false;
   }
 
   blink::WebMediaStreamSource web_media_stream_source;
   const blink::WebString track_id =
       blink::WebString::fromUTF8(base::GenerateGUID());
   web_media_stream_source.initialize(track_id,
                                      blink::WebMediaStreamSource::TypeAudio,
                                      track_id, is_remote, is_readonly);
-
-  MediaStreamAudioSource* media_stream_source(new MediaStreamAudioSource(
-      -1, StreamDeviceInfo(), MediaStreamSource::SourceStoppedCallback(),
-      RenderThreadImpl::current()->GetPeerConnectionDependencyFactory()));
-
-  blink::WebMediaConstraints constraints;
-  constraints.initialize();
-  {
-    // TODO(miu): In an upcoming change, a source purposed for passing audio
-    // directly (i.e., without modification) will replace this "hacky" use of
-    // WebRtcAudioCapturer.  http://crbug.com/577881
-    scoped_ptr<WebRtcAudioCapturer> capturer(
-        WebRtcAudioCapturer::CreateCapturer(-1, StreamDeviceInfo(), constraints,
-                                            nullptr, media_stream_source));
-    capturer->SetCapturerSource(std::move(audio_source), params);
-    media_stream_source->SetAudioCapturer(std::move(capturer));
-  }
-  web_media_stream_source.setExtraData(
-      media_stream_source);  // Takes ownership.
+  MediaStreamAudioSource* const media_stream_source =
+      new ExternalMediaStreamAudioSource(std::move(audio_source), sample_rate,
+                                         channel_layout, frames_per_buffer,
+                                         is_remote);
+  // Takes ownership of |media_stream_source|.
+  web_media_stream_source.setExtraData(media_stream_source);
 
   blink::WebMediaStreamTrack web_media_stream_track;
   web_media_stream_track.initialize(web_media_stream_source);
-  RenderThreadImpl::current()
-      ->GetPeerConnectionDependencyFactory()
-      ->CreateLocalAudioTrack(web_media_stream_track);
-
+  if (!media_stream_source->ConnectToTrack(web_media_stream_track))
+    return false;
   web_media_stream->addTrack(web_media_stream_track);
   return true;
 }
 
 const media::VideoCaptureFormat* GetCurrentVideoTrackFormat(
     const blink::WebMediaStreamTrack& video_track) {
   if (video_track.isNull())
     return nullptr;
 
   content::MediaStreamVideoSource* source =
       content::MediaStreamVideoSource::GetVideoSource(video_track.source());
   if (!source)
     return nullptr;
 
   return source->GetCurrentFormat();
 }
 
 }  // namespace content