From 42b51e3c5c44cb13637e9f639a323353b14e75a1 Mon Sep 17 00:00:00 2001 From: Leonard Kim Date: Wed, 24 May 2017 18:55:52 -0700 Subject: [PATCH] fix(1-on-1): remove torture test workaround, add new tests --- modules/UI/videolayout/Filmstrip.js | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/modules/UI/videolayout/Filmstrip.js b/modules/UI/videolayout/Filmstrip.js index f60bbf83e..24fb5eab1 100644 --- a/modules/UI/videolayout/Filmstrip.js +++ b/modules/UI/videolayout/Filmstrip.js @@ -40,11 +40,8 @@ const Filmstrip = { * @returns {void} */ setRemoteVideoVisibility(shouldShow) { - // FIXME Checking config.debug is a grand hack to avoid fixing the - // torture tests after the 1-on-1 UI was implemented, which hides remote - // videos on 1-on-1 calls. If this check is to be kept, at least create - // new torture tests to verify 1-on-1 mode. - if (config.debug || config.disable1On1Mode) { + // Allow disabling on 1-on-1 UI mode. Used by torture tests. + if (config.disable1On1Mode) { return; }