Bug 1271669 - Clean up some stream-centered code in MediaPipelineFactory. r?bwc
MozReview-Commit-ID: 8esaZmy7HVO
--- a/media/webrtc/signaling/src/peerconnection/MediaPipelineFactory.cpp
+++ b/media/webrtc/signaling/src/peerconnection/MediaPipelineFactory.cpp
@@ -862,36 +862,27 @@ MediaPipelineFactory::GetOrCreateVideoCo
}
nsresult
MediaPipelineFactory::ConfigureVideoCodecMode(const JsepTrack& aTrack,
VideoSessionConduit& aConduit)
{
#if !defined(MOZILLA_EXTERNAL_LINKAGE)
RefPtr<LocalSourceStreamInfo> stream =
- mPCMedia->GetLocalStreamById(aTrack.GetStreamId());
+ mPCMedia->GetLocalStreamByTrackId(aTrack.GetTrackId());
//get video track
RefPtr<mozilla::dom::VideoStreamTrack> videotrack =
stream->GetVideoTrackByTrackId(aTrack.GetTrackId());
if (!videotrack) {
MOZ_MTLOG(ML_ERROR, "video track not available");
return NS_ERROR_FAILURE;
}
- //get video source type
- RefPtr<DOMMediaStream> mediastream =
- mPCMedia->GetLocalStreamById(aTrack.GetStreamId())->GetMediaStream();
-
- DOMLocalMediaStream* domLocalStream = mediastream->AsDOMLocalMediaStream();
- if (!domLocalStream) {
- return NS_OK;
- }
-
dom::MediaSourceEnum source = videotrack->GetSource().GetMediaSource();
webrtc::VideoCodecMode mode = webrtc::kRealtimeVideo;
switch (source) {
case dom::MediaSourceEnum::Browser:
case dom::MediaSourceEnum::Screen:
case dom::MediaSourceEnum::Application:
case dom::MediaSourceEnum::Window:
mode = webrtc::kScreensharing;