From 8a4205bc8d28c185f964df97898298690142816d Mon Sep 17 00:00:00 2001 From: Evan Theurer Date: Mon, 2 Nov 2015 18:15:07 +0100 Subject: [PATCH] Fix depth of if statement firefox screensharing check. --- static/js/services/screensharing.js | 18 ++++++++---------- 1 file changed, 8 insertions(+), 10 deletions(-) diff --git a/static/js/services/screensharing.js b/static/js/services/screensharing.js index 28dd9ac8..b5f3cfc5 100644 --- a/static/js/services/screensharing.js +++ b/static/js/services/screensharing.js @@ -171,19 +171,17 @@ define(['underscore', 'text!partials/screensharedialogff.html', 'webrtc.adapter' } - } else if ($window.webrtcDetectedBrowser === "firefox") { + } else if ($window.webrtcDetectedBrowser === "firefox" && $window.webrtcDetectedVersion >= 36) { // Firefox 36 got screen sharing support. // See https://bugzilla.mozilla.org/show_bug.cgi?id=923225 - if ($window.webrtcDetectedVersion >= 36) { - this.prepare = function(options) { - return selectFirefoxScreenToShare(options); - }; - if (firefoxExtension.available) { - this.supported = true; - } else if (!firefoxExtension.autoinstall.force) { - this.supported = true; - } + this.prepare = function(options) { + return selectFirefoxScreenToShare(options); + }; + if (firefoxExtension.available) { + this.supported = true; + } else if (!firefoxExtension.autoinstall.force) { + this.supported = true; } } else {