From c97a29fd50fbcd3f2aa92ef3aa256ee510c46bbb Mon Sep 17 00:00:00 2001 From: pabloFuente Date: Wed, 4 Jul 2018 10:31:36 +0200 Subject: [PATCH] openvidu-browser BUG FIX: own remote subscription could launch streamPlaying event twice --- openvidu-browser/src/OpenVidu/StreamManager.ts | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/openvidu-browser/src/OpenVidu/StreamManager.ts b/openvidu-browser/src/OpenVidu/StreamManager.ts index 93fcedd2..14dabbf1 100644 --- a/openvidu-browser/src/OpenVidu/StreamManager.ts +++ b/openvidu-browser/src/OpenVidu/StreamManager.ts @@ -230,8 +230,6 @@ export class StreamManager implements EventDispatcher { let returnNumber = 1; - this.initializeVideoProperties(video); - for (const streamManager of this.stream.session.streamManagers) { if (streamManager.disassociateVideo(video)) { returnNumber = -1; @@ -321,7 +319,10 @@ export class StreamManager implements EventDispatcher { * @hidden */ initializeVideoProperties(video: HTMLVideoElement): void { - video.srcObject = this.stream.getMediaStream(); + if (!(this.stream.isLocal() && this.stream.displayMyRemote())) { + // Avoid setting the MediaStream into the srcObject if remote subscription before publishing + video.srcObject = this.stream.getMediaStream(); + } video.autoplay = true; video.controls = false; if (!video.id) {