commit 6df49d8983eb915cb48b49e7e65aae56567b8c8d
author Amit Hilbuch <amithi@webrtc.org> Mon Apr 15 22:47:21 2019
committer Commit Bot <commit-bot@chromium.org> Tue Apr 16 00:34:04 2019
tree ca0612de7eed7d221c06dfb6835295d8eaeb2f77
parent 53625ca8abe29873a76741ca2d6f3f0b04add8dd

Fixing issue with creating StreamParams when track id is not signaled.

Current logic requires a stream id and track ids when creating a stream
that does not have SSRCs signaled.
This change removes the requirement for stream ids. The requirement for
track id is softer, as one should be generated when it is not present.

Bug: webrtc:10551
Change-Id: Ibc0cc181c6b18efa8394b6c0e4820e3a13da70c4
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/133080
Reviewed-by: Steve Anton <steveanton@webrtc.org>
Commit-Queue: Amit Hilbuch <amithi@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#27633}

pc/webrtc_sdp.cc

diff --git a/pc/webrtc_sdp.cc b/pc/webrtc_sdp.cc
index b524a75..984a1e1 100644
--- a/pc/webrtc_sdp.cc
+++ b/pc/webrtc_sdp.cc
@@ -666,8 +666,9 @@
                             const std::vector<RidDescription>& rids,
                             StreamParamsVec* tracks) {
   StreamParams track;
-  if (msid_track_id.empty() || msid_stream_ids.empty()) {
+  if (msid_track_id.empty() && rids.empty()) {
     // We only create an unsignaled track if a=msid lines were signaled.
+    RTC_LOG(LS_INFO) << "MSID not signaled, skipping creation of StreamParams";
     return;
   }
   track.set_stream_ids(msid_stream_ids);