Bug 1035067 - Don't hint we expect a track if we're not going to receive it. r=ehugg, a=lmandel
authorRandell Jesup <rjesup@jesup.org>
Mon, 07 Jul 2014 14:45:36 -0400
changeset 208227 b7316976ca8b
parent 208226 19d074395f1a
child 208228 a979f156f8d1
push id3782
push userryanvm@gmail.com
push date2014-08-04 16:21 +0000
treeherdermozilla-beta@e61371be0f38 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersehugg, lmandel
bugs1035067
milestone32.0
Bug 1035067 - Don't hint we expect a track if we're not going to receive it. r=ehugg, a=lmandel
media/webrtc/signaling/src/sipcc/core/gsm/gsm_sdp.c
--- a/media/webrtc/signaling/src/sipcc/core/gsm/gsm_sdp.c
+++ b/media/webrtc/signaling/src/sipcc/core/gsm/gsm_sdp.c
@@ -5257,17 +5257,20 @@ gsmsdp_negotiate_media_lines (fsm_fcb_t 
                     gsmsdp_set_rtcp_mux_attribute (SDP_ATTR_RTCP_MUX, media->level,
                                                    sdp_p->src_sdp, TRUE);
                   }
 
                   if (notify_stream_added) {
                       /*
                        * Add track to remote streams in dcb
                        */
-                      if (SDP_MEDIA_APPLICATION != media_type) {
+                      if (SDP_MEDIA_APPLICATION != media_type &&
+                          /* Do not expect to receive media if we're sendonly! */
+                          (media->direction == SDP_DIRECTION_SENDRECV ||
+                           media->direction == SDP_DIRECTION_RECVONLY)) {
                           int pc_stream_id = -1;
 
                           /* This is a hack to keep all the media in a single
                              stream.
                              TODO(ekr@rtfm.com): revisit when we have media
                              assigned to streams in the SDP */
                           if (!created_media_stream){
                               lsm_rc = lsm_add_remote_stream (dcb_p->line,