Procházet zdrojové kódy

cleanup: remove code no longer used in unified-plan mode

We switched to using replaceTrack for Firefox and Safari in the unified-plan rewrite
dev1
Jaya Allamsetty před 5 roky
rodič
revize
b5f85a4758

+ 2
- 30
modules/RTC/TraceablePeerConnection.js Zobrazit soubor

@@ -1555,11 +1555,7 @@ TraceablePeerConnection.prototype._addStream = function(mediaStream) {
1555 1555
  * @param {MediaStream} mediaStream
1556 1556
  */
1557 1557
 TraceablePeerConnection.prototype._removeStream = function(mediaStream) {
1558
-    if (browser.supportsRtpSender()) {
1559
-        this._handleSenderRemoveStream(mediaStream);
1560
-    } else {
1561
-        this.peerconnection.removeStream(mediaStream);
1562
-    }
1558
+    this.peerconnection.removeStream(mediaStream);
1563 1559
     this._addedStreams
1564 1560
         = this._addedStreams.filter(stream => stream !== mediaStream);
1565 1561
 };
@@ -1620,11 +1616,7 @@ TraceablePeerConnection.prototype.removeTrack = function(localTrack) {
1620 1616
     this.localSSRCs.delete(localTrack.rtcId);
1621 1617
 
1622 1618
     if (webRtcStream) {
1623
-        if (browser.supportsRtpSender()) {
1624
-            this._handleSenderRemoveStream(webRtcStream);
1625
-        } else {
1626
-            this.peerconnection.removeStream(webRtcStream);
1627
-        }
1619
+        this.peerconnection.removeStream(webRtcStream);
1628 1620
     }
1629 1621
 };
1630 1622
 
@@ -1741,26 +1733,6 @@ TraceablePeerConnection.prototype.removeTrackMute = function(localTrack) {
1741 1733
     return Promise.reject('Stream not found');
1742 1734
 };
1743 1735
 
1744
-/**
1745
- * Remove stream handling for browsers supporting RTPSender
1746
- * @param stream: webrtc media stream
1747
- */
1748
-TraceablePeerConnection.prototype._handleSenderRemoveStream = function(
1749
-        stream) {
1750
-    if (!stream) {
1751
-        // There is nothing to be changed
1752
-        return;
1753
-    }
1754
-
1755
-    const sender = this.findSenderByStream(stream);
1756
-
1757
-    if (sender) {
1758
-        this.peerconnection.removeTrack(sender);
1759
-    } else {
1760
-        logger.log('Cannot remove tracks: no RTPSender.');
1761
-    }
1762
-};
1763
-
1764 1736
 TraceablePeerConnection.prototype.createDataChannel = function(label, opts) {
1765 1737
     this.trace('createDataChannel', label, opts);
1766 1738
 

+ 0
- 9
modules/browser/BrowserCapabilities.js Zobrazit soubor

@@ -139,15 +139,6 @@ export default class BrowserCapabilities extends BrowserDetection {
139 139
         return !this.isFirefox();
140 140
     }
141 141
 
142
-    /**
143
-     * Checks whether the browser supports RTPSender.
144
-     *
145
-     * @returns {boolean}
146
-     */
147
-    supportsRtpSender() {
148
-        return this.isFirefox() || this.isSafari();
149
-    }
150
-
151 142
     /**
152 143
      * Returns whether or not the current browser can support capturing video,
153 144
      * be it camera or desktop, and displaying received video.

Načítá se…
Zrušit
Uložit