Fixes an issue where the pinned video src was getting set when it shouldn't.

11 years ago · 2004b6ef18
parent 1ac99309e7
commit 2004b6ef18
1 changed files with 6 additions and 1 deletions
--- a/videolayout.js
+++ b/videolayout.js
@ -1609,7 +1609,12 @@ var VideoLayout = (function (my) {
                // and VideoLayout.getLargeVideoResource methods.
                var updateLargeVideo = (ssrc2jid[videoSrcToSsrc[selRemoteVideo.attr('src')]]
                    == ssrc2jid[videoSrcToSsrc[largeVideoState.newSrc]]);
-                var updateFocusedVideoSrc = (selRemoteVideo.attr('src') == focusedVideoSrc);
+
+                // We should only update the focused video src if it's not a
+                // falsy value.
+                var updateFocusedVideoSrc
+                    = focusedVideoSrc && focusedVideoSrc !== ''
+                        && (selRemoteVideo.attr('src') == focusedVideoSrc);

                var electedStreamUrl;
                if (largeVideoState.preload_ssrc == primarySSRC)