Browse Source

fix(firefox): Fix simulcast on FF ESR

Make sure we don't have a=simulcast lines on the sendonly m-lines in the remote description
dev1
Jaya Allamsetty 5 years ago
parent
commit
3e20575048
2 changed files with 9 additions and 1 deletions
  1. 8
    0
      modules/RTC/TPCUtils.js
  2. 1
    1
      modules/RTC/TraceablePeerConnection.js

+ 8
- 0
modules/RTC/TPCUtils.js View File

@@ -122,8 +122,16 @@ export class TPCUtils {
122 122
                 if (mline.type === 'video' && i !== idx) {
123 123
                     sdp.media[i].rids = undefined;
124 124
                     sdp.media[i].simulcast = undefined;
125
+
126
+                    // eslint-disable-next-line camelcase
127
+                    sdp.media[i].simulcast_03 = undefined;
125 128
                 }
126 129
             });
130
+
131
+            return new RTCSessionDescription({
132
+                type: desc.type,
133
+                sdp: transform.write(sdp)
134
+            });
127 135
         }
128 136
 
129 137
         // In order of highest to lowest spatial quality

+ 1
- 1
modules/RTC/TraceablePeerConnection.js View File

@@ -1296,7 +1296,7 @@ TraceablePeerConnection.prototype._injectSsrcGroupForUnifiedSimulcast
1296 1296
         const sdp = transform.parse(desc.sdp);
1297 1297
         const video = sdp.media.find(mline => mline.type === 'video');
1298 1298
 
1299
-        if (video.simulcast) {
1299
+        if (video.simulcast || video.simulcast_03) {
1300 1300
             const ssrcs = [];
1301 1301
 
1302 1302
             video.ssrcs.forEach(ssrc => {

Loading…
Cancel
Save