Listen on ontrack event on Safari. (#337)

jianjunz · jianjunz · commit 3e9919735695 · 2020-04-21T14:08:03.000+08:00
This commit only fixes issue for Safari. But we should eventually remove event handlers for both onaddstream and onremovestream. (cherry picked from commit bd0c9b6)
diff --git a/src/sdk/p2p/peerconnection-channel.js b/src/sdk/p2p/peerconnection-channel.js
@@ -661,7 +661,7 @@ class P2PPeerConnectionChannel extends EventDispatcher {
       this._pc.addTransceiver('audio');
       this._pc.addTransceiver('video');
     }
-    if (!this._isUnifiedPlan()) {
+    if (!this._isUnifiedPlan() && !Utils.isSafari()) {
       this._pc.onaddstream = (event) => {
         // TODO: Legacy API, should be removed when all UAs implemented WebRTC 1.0.
         this._onRemoteStreamAdded.apply(this, [event]);

Original file line number	Diff line number	Diff line change
`@@ -661,7 +661,7 @@ class P2PPeerConnectionChannel extends EventDispatcher {`
`661`	`661`	`this._pc.addTransceiver('audio');`
`662`	`662`	`this._pc.addTransceiver('video');`
`663`	`663`	`}`
`664`		`- if (!this._isUnifiedPlan()) {`
	`664`	`+ if (!this._isUnifiedPlan() && !Utils.isSafari()) {`
`665`	`665`	`this._pc.onaddstream = (event) => {`
`666`	`666`	`// TODO: Legacy API, should be removed when all UAs implemented WebRTC 1.0.`
`667`	`667`	`this._onRemoteStreamAdded.apply(this, [event]);`