MediaStream audio sourcing: Bypass audio processing for non-WebRTC cases.

content/renderer/media/media_stream_renderer_factory_impl.cc

 // Copyright 2014 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/media_stream_renderer_factory_impl.h"
 
 #include "base/strings/utf_string_conversions.h"
 #include "content/renderer/media/media_stream.h"
 #include "content/renderer/media/media_stream_audio_track.h"
 #include "content/renderer/media/media_stream_video_renderer_sink.h"
 #include "content/renderer/media/media_stream_video_track.h"
 #include "content/renderer/media/track_audio_renderer.h"
 #include "content/renderer/media/webrtc/peer_connection_dependency_factory.h"
+#include "content/renderer/media/webrtc/peer_connection_remote_audio_source.h"
 #include "content/renderer/media/webrtc_audio_renderer.h"
 #include "content/renderer/render_thread_impl.h"
 #include "media/base/audio_hardware_config.h"
 #include "third_party/WebKit/public/platform/WebMediaStream.h"
 #include "third_party/WebKit/public/platform/WebURL.h"
 #include "third_party/WebKit/public/web/WebMediaStreamRegistry.h"
 #include "third_party/webrtc/api/mediastreaminterface.h"
 
 namespace content {
 
@@ skipping 78 matching lines @@
   DVLOG(1) << "MediaStreamRendererFactoryImpl::GetAudioRenderer stream:"
            << base::UTF16ToUTF8(base::StringPiece16(web_stream.id()));
 
   // TODO(tommi): We need to fix the data flow so that
   // it works the same way for all track implementations, local, remote or what
   // have you.
   // In this function, we should simply create a renderer object that receives
   // and mixes audio from all the tracks that belong to the media stream.
   // For now, we have separate renderers depending on if the first audio track
   // in the stream is local or remote.
-  MediaStreamAudioTrack* audio_track = MediaStreamAudioTrack::GetTrack(
-      audio_tracks[0]);
+  MediaStreamAudioTrack* const audio_track =
+      MediaStreamAudioTrack::Get(audio_tracks[0]);
   if (!audio_track) {
     // This can happen if the track was cloned.
     // TODO(tommi, perkj): Fix cloning of tracks to handle extra data too.
     LOG(ERROR) << "No native track for WebMediaStreamTrack.";
     return nullptr;
   }
 
   // If the track has a local source, or is a remote track that does not use the
   // WebRTC audio pipeline, return a new TrackAudioRenderer instance.
-  //
-  // TODO(miu): In a soon up-coming change, I'll introduce a cleaner way (i.e.,
-  // rather than calling GetAudioAdapter()) to determine whether a remote source
-  // is via WebRTC or something else.
-  if (audio_track->is_local_track() || !audio_track->GetAudioAdapter()) {
+  PeerConnectionRemoteAudioTrack* const pc_remote_audio_track =
+      PeerConnectionRemoteAudioTrack::From(audio_track);
+  if (audio_track->is_local_track() || !pc_remote_audio_track) {
     // TODO(xians): Add support for the case where the media stream contains
     // multiple audio tracks.
     DVLOG(1) << "Creating TrackAudioRenderer for "
              << (audio_track->is_local_track() ? "local" : "remote")
              << " track.";
     return new TrackAudioRenderer(audio_tracks[0], render_frame_id,
                                   0 /* no session_id */, device_id,
                                   security_origin);
   }
 
@@ skipping 14 matching lines @@
         device_id, security_origin);
 
     if (!audio_device->SetAudioRenderer(renderer.get()))
       return nullptr;
   }
 
   return renderer->CreateSharedAudioRendererProxy(web_stream);
 }
 
 }  // namespace content