fix(eslint): Add curly rule

.eslintrc.js

@@ -75,6 +75,7 @@ module.exports = {
         'block-scoped-var': 0,
         'complexity': 0,
         'consistent-return': 0,
+        'curly': 2,
 
         'prefer-const': 2,
         'prefer-reflect': 0,

JitsiConference.js

@@ -95,7 +95,7 @@ function JitsiConference(options) {
  */
 JitsiConference.prototype._init = function (options) {
     if (!options)
-        options = {};
+        {options = {};}
 
     // Override connection and xmpp properties (Usefull if the connection
     // reloaded)
@@ -152,7 +152,7 @@ JitsiConference.prototype._init = function (options) {
  */
 JitsiConference.prototype.join = function (password) {
     if (this.room)
-        this.room.join(password);
+        {this.room.join(password);}
 };
 
 /**
@@ -176,7 +176,7 @@ JitsiConference.prototype.leave = function () {
 
     this.rtc.closeAllDataChannels();
     if (this.statistics)
-        this.statistics.dispose();
+        {this.statistics.dispose();}
 
     // leave the conference
     if (this.room) {
@@ -295,7 +295,7 @@ JitsiConference.prototype.getLocalVideoTrack = function () {
  */
 JitsiConference.prototype.on = function (eventId, handler) {
     if (this.eventEmitter)
-        this.eventEmitter.on(eventId, handler);
+        {this.eventEmitter.on(eventId, handler);}
 };
 
 /**
@@ -307,7 +307,7 @@ JitsiConference.prototype.on = function (eventId, handler) {
  */
 JitsiConference.prototype.off = function (eventId, handler) {
     if (this.eventEmitter)
-        this.eventEmitter.removeListener(eventId, handler);
+        {this.eventEmitter.removeListener(eventId, handler);}
 };
 
 // Common aliases for event emitter
@@ -322,7 +322,7 @@ JitsiConference.prototype.removeEventListener = JitsiConference.prototype.off;
  */
  JitsiConference.prototype.addCommandListener = function (command, handler) {
     if (this.room)
-        this.room.addPresenceListener(command, handler);
+        {this.room.addPresenceListener(command, handler);}
  };
 
 /**
@@ -331,7 +331,7 @@ JitsiConference.prototype.removeEventListener = JitsiConference.prototype.off;
   */
  JitsiConference.prototype.removeCommandListener = function (command) {
     if (this.room)
-        this.room.removePresenceListener(command);
+        {this.room.removePresenceListener(command);}
  };
 
 /**
@@ -340,7 +340,7 @@ JitsiConference.prototype.removeEventListener = JitsiConference.prototype.off;
  */
 JitsiConference.prototype.sendTextMessage = function (message) {
     if (this.room)
-        this.room.sendMessage(message);
+        {this.room.sendMessage(message);}
 };
 
 /**
@@ -371,7 +371,7 @@ JitsiConference.prototype.sendCommandOnce = function (name, values) {
  **/
 JitsiConference.prototype.removeCommand = function (name) {
     if (this.room)
-        this.room.removeFromPresence(name);
+        {this.room.removeFromPresence(name);}
 };
 
 /**
@@ -488,7 +488,7 @@ JitsiConference.prototype.onLocalTrackRemoved = function (track) {
     // FIXME: we assume we have only one screen sharing track
     // if we change this we need to fix this check
     if (track.isVideoTrack() && track.videoType === VideoType.DESKTOP)
-        this.statistics.sendScreenSharingEvent(false);
+        {this.statistics.sendScreenSharingEvent(false);}
 
     this.eventEmitter.emit(JitsiConferenceEvents.TRACK_REMOVED, track);
 };
@@ -626,7 +626,7 @@ JitsiConference.prototype._setupNewTrack = function (newTrack) {
     // FIXME: we assume we have only one screen sharing track
     // if we change this we need to fix this check
     if (newTrack.isVideoTrack() && newTrack.videoType === VideoType.DESKTOP)
-        this.statistics.sendScreenSharingEvent(true);
+        {this.statistics.sendScreenSharingEvent(true);}
 
     this.eventEmitter.emit(JitsiConferenceEvents.TRACK_ADDED, newTrack);
 };
@@ -878,8 +878,8 @@ JitsiConference.prototype.onMemberLeft = function (jid) {
 
     // there can be no participant in case the member that left is focus
     if (participant)
-        this.eventEmitter.emit(
-            JitsiConferenceEvents.USER_LEFT, id, participant);
+        {this.eventEmitter.emit(
+            JitsiConferenceEvents.USER_LEFT, id, participant);}
 };
 
 JitsiConference.prototype.onUserRoleChanged = function (jid, role) {
@@ -900,7 +900,7 @@ JitsiConference.prototype.onDisplayNameChanged = function (jid, displayName) {
     }
 
     if (participant._displayName === displayName)
-        return;
+        {return;}
 
     participant._displayName = displayName;
     this.eventEmitter.emit(JitsiConferenceEvents.DISPLAY_NAME_CHANGED, id, displayName);
@@ -1218,7 +1218,7 @@ JitsiConference.prototype.sendTones = function (tones, duration, pause) {
  */
 JitsiConference.prototype.isRecordingSupported = function () {
     if (this.room)
-        return this.room.isRecordingSupported();
+        {return this.room.isRecordingSupported();}
     return false;
 };
 
@@ -1242,10 +1242,10 @@ JitsiConference.prototype.getRecordingURL = function () {
  */
 JitsiConference.prototype.toggleRecording = function (options) {
     if (this.room)
-        return this.room.toggleRecording(options, function (status, error) {
+        {return this.room.toggleRecording(options, function (status, error) {
             this.eventEmitter.emit(
                 JitsiConferenceEvents.RECORDER_STATE_CHANGED, status, error);
-        }.bind(this));
+        }.bind(this));}
     this.eventEmitter.emit(
         JitsiConferenceEvents.RECORDER_STATE_CHANGED, "error",
         new Error("The conference is not created yet!"));
@@ -1256,7 +1256,7 @@ JitsiConference.prototype.toggleRecording = function (options) {
  */
 JitsiConference.prototype.isSIPCallingSupported = function () {
     if (this.room)
-        return this.room.isSIPCallingSupported();
+        {return this.room.isSIPCallingSupported();}
     return false;
 };
 
@@ -1266,7 +1266,7 @@ JitsiConference.prototype.isSIPCallingSupported = function () {
  */
 JitsiConference.prototype.dial = function (number) {
     if (this.room)
-        return this.room.dial(number);
+        {return this.room.dial(number);}
     return new Promise(function(resolve, reject){
         reject(new Error("The conference is not created yet!"));});
 };
@@ -1276,7 +1276,7 @@ JitsiConference.prototype.dial = function (number) {
  */
 JitsiConference.prototype.hangup = function () {
     if (this.room)
-        return this.room.hangup();
+        {return this.room.hangup();}
     return new Promise(function(resolve, reject){
         reject(new Error("The conference is not created yet!"));});
 };
@@ -1286,7 +1286,7 @@ JitsiConference.prototype.hangup = function () {
  */
 JitsiConference.prototype.getPhoneNumber = function () {
     if (this.room)
-        return this.room.getPhoneNumber();
+        {return this.room.getPhoneNumber();}
     return null;
 };
 
@@ -1295,7 +1295,7 @@ JitsiConference.prototype.getPhoneNumber = function () {
  */
 JitsiConference.prototype.getPhonePin = function () {
     if (this.room)
-        return this.room.getPhonePin();
+        {return this.room.getPhonePin();}
     return null;
 };
 

JitsiConferenceEventManager.js

@@ -24,7 +24,7 @@ function JitsiConferenceEventManager(conference) {
     conference.on(JitsiConferenceEvents.TRACK_MUTE_CHANGED,
         function (track) {
             if(!track.isLocal() || !conference.statistics)
-                return;
+                {return;}
             conference.statistics.sendMuteEvent(track.isMuted(),
                 track.getType());
         });
@@ -528,12 +528,12 @@ JitsiConferenceEventManager.prototype.setupXMPPListeners = function () {
 JitsiConferenceEventManager.prototype.setupStatisticsListeners = function () {
     var conference = this.conference;
     if(!conference.statistics)
-        return;
+        {return;}
 
     conference.statistics.addAudioLevelListener(function (ssrc, level) {
         var resource = conference.rtc.getResourceBySSRC(ssrc);
         if (!resource)
-            return;
+            {return;}
 
         conference.rtc.setAudioLevel(resource, level);
     });
@@ -579,7 +579,7 @@ JitsiConferenceEventManager.prototype.setupStatisticsListeners = function () {
         conference.getLocalTracks(MediaType.AUDIO).forEach(function (track) {
             const ssrc = track.getSSRC();
             if (!ssrc || !stats.hasOwnProperty(ssrc))
-                return;
+                {return;}
 
             track._setByteSent(stats[ssrc]);
         });

JitsiConnection.js

@@ -30,8 +30,8 @@ function JitsiConnection(appID, token, options) {
             // and then there are no msgs, but we want to log only disconnects
             // when there is real error
             if(msg)
-                Statistics.analytics.sendEvent(
-                    'connection.disconnected.' + msg);
+                {Statistics.analytics.sendEvent(
+                    'connection.disconnected.' + msg);}
             Statistics.sendLog(
                 JSON.stringify({id: "connection.disconnected", msg: msg}));
         });
@@ -44,7 +44,7 @@ function JitsiConnection(appID, token, options) {
  */
 JitsiConnection.prototype.connect = function (options) {
     if(!options)
-        options = {};
+        {options = {};}
 
     this.xmpp.connect(options.id, options.password);
 };

JitsiMeetJS.js

@@ -32,7 +32,7 @@ var USER_MEDIA_PERMISSION_PROMPT_TIMEOUT = 500;
 
 function getLowerResolution(resolution) {
     if(!Resolutions[resolution])
-        return null;
+        {return null;}
     var order = Resolutions[resolution].order;
     var res = null;
     var resName = null;
@@ -224,7 +224,7 @@ var LibJitsiMeet = {
         }
 
         if(!window.connectionTimes)
-            window.connectionTimes = {};
+            {window.connectionTimes = {};}
         window.connectionTimes["obtainPermissions.start"] =
             window.performance.now();
 
@@ -239,7 +239,7 @@ var LibJitsiMeet = {
                     "getUserMedia.success", options), {value: options});
 
                 if(!RTC.options.disableAudioLevels)
-                    for(let i = 0; i < tracks.length; i++) {
+                    {for(let i = 0; i < tracks.length; i++) {
                         const track = tracks[i];
                         var mStream = track.getOriginalStream();
                         if(track.getType() === MediaType.AUDIO){
@@ -251,7 +251,7 @@ var LibJitsiMeet = {
                                     Statistics.stopLocalStats(mStream);
                                 });
                         }
-                    }
+                    }}
 
                 // set real device ids
                 var currentlyAvailableMediaDevices

doc/example/example.js

@@ -48,7 +48,7 @@ function onLocalTracks(tracks)
             localTracks[i].attach($("#localAudio" + i)[0]);
         }
         if(isJoined)
-            room.addTrack(localTracks[i]);
+            {room.addTrack(localTracks[i]);}
     }
 }
 
@@ -58,10 +58,10 @@ function onLocalTracks(tracks)
  */
 function onRemoteTrack(track) {
     if(track.isLocal())
-        return;
+        {return;}
     var participant = track.getParticipantId();
     if(!remoteTracks[participant])
-        remoteTracks[participant] = [];
+        {remoteTracks[participant] = [];}
     var idx = remoteTracks[participant].push(track);
     track.addEventListener(JitsiMeetJS.events.track.TRACK_AUDIO_LEVEL_CHANGED,
         function (audioLevel) {
@@ -95,16 +95,16 @@ function onConferenceJoined () {
     console.log("conference joined!");
     isJoined = true;
     for(var i = 0; i < localTracks.length; i++)
-        room.addTrack(localTracks[i]);
+        {room.addTrack(localTracks[i]);}
 }
 
 function onUserLeft(id) {
     console.log("user left");
     if(!remoteTracks[id])
-        return;
+        {return;}
     var tracks = remoteTracks[id];
     for(var i = 0; i< tracks.length; i++)
-        tracks[i].detach($("#" + id + tracks[i].getType()));
+        {tracks[i].detach($("#" + id + tracks[i].getType()));}
 }
 
 /**
@@ -168,7 +168,7 @@ function disconnect(){
 
 function unload() {
     for(var i = 0; i < localTracks.length; i++)
-        localTracks[i].stop();
+        {localTracks[i].stop();}
     room.leave();
     connection.disconnect();
 }

modules/RTC/DataChannels.js

@@ -163,7 +163,7 @@ DataChannels.prototype.onDataChannel = function (event) {
         logger.info("The Data Channel closed", dataChannel);
         var idx = self._dataChannels.indexOf(dataChannel);
         if (idx > -1)
-            self._dataChannels = self._dataChannels.splice(idx, 1);
+            {self._dataChannels = self._dataChannels.splice(idx, 1);}
     };
     this._dataChannels.push(dataChannel);
 };
@@ -240,9 +240,9 @@ DataChannels.prototype._some = function (callback, thisArg) {
 
     if (dataChannels && dataChannels.length !== 0) {
         if (thisArg)
-            return dataChannels.some(callback, thisArg);
+            {return dataChannels.some(callback, thisArg);}
         else
-            return dataChannels.some(callback);
+            {return dataChannels.some(callback);}
     } else {
         return false;
     }

modules/RTC/JitsiLocalTrack.js

@@ -35,8 +35,8 @@ function JitsiLocalTrack(stream, track, mediaType, videoType, resolution,
         null /* RTC */, stream, track,
         function () {
             if(!this.dontFireRemoveEvent)
-                this.eventEmitter.emit(
-                    JitsiTrackEvents.LOCAL_TRACK_STOPPED);
+                {this.eventEmitter.emit(
+                    JitsiTrackEvents.LOCAL_TRACK_STOPPED);}
             this.dontFireRemoveEvent = false;
         }.bind(this) /* inactiveHandler */,
         mediaType, videoType, null /* ssrc */);
@@ -46,7 +46,7 @@ function JitsiLocalTrack(stream, track, mediaType, videoType, resolution,
     // FIXME: currently firefox is ignoring our constraints about resolutions
     // so we do not store it, to avoid wrong reporting of local track resolution
     if (RTCBrowserType.isFirefox())
-        this.resolution = null;
+        {this.resolution = null;}
 
     this.deviceId = deviceId;
     this.startMuted = false;
@@ -183,7 +183,7 @@ JitsiLocalTrack.prototype._clearNoDataFromSourceMuteResources = function () {
 JitsiLocalTrack.prototype._onNoDataFromSourceError = function () {
     this._clearNoDataFromSourceMuteResources();
     if(this._checkForCameraIssues())
-        this._fireNoDataFromSourceEvent();
+        {this._fireNoDataFromSourceEvent();}
 };
 
 /**
@@ -293,7 +293,7 @@ JitsiLocalTrack.prototype._setMute = function (mute) {
         this.videoType === VideoType.DESKTOP ||
         RTCBrowserType.isFirefox()) {
         if(this.track)
-            this.track.enabled = !mute;
+            {this.track.enabled = !mute;}
     } else {
         if(mute) {
             this.dontFireRemoveEvent = true;
@@ -316,7 +316,7 @@ JitsiLocalTrack.prototype._setMute = function (mute) {
                 facingMode: this.getCameraFacingMode()
             };
             if (this.resolution)
-                streamOptions.resolution = this.resolution;
+                {streamOptions.resolution = this.resolution;}
 
             promise = RTCUtils.obtainAudioAndVideoPermissions(streamOptions)
                 .then(function (streamsInfo) {
@@ -475,7 +475,7 @@ JitsiLocalTrack.prototype.dispose = function () {
 JitsiLocalTrack.prototype.isMuted = function () {
     // this.stream will be null when we mute local video on Chrome
     if (!this.stream)
-        return true;
+        {return true;}
     if (this.isVideoTrack() && !this.isActive()) {
         return true;
     } else {
@@ -519,11 +519,11 @@ JitsiLocalTrack.prototype._setConference = function(conference) {
  */
 JitsiLocalTrack.prototype.getSSRC = function () {
     if(this.ssrc && this.ssrc.groups && this.ssrc.groups.length)
-        return this.ssrc.groups[0].ssrcs[0];
+        {return this.ssrc.groups[0].ssrcs[0];}
     else if(this.ssrc && this.ssrc.ssrcs && this.ssrc.ssrcs.length)
-        return this.ssrc.ssrcs[0];
+        {return this.ssrc.ssrcs[0];}
     else
-        return null;
+        {return null;}
 };
 
 /**
@@ -622,7 +622,7 @@ JitsiLocalTrack.prototype._stopMediaStream = function () {
 JitsiLocalTrack.prototype._checkForCameraIssues = function () {
     if(!this.isVideoTrack() || this.stopStreamInProgress ||
         this.videoType === VideoType.DESKTOP)
-        return false;
+        {return false;}
 
     return !this._isReceivingData();
 };
@@ -638,7 +638,7 @@ JitsiLocalTrack.prototype._checkForCameraIssues = function () {
  */
 JitsiLocalTrack.prototype._isReceivingData = function () {
     if(!this.stream)
-        return false;
+        {return false;}
     // In older version of the spec there is no muted property and
     // readyState can have value muted. In the latest versions
     // readyState can have values "live" and "ended" and there is

modules/RTC/JitsiRemoteTrack.js

@@ -38,7 +38,7 @@ function JitsiRemoteTrack(rtc, conference, ownerEndpointId, stream, track,
     this.hasBeenMuted = muted;
     // Bind 'onmute' and 'onunmute' event handlers
     if (this.rtc && this.track)
-        this._bindMuteHandlers();
+        {this._bindMuteHandlers();}
 }
 
 JitsiRemoteTrack.prototype = Object.create(JitsiTrack.prototype);
@@ -77,14 +77,14 @@ JitsiRemoteTrack.prototype._bindMuteHandlers = function() {
  */
 JitsiRemoteTrack.prototype.setMute = function (value) {
     if(this.muted === value)
-        return;
+        {return;}
 
     if(value)
-        this.hasBeenMuted = true;
+        {this.hasBeenMuted = true;}
 
     // we can have a fake video stream
     if(this.stream)
-        this.stream.muted = value;
+        {this.stream.muted = value;}
 
     this.muted = value;
     this.eventEmitter.emit(JitsiTrackEvents.TRACK_MUTE_CHANGED, this);
@@ -129,7 +129,7 @@ JitsiRemoteTrack.prototype.getSSRC = function () {
  */
 JitsiRemoteTrack.prototype._setVideoType = function (type) {
     if(this.videoType === type)
-        return;
+        {return;}
     this.videoType = type;
     this.eventEmitter.emit(JitsiTrackEvents.TRACK_VIDEOTYPE_CHANGED, type);
 };
@@ -150,7 +150,7 @@ JitsiRemoteTrack.prototype._playCallback = function () {
     console.log("(TIME) TTFM " + type + ":\t", ttfm);
     var eventName = type +'.ttfm';
     if(this.hasBeenMuted)
-        eventName += '.muted';
+        {eventName += '.muted';}
     Statistics.analytics.sendEvent(eventName, {value: ttfm});
 };
 
@@ -165,12 +165,12 @@ JitsiRemoteTrack.prototype._playCallback = function () {
 JitsiRemoteTrack.prototype._attachTTFMTracker = function (container) {
     if((ttfmTrackerAudioAttached && this.isAudioTrack())
         || (ttfmTrackerVideoAttached && this.isVideoTrack()))
-        return;
+        {return;}
 
     if (this.isAudioTrack())
-        ttfmTrackerAudioAttached = true;
+        {ttfmTrackerAudioAttached = true;}
     if (this.isVideoTrack())
-        ttfmTrackerVideoAttached = true;
+        {ttfmTrackerVideoAttached = true;}
 
     if (RTCBrowserType.isTemasysPluginUsed()) {
         // XXX Don't require Temasys unless it's to be used because it doesn't

modules/RTC/JitsiTrack.js

@@ -27,7 +27,7 @@ function implementOnEndedHandling(jitsiTrack) {
     var stream = jitsiTrack.getOriginalStream();
 
     if(!stream)
-        return;
+        {return;}
 
     var originalStop = stream.stop;
     stream.stop = function () {
@@ -46,9 +46,9 @@ function implementOnEndedHandling(jitsiTrack) {
 function addMediaStreamInactiveHandler(mediaStream, handler) {
     // Temasys will use onended
     if(typeof mediaStream.active !== "undefined")
-        mediaStream.oninactive = handler;
+        {mediaStream.oninactive = handler;}
     else
-        mediaStream.onended = handler;
+        {mediaStream.onended = handler;}
 }
 
 /**
@@ -102,7 +102,7 @@ function JitsiTrack(conference, stream, track, streamInactiveHandler, trackMedia
 JitsiTrack.prototype._setHandler = function (type, handler) {
     this.handlers[type] = handler;
     if(!this.stream)
-        return;
+        {return;}
 
     if(type === "inactive") {
         if (RTCBrowserType.isFirefox()) {
@@ -316,9 +316,9 @@ JitsiTrack.prototype.isScreenSharing = function() {
  */
 JitsiTrack.prototype.getId = function () {
     if(this.stream)
-        return RTCUtils.getStreamID(this.stream);
+        {return RTCUtils.getStreamID(this.stream);}
     else
-        return null;
+        {return null;}
 };
 
 /**
@@ -329,9 +329,9 @@ JitsiTrack.prototype.getId = function () {
  */
 JitsiTrack.prototype.isActive = function () {
     if(typeof this.stream.active !== "undefined")
-        return this.stream.active;
+        {return this.stream.active;}
     else
-        return true;
+        {return true;}
 };
 
 /**
@@ -342,7 +342,7 @@ JitsiTrack.prototype.isActive = function () {
  */
 JitsiTrack.prototype.on = function (eventId, handler) {
     if(this.eventEmitter)
-        this.eventEmitter.on(eventId, handler);
+        {this.eventEmitter.on(eventId, handler);}
 };
 
 /**
@@ -352,7 +352,7 @@ JitsiTrack.prototype.on = function (eventId, handler) {
  */
 JitsiTrack.prototype.off = function (eventId, handler) {
     if(this.eventEmitter)
-        this.eventEmitter.removeListener(eventId, handler);
+        {this.eventEmitter.removeListener(eventId, handler);}
 };
 
 // Common aliases for event emitter

modules/RTC/RTC.js

@@ -165,7 +165,7 @@ export default class RTC extends Listenable {
         // cache the value if channel is missing, till we open it
         this.selectedEndpoint = id;
         if(this.dataChannels && this.dataChannelsOpen)
-            this.dataChannels.sendSelectedEndpointMessage(id);
+            {this.dataChannels.sendSelectedEndpointMessage(id);}
     }
 
     /**
@@ -253,7 +253,7 @@ export default class RTC extends Listenable {
 
     addLocalTrack (track) {
         if (!track)
-            throw new Error('track must not be null nor undefined');
+            {throw new Error('track must not be null nor undefined');}
 
         this.localTracks.push(track);
 
@@ -331,9 +331,9 @@ export default class RTC extends Listenable {
      */
     getRemoteTrackByType (type, resource) {
         if (this.remoteTracks[resource])
-            return this.remoteTracks[resource][type];
+            {return this.remoteTracks[resource][type];}
         else
-            return null;
+            {return null;}
     }
 
     /**
@@ -633,7 +633,7 @@ export default class RTC extends Listenable {
 
     setAudioLevel (resource, audioLevel) {
         if(!resource)
-            return;
+            {return;}
         var audioTrack = this.getRemoteAudioTrack(resource);
         if(audioTrack) {
             audioTrack.setAudioLevel(audioLevel);

modules/RTC/RTCBrowserType.js

@@ -336,7 +336,7 @@ function detectBrowser() {
     for (var i = 0; i < detectors.length; i++) {
         version = detectors[i]();
         if (version)
-            return version;
+            {return version;}
     }
     logger.warn("Browser type defaults to Safari ver 1");
     currentBrowser = RTCBrowserType.RTC_BROWSER_SAFARI;

modules/RTC/RTCUtils.js

@@ -117,11 +117,11 @@ function setResolutionConstraints(constraints, resolution) {
     }
 
     if (constraints.video.mandatory.minWidth)
-        constraints.video.mandatory.maxWidth =
-            constraints.video.mandatory.minWidth;
+        {constraints.video.mandatory.maxWidth =
+            constraints.video.mandatory.minWidth;}
     if (constraints.video.mandatory.minHeight)
-        constraints.video.mandatory.maxHeight =
-            constraints.video.mandatory.minHeight;
+        {constraints.video.mandatory.maxHeight =
+            constraints.video.mandatory.minHeight;}
 }
 
 /**
@@ -772,7 +772,7 @@ class RTCUtils extends Listenable {
                     if (element) {
                         defaultSetVideoSrc(element, stream);
                         if (stream)
-                            element.play();
+                            {element.play();}
                     }
                     return element;
                 });
@@ -1146,7 +1146,7 @@ class RTCUtils extends Listenable {
 
     _isDeviceListAvailable () {
         if (!rtcReady)
-            throw new Error("WebRTC not ready yet");
+            {throw new Error("WebRTC not ready yet");}
         var isEnumerateDevicesAvailable
             = navigator.mediaDevices && navigator.mediaDevices.enumerateDevices;
         if (isEnumerateDevicesAvailable) {

modules/RTC/ScreenObtainer.js

@@ -74,7 +74,7 @@ var ScreenObtainer = {
         gumFunction = gum;
 
         if (RTCBrowserType.isFirefox())
-            initFirefoxExtensionDetection(options);
+            {initFirefoxExtensionDetection(options);}
 
         if (RTCBrowserType.isNWJS()) {
             obtainDesktopStream = (options, onSuccess, onFailure) => {
@@ -207,7 +207,7 @@ var ScreenObtainer = {
             window.setTimeout(
                 () => {
                     if (firefoxExtInstalled === null)
-                        firefoxExtInstalled = false;
+                        {firefoxExtInstalled = false;}
                     this.obtainScreenOnFirefox(callback, errorCallback);
                 },
                 300);
@@ -354,9 +354,9 @@ function isUpdateRequired(minVersion, extVersion) {
                 n2 = 0;
 
             if (i < s1.length)
-                n1 = parseInt(s1[i]);
+                {n1 = parseInt(s1[i]);}
             if (i < s2.length)
-                n2 = parseInt(s2[i]);
+                {n2 = parseInt(s2[i]);}
 
             if (isNaN(n1) || isNaN(n2)) {
                 return true;
@@ -534,7 +534,7 @@ function initFirefoxExtensionDetection(options) {
         return;
     }
     if (firefoxExtInstalled === false || firefoxExtInstalled === true)
-        return;
+        {return;}
     if (!options.desktopSharingFirefoxExtId) {
         firefoxExtInstalled = false;
         return;

modules/RTC/TraceablePeerConnection.js

@@ -607,7 +607,7 @@ Object.keys(getters).forEach(function (prop) {
 TraceablePeerConnection.prototype.addStream = function (stream, ssrcInfo) {
     this.trace('addStream', stream ? stream.id : "null");
     if (stream)
-        this.peerconnection.addStream(stream);
+        {this.peerconnection.addStream(stream);}
     if (ssrcInfo && ssrcInfo.type === "addMuted") {
         this.sdpConsistency.setPrimarySsrc(ssrcInfo.ssrcs[0]);
         const simGroup
@@ -913,7 +913,7 @@ TraceablePeerConnection.prototype.getStats = function(callback, errback) {
             || RTCBrowserType.isReactNative()) {
         // ignore for now...
         if(!errback)
-            errback = function () {};
+            {errback = function () {};}
         this.peerconnection.getStats(null, callback, errback);
     } else {
         this.peerconnection.getStats(callback);

modules/TalkMutedDetection.js

@@ -60,7 +60,7 @@ export default class TalkMutedDetection {
         // We are interested in the local audio stream only and if event is not
         // sent yet.
         if (!isLocal || !this.audioTrack || this._eventFired)
-            return;
+            {return;}
 
         if (this.audioTrack.isMuted() && audioLevel > 0.6) {
             this._eventFired = true;
@@ -92,7 +92,7 @@ export default class TalkMutedDetection {
      */
     _trackAdded(track) {
         if (this._isLocalAudioTrack(track))
-            this.audioTrack = track;
+            {this.audioTrack = track;}
     }
 
     /**
@@ -105,6 +105,6 @@ export default class TalkMutedDetection {
      */
     _trackMuteChanged(track) {
         if (this._isLocalAudioTrack(track) && track.isMuted())
-            this._eventFired = false;
+            {this._eventFired = false;}
     }
 }

modules/statistics/CallStats.js

@@ -213,7 +213,7 @@ CallStats.feedbackEnabled = false;
 CallStats._checkInitialize = function () {
     if (CallStats.initialized || !CallStats.initializeFailed
         || !callStats || CallStats.initializeInProgress)
-        return;
+        {return;}
 
     // callstats object created, not initialized and it had previously failed,
     // and there is no init in progress, so lets try initialize it again
@@ -237,7 +237,7 @@ var reportType = {
 
 CallStats.prototype.pcCallback = _try_catch(function (err, msg) {
     if (callStats && err !== 'success')
-        logger.error("Monitoring status: "+ err + " msg: " + msg);
+        {logger.error("Monitoring status: "+ err + " msg: " + msg);}
 });
 
 /**

modules/statistics/LocalStatsCollector.js

@@ -47,7 +47,7 @@ function timeDomainDataToAudioLevel(samples) {
 
     for (var i = 0; i < length; i++) {
         if (maxVolume < samples[i])
-            maxVolume = samples[i];
+            {maxVolume = samples[i];}
     }
 
     return parseFloat(((maxVolume - 127) / 128).toFixed(3));
@@ -98,7 +98,7 @@ function LocalStatsCollector(stream, interval, callback) {
 LocalStatsCollector.prototype.start = function () {
     if (!context ||
         RTCBrowserType.isTemasysPluginUsed())
-        return;
+        {return;}
     context.resume();
     var analyser = context.createAnalyser();
     analyser.smoothingTimeConstant = WEBAUDIO_ANALYZER_SMOOTING_TIME;

modules/statistics/RTPStatsCollector.js

@@ -65,7 +65,7 @@ KEYS_BY_BROWSER_TYPE[RTCBrowserType.RTC_BROWSER_REACT_NATIVE] =
  */
 function calculatePacketLoss(lostPackets, totalPackets) {
     if(!totalPackets || totalPackets <= 0 || !lostPackets || lostPackets <= 0)
-        return 0;
+        {return 0;}
     return Math.round((lostPackets/totalPackets)*100);
 }
 
@@ -180,7 +180,7 @@ function StatsCollector(
     this._browserType = RTCBrowserType.getBrowserType();
     var keys = KEYS_BY_BROWSER_TYPE[this._browserType];
     if (!keys)
-        throw "The browser type '" + this._browserType + "' isn't supported!";
+        {throw "The browser type '" + this._browserType + "' isn't supported!";}
     /**
      * The function which is to be used to retrieve the value associated in a
      * report returned by RTCPeerConnection#getStats with a LibJitsiMeet
@@ -316,9 +316,9 @@ StatsCollector.prototype._defineGetStatValueMethod = function (keys) {
     var keyFromName = function (name) {
         var key = keys[name];
         if (key)
-            return key;
+            {return key;}
         else
-            throw "The property '" + name + "' isn't supported!";
+            {throw "The property '" + name + "' isn't supported!";}
     };
 
     // Define the function which retrieves the value from a specific report
@@ -416,7 +416,7 @@ StatsCollector.prototype.processStatsReport = function () {
             }
             catch(e){/*not supported*/}
             if(!ip || !type || !localip || active != "true")
-                continue;
+                {continue;}
             // Save the address unless it has been saved already.
             var conferenceStatsTransport = this.conferenceStats.transport;
             if(!conferenceStatsTransport.some(function (t) { return (
@@ -430,7 +430,7 @@ StatsCollector.prototype.processStatsReport = function () {
 
         if(now.type == "candidatepair") {
             if(now.state == "succeeded")
-                continue;
+                {continue;}
 
             var local = this.currentStatsReport[now.localCandidateId];
             var remote = this.currentStatsReport[now.remoteCandidateId];
@@ -469,7 +469,7 @@ StatsCollector.prototype.processStatsReport = function () {
             }
         }
         if (!packetsNow || packetsNow < 0)
-            packetsNow = 0;
+            {packetsNow = 0;}
 
         var packetsBefore = getNonNegativeStat(before, key);
         var packetsDiff = Math.max(0, packetsNow - packetsBefore);
@@ -612,7 +612,7 @@ StatsCollector.prototype.processAudioLevelReport = function () {
         var now = this.currentAudioLevelsReport[idx];
 
         if (now.type != 'ssrc')
-            continue;
+            {continue;}
 
         var before = this.baselineAudioLevelsReport[idx];
         var ssrc = getStatValue(now, 'ssrc');
@@ -623,7 +623,7 @@ StatsCollector.prototype.processAudioLevelReport = function () {
 
         if (!ssrc) {
             if ((Date.now() - now.timestamp) < 3000)
-                logger.warn("No ssrc: ");
+                {logger.warn("No ssrc: ");}
             continue;
         }
 

modules/statistics/statistics.js

@@ -90,7 +90,7 @@ function Statistics(xmpp, options) {
             // requests to any third parties.
             && (Statistics.disableThirdPartyRequests !== true);
     if(this.callStatsIntegrationEnabled)
-        loadCallStatsAPI(this.options.callStatsCustomScriptUrl);
+        {loadCallStatsAPI(this.options.callStatsCustomScriptUrl);}
     this.callStats = null;
     // Flag indicates whether or not the CallStats have been started for this
     // Statistics instance
@@ -125,7 +125,7 @@ Statistics.localStats = [];
 
 Statistics.startLocalStats = function (stream, callback) {
     if(!Statistics.audioLevelsEnabled)
-        return;
+        {return;}
     var localStats = new LocalStats(stream, Statistics.audioLevelsInterval,
         callback);
     this.localStats.push(localStats);
@@ -134,13 +134,13 @@ Statistics.startLocalStats = function (stream, callback) {
 
 Statistics.prototype.addAudioLevelListener = function(listener) {
     if(!Statistics.audioLevelsEnabled)
-        return;
+        {return;}
     this.eventEmitter.on(StatisticsEvents.AUDIO_LEVEL, listener);
 };
 
 Statistics.prototype.removeAudioLevelListener = function(listener) {
     if(!Statistics.audioLevelsEnabled)
-        return;
+        {return;}
     this.eventEmitter.removeListener(StatisticsEvents.AUDIO_LEVEL, listener);
 };
 
@@ -177,19 +177,19 @@ Statistics.prototype.dispose = function () {
     this.stopCallStats();
     this.stopRemoteStats();
     if(this.eventEmitter)
-        this.eventEmitter.removeAllListeners();
+        {this.eventEmitter.removeAllListeners();}
 };
 
 Statistics.stopLocalStats = function (stream) {
     if(!Statistics.audioLevelsEnabled)
-        return;
+        {return;}
 
     for(var i = 0; i < Statistics.localStats.length; i++)
-        if(Statistics.localStats[i].stream === stream){
+        {if(Statistics.localStats[i].stream === stream){
             var localStats = Statistics.localStats.splice(i, 1);
             localStats[0].stop();
             break;
-        }
+        }}
 };
 
 Statistics.prototype.stopRemoteStats = function () {
@@ -224,7 +224,7 @@ Statistics.prototype.stopCallStats = function () {
     if(this.callStatsStarted) {
         var index = Statistics.callsStatsInstances.indexOf(this.callstats);
         if(index > -1)
-            Statistics.callsStatsInstances.splice(index, 1);
+            {Statistics.callsStatsInstances.splice(index, 1);}
         // The next line is commented because we need to be able to send
         // feedback even after the conference has been destroyed.
         // this.callstats = null;
@@ -250,7 +250,7 @@ Statistics.prototype.isCallstatsEnabled = function () {
  */
 Statistics.prototype.sendIceConnectionFailedEvent = function (pc) {
     if(this.callstats)
-        this.callstats.sendIceConnectionFailedEvent(pc, this.callstats);
+        {this.callstats.sendIceConnectionFailedEvent(pc, this.callstats);}
     Statistics.analytics.sendEvent('connection.ice_failed');
 };
 
@@ -261,7 +261,7 @@ Statistics.prototype.sendIceConnectionFailedEvent = function (pc) {
  */
 Statistics.prototype.sendMuteEvent = function (muted, type) {
     if(this.callstats)
-        CallStats.sendMuteEvent(muted, type, this.callstats);
+        {CallStats.sendMuteEvent(muted, type, this.callstats);}
 };
 
 /**
@@ -271,7 +271,7 @@ Statistics.prototype.sendMuteEvent = function (muted, type) {
  */
 Statistics.prototype.sendScreenSharingEvent = function (start) {
     if(this.callstats)
-        CallStats.sendScreenSharingEvent(start, this.callstats);
+        {CallStats.sendScreenSharingEvent(start, this.callstats);}
 };
 
 /**
@@ -280,7 +280,7 @@ Statistics.prototype.sendScreenSharingEvent = function (start) {
  */
 Statistics.prototype.sendDominantSpeakerEvent = function () {
     if(this.callstats)
-        CallStats.sendDominantSpeakerEvent(this.callstats);
+        {CallStats.sendDominantSpeakerEvent(this.callstats);}
 };
 
 /**
@@ -349,7 +349,7 @@ Statistics.sendGetUserMediaFailed = function (e) {
  */
 Statistics.prototype.sendCreateOfferFailed = function (e, pc) {
     if(this.callstats)
-        CallStats.sendCreateOfferFailed(e, pc, this.callstats);
+        {CallStats.sendCreateOfferFailed(e, pc, this.callstats);}
 };
 
 /**
@@ -360,7 +360,7 @@ Statistics.prototype.sendCreateOfferFailed = function (e, pc) {
  */
 Statistics.prototype.sendCreateAnswerFailed = function (e, pc) {
     if(this.callstats)
-        CallStats.sendCreateAnswerFailed(e, pc, this.callstats);
+        {CallStats.sendCreateAnswerFailed(e, pc, this.callstats);}
 };
 
 /**
@@ -371,7 +371,7 @@ Statistics.prototype.sendCreateAnswerFailed = function (e, pc) {
  */
 Statistics.prototype.sendSetLocalDescFailed = function (e, pc) {
     if(this.callstats)
-        CallStats.sendSetLocalDescFailed(e, pc, this.callstats);
+        {CallStats.sendSetLocalDescFailed(e, pc, this.callstats);}
 };
 
 /**
@@ -382,7 +382,7 @@ Statistics.prototype.sendSetLocalDescFailed = function (e, pc) {
  */
 Statistics.prototype.sendSetRemoteDescFailed = function (e, pc) {
     if(this.callstats)
-        CallStats.sendSetRemoteDescFailed(e, pc, this.callstats);
+        {CallStats.sendSetRemoteDescFailed(e, pc, this.callstats);}
 };
 
 /**
@@ -393,7 +393,7 @@ Statistics.prototype.sendSetRemoteDescFailed = function (e, pc) {
  */
 Statistics.prototype.sendAddIceCandidateFailed = function (e, pc) {
     if(this.callstats)
-        CallStats.sendAddIceCandidateFailed(e, pc, this.callstats);
+        {CallStats.sendAddIceCandidateFailed(e, pc, this.callstats);}
 };
 
 /**
@@ -419,7 +419,7 @@ Statistics.sendLog = function (m) {
  */
 Statistics.prototype.sendFeedback = function(overall, detailed) {
     if(this.callstats)
-        this.callstats.sendFeedback(overall, detailed);
+        {this.callstats.sendFeedback(overall, detailed);}
     Statistics.analytics.sendEvent("feedback.rating",
         {value: overall, detailed: detailed});
 };

modules/util/EventEmitterForwarder.js

@@ -8,7 +8,7 @@
 function EventEmitterForwarder (src, dest) {
     if (!src || !dest || typeof src.addListener !== "function" ||
         typeof dest.emit !== "function")
-        throw new Error("Invalid arguments passed to EventEmitterForwarder");
+        {throw new Error("Invalid arguments passed to EventEmitterForwarder");}
     this.src = src;
     this.dest = dest;
 }

modules/util/GlobalOnErrorHandler.js

@@ -23,7 +23,7 @@ function JitsiGlobalErrorHandler(message, source, lineno, colno, error) {
         handler(message, source, lineno, colno, error);
     });
     if (oldOnErrorHandler)
-        oldOnErrorHandler(message, source, lineno, colno, error);
+        {oldOnErrorHandler(message, source, lineno, colno, error);}
 }
 
 // If an old handler exists, also fire its events.
@@ -38,7 +38,7 @@ function JitsiGlobalUnhandledRejection(event) {
         handler(null, null, null, null, event.reason);
     });
     if(oldOnUnhandledRejection)
-        oldOnUnhandledRejection(event);
+        {oldOnUnhandledRejection(event);}
 }
 
 // Setting the custom error handlers.
@@ -61,7 +61,7 @@ var GlobalOnErrorHandler = {
     callErrorHandler: function (error) {
         var errHandler = window.onerror;
         if(!errHandler)
-            return;
+            {return;}
         errHandler(null, null, null, null, error);
     },
     /**
@@ -71,7 +71,7 @@ var GlobalOnErrorHandler = {
     callUnhandledRejectionHandler: function (error) {
         var errHandler = window.onunhandledrejection;
         if(!errHandler)
-            return;
+            {return;}
         errHandler(error);
     }
 };

modules/util/ScriptUtil.js

@@ -39,14 +39,14 @@ var ScriptUtil = {
                 var baseScriptSrc
                     = scriptSrc.substring(0, scriptSrc.lastIndexOf('/') + 1);
                 if (scriptSrc && baseScriptSrc)
-                    src = baseScriptSrc + src;
+                    {src = baseScriptSrc + src;}
             }
         }
 
         if (loadCallback)
-            script.onload = loadCallback;
+            {script.onload = loadCallback;}
         if (errorCallback)
-            script.onerror = errorCallback;
+            {script.onerror = errorCallback;}
 
         script.src = src;
         if (prepend) {

modules/version/ComponentsVersions.js

@@ -75,7 +75,7 @@ ComponentsVersions.prototype.processPresence =
 
     // logs versions to stats
     if (log.length > 0)
-        Statistics.sendLog(JSON.stringify(log));
+        {Statistics.sendLog(JSON.stringify(log));}
 };
 
 /**

modules/xmpp/ChatRoom.js

@@ -38,9 +38,9 @@ var parser = {
             }
             packet.c(node.tagName, node.attributes);
             if(node.value)
-                packet.t(node.value);
+                {packet.t(node.value);}
             if(node.children)
-                this.json2packet(node.children, packet);
+                {this.json2packet(node.children, packet);}
             packet.up();
         }
         // packet.up();
@@ -55,8 +55,8 @@ var parser = {
 function filterNodeFromPresenceJSON(pres, nodeName){
     var res = [];
     for(let i = 0; i < pres.length; i++)
-        if(pres[i].tagName === nodeName)
-            res.push(pres[i]);
+        {if(pres[i].tagName === nodeName)
+            {res.push(pres[i]);}}
 
     return res;
 }
@@ -303,7 +303,7 @@ export default class ChatRoom extends Listenable {
 
                 // set correct initial state of locked
                 if (this.password)
-                    this.locked = true;
+                    {this.locked = true;}
 
                 this.eventEmitter.emit(XMPPEvents.MUC_JOINED);
             }
@@ -343,7 +343,7 @@ export default class ChatRoom extends Listenable {
 
             // store the new display name
             if(member.displayName)
-                memberOfThis.displayName = member.displayName;
+                {memberOfThis.displayName = member.displayName;}
         }
 
         // after we had fired member or room joined events, lets fire events
@@ -376,7 +376,7 @@ export default class ChatRoom extends Listenable {
                 case "call-control":
                     var att = node.attributes;
                     if(!att)
-                        break;
+                        {break;}
                     this.phoneNumber = att.phone || null;
                     this.phonePin = att.pin || null;
                     this.eventEmitter.emit(XMPPEvents.PHONE_NUMBER_CHANGED);
@@ -395,7 +395,7 @@ export default class ChatRoom extends Listenable {
         {
             this.lastJibri = jibri;
             if(this.recording)
-                this.recording.handleJibriPresence(jibri);
+                {this.recording.handleJibriPresence(jibri);}
         }
     }
 
@@ -411,7 +411,7 @@ export default class ChatRoom extends Listenable {
                 this.eventEmitter, this.connection, this.focusMucJid,
                 this.options.jirecon, this.roomjid);
             if(this.lastJibri)
-                this.recording.handleJibriPresence(this.lastJibri);
+                {this.recording.handleJibriPresence(this.lastJibri);}
         }
         logger.info("Ignore focus: " + from + ", real JID: " + mucJid);
     }
@@ -469,7 +469,7 @@ export default class ChatRoom extends Listenable {
         delete this.lastPresences[jid];
 
         if(skipEvents)
-            return;
+            {return;}
 
         this.eventEmitter.emit(XMPPEvents.MUC_MEMBER_LEFT, jid);
 
@@ -520,7 +520,7 @@ export default class ChatRoom extends Listenable {
             // we fire muc_left only if this is not a kick,
             // kick has both statuses 110 and 307.
             if (!isKick)
-                this.eventEmitter.emit(XMPPEvents.MUC_LEFT);
+                {this.eventEmitter.emit(XMPPEvents.MUC_LEFT);}
         }
 
         if (isKick && this.myroomjid === from) {
@@ -694,7 +694,7 @@ export default class ChatRoom extends Listenable {
     setVideoMute (mute, callback) {
         this.sendVideoInfoPresence(mute);
         if(callback)
-            callback(mute);
+            {callback(mute);}
     }
 
     setAudioMute (mute, callback) {
@@ -715,7 +715,7 @@ export default class ChatRoom extends Listenable {
             this.sendPresence();
         }
         if(callback)
-            callback();
+            {callback();}
     }
 
     addVideoInfoToPresence (mute) {
@@ -729,7 +729,7 @@ export default class ChatRoom extends Listenable {
     sendVideoInfoPresence (mute) {
         this.addVideoInfoToPresence(mute);
         if(!this.connection)
-            return;
+            {return;}
         this.sendPresence();
     }
 
@@ -781,7 +781,7 @@ export default class ChatRoom extends Listenable {
      */
     isRecordingSupported () {
         if(this.recording)
-            return this.recording.isSupported();
+            {return this.recording.isSupported();}
         return false;
     }
 
@@ -807,7 +807,7 @@ export default class ChatRoom extends Listenable {
      */
     toggleRecording (options, statusChangeHandler) {
         if(this.recording)
-            return this.recording.toggleRecording(options, statusChangeHandler);
+            {return this.recording.toggleRecording(options, statusChangeHandler);}
 
         return statusChangeHandler("error",
             new Error("The conference is not created yet!"));
@@ -818,7 +818,7 @@ export default class ChatRoom extends Listenable {
      */
     isSIPCallingSupported () {
         if(this.moderator)
-            return this.moderator.isSipGatewayEnabled();
+            {return this.moderator.isSipGatewayEnabled();}
         return false;
     }
 

modules/xmpp/JingleSessionPC.js

@@ -131,10 +131,10 @@ export default class JingleSessionPC extends JingleSession {
                     protocol = protocol.toLowerCase();
                     if (protocol === 'tcp' || protocol === 'ssltcp') {
                         if (this.webrtcIceTcpDisable)
-                            return;
+                            {return;}
                     } else if (protocol == 'udp') {
                         if (this.webrtcIceUdpDisable)
-                            return;
+                            {return;}
                     }
                 }
             }
@@ -148,7 +148,7 @@ export default class JingleSessionPC extends JingleSession {
         // "closed" instead.
         // I suppose at some point this will be moved to onconnectionstatechange
         this.peerconnection.onsignalingstatechange = () => {
-            if (!this.peerconnection) return;
+            if (!this.peerconnection) {return;}
             if (this.peerconnection.signalingState === 'stable') {
                 this.wasstable = true;
             } else if (
@@ -165,7 +165,7 @@ export default class JingleSessionPC extends JingleSession {
          * the value of RTCPeerConnection.iceConnectionState changes.
          */
         this.peerconnection.oniceconnectionstatechange = () => {
-            if (!this.peerconnection) return;
+            if (!this.peerconnection) {return;}
             const now = window.performance.now();
             this.room.connectionTimes["ice.state." +
             this.peerconnection.iceConnectionState] = now;
@@ -190,13 +190,13 @@ export default class JingleSessionPC extends JingleSession {
                     break;
                 case 'disconnected':
                     if (this.closed)
-                        break;
+                        {break;}
                     this.isreconnect = true;
                     // Informs interested parties that the connection has been
                     // interrupted.
                     if (this.wasstable)
-                        this.room.eventEmitter.emit(
-                            XMPPEvents.CONNECTION_INTERRUPTED);
+                        {this.room.eventEmitter.emit(
+                            XMPPEvents.CONNECTION_INTERRUPTED);}
                     break;
                 case 'failed':
                     this.room.eventEmitter.emit(
@@ -228,7 +228,7 @@ export default class JingleSessionPC extends JingleSession {
                 if (this.drip_container.length === 0) {
                     // start 20ms callout
                     setTimeout(() => {
-                        if (this.drip_container.length === 0) return;
+                        if (this.drip_container.length === 0) {return;}
                         this.sendIceCandidates(this.drip_container);
                         this.drip_container = [];
                     }, 20);
@@ -659,13 +659,13 @@ export default class JingleSessionPC extends JingleSession {
                 $(this).find('>parameter').each(function () {
                     lines += 'a=ssrc:' + ssrc + ' ' + $(this).attr('name');
                     if ($(this).attr('value') && $(this).attr('value').length)
-                        lines += ':' + $(this).attr('value');
+                        {lines += ':' + $(this).attr('value');}
                     lines += '\r\n';
                 });
             });
             currentRemoteSdp.media.forEach(function(media, idx) {
                 if (!SDPUtil.find_line(media, 'a=mid:' + name))
-                    return;
+                    {return;}
                 if (!addSsrcInfo[idx]) {
                     addSsrcInfo[idx] = '';
                 }
@@ -1023,7 +1023,7 @@ export default class JingleSessionPC extends JingleSession {
             });
             currentRemoteSdp.media.forEach(function(media, idx) {
                 if (!SDPUtil.find_line(media, 'a=mid:' + name))
-                    return;
+                    {return;}
                 if (!removeSsrcInfo[idx]) {
                     removeSsrcInfo[idx] = '';
                 }
@@ -1318,7 +1318,7 @@ export default class JingleSessionPC extends JingleSession {
                 error.code = errorElSel.attr('code');
                 const errorReasonSel = $(errResponse).find('error :first');
                 if (errorReasonSel.length)
-                    error.reason = errorReasonSel[0].tagName;
+                    {error.reason = errorReasonSel[0].tagName;}
             }
 
             if (!errResponse) {
@@ -1431,7 +1431,7 @@ export default class JingleSessionPC extends JingleSession {
                 const desc = $(jingle.tree()).find(">jingle>content[name=\"" +
                     ssrcObj.mtype + "\"]>description");
                 if (!desc || !desc.length)
-                    return;
+                    {return;}
                 ssrcObj.ssrcs.forEach(function (ssrc) {
                     const sourceNode = desc.find(">source[ssrc=\"" +
                         ssrc + "\"]");
@@ -1496,7 +1496,7 @@ export default class JingleSessionPC extends JingleSession {
         let ssrcs = this.modifiedSSRCs["mute"];
         this.modifiedSSRCs["mute"] = [];
         if (ssrcs && ssrcs.length)
-            ssrcs.forEach(function (ssrcObj) {
+            {ssrcs.forEach(function (ssrcObj) {
                 ssrcObj.ssrcs.forEach(function (ssrc) {
                     const sourceNode
                         = $(jingle.tree()).find(">jingle>content[name=\"" +
@@ -1513,12 +1513,12 @@ export default class JingleSessionPC extends JingleSession {
                             group.ssrcs[0] + "\"])");
                     groupNode.remove();
                 });
-            });
+            });}
 
         ssrcs = this.modifiedSSRCs["remove"];
         this.modifiedSSRCs["remove"] = [];
         if (ssrcs && ssrcs.length)
-            ssrcs.forEach(function (ssrcObj) {
+            {ssrcs.forEach(function (ssrcObj) {
                 const desc
                     = JingleSessionPC.createDescriptionNode(
                         jingle, ssrcObj.mtype);
@@ -1548,7 +1548,7 @@ export default class JingleSessionPC extends JingleSession {
                             "</ssrc-group>");
                     }
                 });
-            });
+            });}
     }
 
     /**

modules/xmpp/SDP.js

@@ -93,7 +93,7 @@ SDP.prototype.containsSSRC = function (ssrc) {
     var result = false;
     Object.keys(medias).forEach(function (mediaindex) {
         if (result)
-            return;
+            {return;}
         if (medias[mediaindex].ssrcs[ssrc]) {
             result = true;
         }
@@ -109,14 +109,14 @@ SDP.prototype.mangle = function () {
         lines.pop(); // remove empty last element
         mline = SDPUtil.parse_mline(lines.shift());
         if (mline.media != 'audio')
-            continue;
+            {continue;}
         newdesc = '';
         mline.fmt.length = 0;
         for (j = 0; j < lines.length; j++) {
             if (lines[j].substr(0, 9) == 'a=rtpmap:') {
                 rtpmap = SDPUtil.parse_rtpmap(lines[j]);
                 if (rtpmap.name == 'CN' || rtpmap.name == 'ISAC')
-                    continue;
+                    {continue;}
                 mline.fmt.push(rtpmap.id);
             }
             newdesc += lines[j] + '\r\n';
@@ -369,7 +369,7 @@ SDP.prototype.transportToJingle = function (mediaindex, elem) {
             });
         // Optional stream count attribute
         if (sctpAttrs.length > 2)
-            elem.attrs({ streams: sctpAttrs[2]});
+            {elem.attrs({ streams: sctpAttrs[2]});}
         elem.up();
     }
     // XEP-0320
@@ -521,14 +521,14 @@ SDP.prototype.jingle2media = function (content) {
 
         var streamCount = sctp.attr('streams');
         if (streamCount)
-            media += ' ' + streamCount + '\r\n';
+            {media += ' ' + streamCount + '\r\n';}
         else
-            media += '\r\n';
+            {media += '\r\n';}
     }
 
     media += 'c=IN IP4 0.0.0.0\r\n';
     if (!sctp.length)
-        media += 'a=rtcp:1 IN IP4 0.0.0.0\r\n';
+        {media += 'a=rtcp:1 IN IP4 0.0.0.0\r\n';}
     tmp = content.find('>transport[xmlns="urn:xmpp:jingle:transports:ice-udp:1"]');
     if (tmp.length) {
         if (tmp.attr('ufrag')) {
@@ -641,7 +641,7 @@ SDP.prototype.jingle2media = function (content) {
             value = SDPUtil.filter_special_chars(value);
             media += 'a=ssrc:' + ssrc + ' ' + name;
             if (value && value.length)
-                media += ':' + value;
+                {media += ':' + value;}
             media += '\r\n';
         });
     });

modules/xmpp/SDPDiffer.js

@@ -16,18 +16,18 @@ SDPDiffer.prototype.getNewMedia = function() {
     function arrayEquals(array) {
         // if the other array is a falsy value, return
         if (!array)
-            return false;
+            {return false;}
 
         // compare lengths - can save a lot of time
         if (this.length != array.length)
-            return false;
+            {return false;}
 
         for (var i = 0, l=this.length; i < l; i++) {
             // Check if we have nested arrays
             if (this[i] instanceof Array && array[i] instanceof Array) {
                 // recurse into the nested arrays
                 if (!this[i].equals(array[i]))
-                    return false;
+                    {return false;}
             }
             else if (this[i] != array[i]) {
                 // Warning - two different object instances will never be

modules/xmpp/SDPUtil.js

@@ -246,7 +246,7 @@ var SDPUtil = {
             needles = [];
         for (var i = 0; i < lines.length; i++) {
             if (lines[i].substring(0, needle.length) == needle)
-                needles.push(lines[i]);
+                {needles.push(lines[i]);}
         }
         if (needles.length || !sessionpart) {
             return needles;
@@ -271,7 +271,7 @@ var SDPUtil = {
             return null;
         }
         if (line.substring(line.length - 2) == '\r\n') // chomp it
-            line = line.substring(0, line.length - 2);
+            {line = line.substring(0, line.length - 2);}
         var candidate = {},
             elems = line.split(' '),
             i;

modules/xmpp/recording.js

@@ -56,13 +56,13 @@ Recording.action = {
 Recording.prototype.handleJibriPresence = function (jibri) {
     var attributes = jibri.attributes;
     if(!attributes)
-        return;
+        {return;}
 
     var newState = attributes.status;
     logger.log("Handle jibri presence : ", newState);
 
     if (newState === this.state)
-        return;
+        {return;}
 
     if (newState === "undefined") {
         this.state = Recording.status.UNAVAILABLE;
@@ -71,9 +71,9 @@ Recording.prototype.handleJibriPresence = function (jibri) {
         if (!this.state
             || this.state === "undefined"
             || this.state === Recording.status.UNAVAILABLE)
-            this.state = Recording.status.AVAILABLE;
+            {this.state = Recording.status.AVAILABLE;}
         else
-            this.state = Recording.status.OFF;
+            {this.state = Recording.status.OFF;}
     }
     else {
         this.state = newState;
@@ -225,11 +225,11 @@ Recording.prototype.toggleRecording = function (options, statusChangeHandler) {
     // If the recorder is currently unavailable we throw an error.
     if (oldState === Recording.status.UNAVAILABLE
         || oldState === Recording.status.FAILED)
-        statusChangeHandler(Recording.status.FAILED,
-                            JitsiRecorderErrors.RECORDER_UNAVAILABLE);
+        {statusChangeHandler(Recording.status.FAILED,
+                            JitsiRecorderErrors.RECORDER_UNAVAILABLE);}
     else if (oldState === Recording.status.BUSY)
-        statusChangeHandler(Recording.status.BUSY,
-                            JitsiRecorderErrors.RECORDER_BUSY);
+        {statusChangeHandler(Recording.status.BUSY,
+                            JitsiRecorderErrors.RECORDER_BUSY);}
 
     // If we're about to turn ON the recording we need either a streamId or
     // an authentication token depending on the recording type. If we don't

modules/xmpp/strophe.emuc.js

@@ -62,7 +62,7 @@ class MucConnectionPlugin extends ConnectionPluginListenable {
 
         const room = this.rooms[Strophe.getBareJidFromJid(from)];
         if(!room)
-            return;
+            {return;}
 
         // Parse status.
         if ($(pres).find('>x[xmlns="http://jabber.org/protocol/muc#user"]' +
@@ -79,7 +79,7 @@ class MucConnectionPlugin extends ConnectionPluginListenable {
         const from = pres.getAttribute('from');
         const room = this.rooms[Strophe.getBareJidFromJid(from)];
         if(!room)
-            return;
+            {return;}
 
         room.onPresenceUnavailable(pres, from);
         return true;
@@ -89,7 +89,7 @@ class MucConnectionPlugin extends ConnectionPluginListenable {
         const from = pres.getAttribute('from');
         const room = this.rooms[Strophe.getBareJidFromJid(from)];
         if(!room)
-            return;
+            {return;}
 
         room.onPresenceError(pres, from);
         return true;
@@ -100,7 +100,7 @@ class MucConnectionPlugin extends ConnectionPluginListenable {
         const from = msg.getAttribute('from');
         const room = this.rooms[Strophe.getBareJidFromJid(from)];
         if(!room)
-            return;
+            {return;}
 
         room.onMessage(msg, from);
         return true;
@@ -110,7 +110,7 @@ class MucConnectionPlugin extends ConnectionPluginListenable {
         const from = iq.getAttribute('from');
         const room = this.rooms[Strophe.getBareJidFromJid(from)];
         if(!room)
-            return;
+            {return;}
 
         room.onMute(iq);
         return true;

modules/xmpp/xmpp.js

@@ -120,13 +120,13 @@ export default class XMPP extends Listenable {
                 pingJid,
                 function (hasPing) {
                     if (hasPing)
-                        this.connection.ping.startInterval(pingJid);
+                        {this.connection.ping.startInterval(pingJid);}
                     else
-                        logger.warn("Ping NOT supported by " + pingJid);
+                        {logger.warn("Ping NOT supported by " + pingJid);}
                 }.bind(this));
 
             if (password)
-                this.authenticatedUser = true;
+                {this.authenticatedUser = true;}
             if (this.connection && this.connection.connected &&
                 Strophe.getResourceFromJid(this.connection.jid)) {
                 // .connected is true while connecting?