Ver código fonte

fix(Jingle): stop reverting the SSRCs from Firefox

tags/v0.0.2
Nils Ohlmeier 4 anos atrás
pai
commit
88ccab4d7e
2 arquivos alterados com 1 adições e 12 exclusões
  1. 0
    5
      modules/RTC/TraceablePeerConnection.js
  2. 1
    7
      modules/sdp/SDP.js

+ 0
- 5
modules/RTC/TraceablePeerConnection.js Ver arquivo

@@ -1482,11 +1482,6 @@ TraceablePeerConnection.prototype._injectSsrcGroupForUnifiedSimulcast
1482 1482
                 return desc;
1483 1483
             }
1484 1484
 
1485
-            // Reverse the order of the SSRCs when signaling them to the bridge. On Firefox, the first SSRC corresponds
1486
-            // to the highest resolution stream whereas the bridge assumes it to be that of the lowest resolution
1487
-            // stream as is the case with the other browsers.
1488
-            browser.isFirefox() && ssrcs.reverse();
1489
-
1490 1485
             video.ssrcGroups.push({
1491 1486
                 semantics: 'SIM',
1492 1487
                 ssrcs: ssrcs.join(' ')

+ 1
- 7
modules/sdp/SDP.js Ver arquivo

@@ -143,16 +143,10 @@ SDP.prototype.toJingle = function(elem, thecreator) {
143 143
         }
144 144
 
145 145
         let ssrc;
146
-        const simGroup = SDPUtil.findLine(this.media[i], 'a=ssrc-group:SIM');
147 146
         const assrcline = SDPUtil.findLine(this.media[i], 'a=ssrc:');
148 147
 
149 148
         if (assrcline) {
150
-            if (browser.isFirefox() && simGroup) {
151
-                // Use the first ssrc from the SIM group since the order of the SSRCs need to be reversed for Firefox.
152
-                ssrc = simGroup.substring(17).split(' ')[0];
153
-            } else {
154
-                ssrc = assrcline.substring(7).split(' ')[0]; // take the first
155
-            }
149
+            ssrc = assrcline.substring(7).split(' ')[0]; // take the first
156 150
         } else {
157 151
             ssrc = false;
158 152
         }

Carregando…
Cancelar
Salvar