[eslint] max-len

.eslintrc.js

@@ -179,6 +179,7 @@ module.exports = {
                 'beforeLineComment': true
             }
         ],
+        'max-len': [ 'error', 80 ],
         'max-lines': 0,
         'max-nested-callbacks': 2,
         'max-statements': 0,

JitsiConference.js

@@ -30,9 +30,11 @@ const logger = getLogger(__filename);
  * Creates a JitsiConference object with the given name and properties.
  * Note: this constructor is not a part of the public API (objects should be
  * created using JitsiConnection.createConference).
- * @param options.config properties / settings related to the conference that will be created.
+ * @param options.config properties / settings related to the conference that
+ * will be created.
  * @param options.name the name of the conference
- * @param options.connection the JitsiConnection object for this JitsiConference.
+ * @param options.connection the JitsiConnection object for this
+ * JitsiConference.
  * @constructor
  */
 function JitsiConference(options) {
@@ -133,7 +135,8 @@ JitsiConference.prototype._init = function(options) {
             callStatsID: this.options.config.callStatsID,
             callStatsSecret: this.options.config.callStatsSecret,
             callStatsConfIDNamespace:
-                this.options.config.callStatsConfIDNamespace || window.location.hostname,
+                this.options.config.callStatsConfIDNamespace
+                    || window.location.hostname,
             callStatsCustomScriptUrl:
                 this.options.config.callStatsCustomScriptUrl,
             roomName: this.options.name
@@ -303,12 +306,13 @@ JitsiConference.prototype.getLocalVideoTrack = function() {
 };
 
 /**
- * Attaches a handler for events(For example - "participant joined".) in the conference. All possible event are defined
- * in JitsiConferenceEvents.
+ * Attaches a handler for events(For example - "participant joined".) in the
+ * conference. All possible event are defined in JitsiConferenceEvents.
  * @param eventId the event ID.
  * @param handler handler for the event.
  *
- * Note: consider adding eventing functionality by extending an EventEmitter impl, instead of rolling ourselves
+ * Note: consider adding eventing functionality by extending an EventEmitter
+ * impl, instead of rolling ourselves
  */
 JitsiConference.prototype.on = function(eventId, handler) {
     if (this.eventEmitter) {
@@ -321,7 +325,8 @@ JitsiConference.prototype.on = function(eventId, handler) {
  * @param eventId the event ID.
  * @param [handler] optional, the specific handler to unbind
  *
- * Note: consider adding eventing functionality by extending an EventEmitter impl, instead of rolling ourselves
+ * Note: consider adding eventing functionality by extending an EventEmitter
+ * impl, instead of rolling ourselves
  */
 JitsiConference.prototype.off = function(eventId, handler) {
     if (this.eventEmitter) {
@@ -561,7 +566,8 @@ JitsiConference.prototype.replaceTrack = function(oldTrack, newTrack) {
                 new JitsiTrackError(JitsiTrackErrors.TRACK_IS_DISPOSED));
         }
 
-        // Set up the ssrcHandler for the new track before we add it at the lower levels
+        // Set up the ssrcHandler for the new track before we add it at the
+        // lower levels
         newTrack.ssrcHandler = function(conference, ssrcMap) {
             const trackSSRCInfo = ssrcMap.get(this.getMSID());
 
@@ -583,7 +589,8 @@ JitsiConference.prototype.replaceTrack = function(oldTrack, newTrack) {
                 this.onLocalTrackRemoved(oldTrack);
             }
             if (newTrack) {
-                // Now handle the addition of the newTrack at the JitsiConference level
+                // Now handle the addition of the newTrack at the
+                // JitsiConference level
                 this._setupNewTrack(newTrack);
             }
 
@@ -675,10 +682,12 @@ JitsiConference.prototype._setupNewTrack = function(newTrack) {
 /**
  * Adds loca WebRTC stream to the conference.
  * @param {MediaStream} stream new stream that will be added.
- * @param {function} callback callback executed after successful stream addition.
- * @param {function(error)} errorCallback callback executed if stream addition fail.
- * @param {object} ssrcInfo object with information about the SSRCs associated with the
- * stream.
+ * @param {function} callback callback executed after successful stream
+ * addition.
+ * @param {function(error)} errorCallback callback executed if stream addition
+ * fail.
+ * @param {object} ssrcInfo object with information about the SSRCs associated
+ * with the stream.
  * @param {boolean} [dontModifySources] if <tt>true</tt> _modifySources won't be
  * called. The option is used for adding stream, before the Jingle call is
  * started. That is before the 'session-accept' is sent.
@@ -709,8 +718,9 @@ JitsiConference.prototype.removeLocalStream
             this.jingleSession.removeStream(
             stream, callback, errorCallback, ssrcInfo);
         } else {
-        // We are done immediately
-            logger.warn('Remove local MediaStream - no JingleSession started yet');
+            // We are done immediately
+            logger.warn(
+                'Remove local MediaStream - no JingleSession started yet');
             callback();
         }
     };
@@ -903,7 +913,10 @@ JitsiConference.prototype.onMemberJoined
 
         participant._role = role;
         this.participants[id] = participant;
-        this.eventEmitter.emit(JitsiConferenceEvents.USER_JOINED, id, participant);
+        this.eventEmitter.emit(
+            JitsiConferenceEvents.USER_JOINED,
+            id,
+            participant);
         this.xmpp.caps.getFeatures(jid).then(features => {
             participant._supportsDTMF = features.has('urn:xmpp:jingle:dtmf:0');
             this.updateDTMFSupport();
@@ -957,7 +970,10 @@ JitsiConference.prototype.onDisplayNameChanged = function(jid, displayName) {
     }
 
     participant._displayName = displayName;
-    this.eventEmitter.emit(JitsiConferenceEvents.DISPLAY_NAME_CHANGED, id, displayName);
+    this.eventEmitter.emit(
+        JitsiConferenceEvents.DISPLAY_NAME_CHANGED,
+        id,
+        displayName);
 };
 
 /**
@@ -1113,7 +1129,9 @@ JitsiConference.prototype.onIncomingCall
          *  consistent
          */
 
-        if (localTrack.isVideoTrack() && localTrack.isMuted() && !RTCBrowserType.isFirefox()) {
+        if (localTrack.isVideoTrack()
+                && localTrack.isMuted()
+                && !RTCBrowserType.isFirefox()) {
             /**
              * Handles issues when the stream is added before the peerconnection
              * is created. The peerconnection is created when second participant
@@ -1244,7 +1262,9 @@ JitsiConference.prototype.updateDTMFSupport = function() {
     }
     if (somebodySupportsDTMF !== this.somebodySupportsDTMF) {
         this.somebodySupportsDTMF = somebodySupportsDTMF;
-        this.eventEmitter.emit(JitsiConferenceEvents.DTMF_SUPPORT_CHANGED, somebodySupportsDTMF);
+        this.eventEmitter.emit(
+            JitsiConferenceEvents.DTMF_SUPPORT_CHANGED,
+            somebodySupportsDTMF);
     }
 };
 
@@ -1262,7 +1282,11 @@ JitsiConference.prototype.isDTMFSupported = function() {
  * @return {string} local user's ID
  */
 JitsiConference.prototype.myUserId = function() {
-    return this.room && this.room.myroomjid ? Strophe.getResourceFromJid(this.room.myroomjid) : null;
+    return (
+        this.room
+            && this.room.myroomjid
+                ? Strophe.getResourceFromJid(this.room.myroomjid)
+                : null);
 };
 
 JitsiConference.prototype.sendTones = function(tones, duration, pause) {

JitsiConnection.js

@@ -5,11 +5,13 @@ import XMPP from './modules/xmpp/xmpp';
 const Statistics = require('./modules/statistics/statistics');
 
 /**
- * Creates new connection object for the Jitsi Meet server side video conferencing service. Provides access to the
- * JitsiConference interface.
- * @param appID identification for the provider of Jitsi Meet video conferencing services.
+ * Creates new connection object for the Jitsi Meet server side video
+ * conferencing service. Provides access to the JitsiConference interface.
+ * @param appID identification for the provider of Jitsi Meet video conferencing
+ * services.
  * @param token the JWT token used to authenticate with the server(optional)
- * @param options Object with properties / settings related to connection with the server.
+ * @param options Object with properties / settings related to connection with
+ * the server.
  * @constructor
  */
 function JitsiConnection(appID, token, options) {

JitsiMeetJS.js

@@ -1,7 +1,8 @@
 /* global __filename */
 
 import AuthUtil from './modules/util/AuthUtil';
-import * as ConnectionQualityEvents from './service/connectivity/ConnectionQualityEvents';
+import * as ConnectionQualityEvents
+    from './service/connectivity/ConnectionQualityEvents';
 import GlobalOnErrorHandler from './modules/util/GlobalOnErrorHandler';
 import * as JitsiConferenceErrors from './JitsiConferenceErrors';
 import * as JitsiConferenceEvents from './JitsiConferenceEvents';
@@ -183,13 +184,15 @@ const LibJitsiMeet = {
 
     /**
      * Creates the media tracks and returns them trough the callback.
-     * @param options Object with properties / settings specifying the tracks which should be created.
-     * should be created or some additional configurations about resolution for example.
+     * @param options Object with properties / settings specifying the tracks
+     * which should be created. should be created or some additional
+     * configurations about resolution for example.
      * @param {Array} options.devices the devices that will be requested
      * @param {string} options.resolution resolution constraints
-     * @param {bool} options.dontCreateJitsiTrack if <tt>true</tt> objects with the following structure {stream: the Media Stream,
-     * type: "audio" or "video", videoType: "camera" or "desktop"}
-     * will be returned trough the Promise, otherwise JitsiTrack objects will be returned.
+     * @param {bool} options.dontCreateJitsiTrack if <tt>true</tt> objects with
+     * the following structure {stream: the Media Stream, type: "audio" or
+     * "video", videoType: "camera" or "desktop"} will be returned trough the
+     * Promise, otherwise JitsiTrack objects will be returned.
      * @param {string} options.cameraDeviceId
      * @param {string} options.micDeviceId
      * @param {object} options.desktopSharingExtensionExternalInstallation -

doc/example/example.js

@@ -6,7 +6,9 @@ const options = {
         muc: 'conference.jitsi-meet.example.com' // FIXME: use XEP-0030
     },
     bosh: '//jitsi-meet.example.com/http-bind', // FIXME: use xep-0156 for that
-    clientNode: 'http://jitsi.org/jitsimeet' // The name of client node advertised in XEP-0115 'c' stanza
+
+    // The name of client node advertised in XEP-0115 'c' stanza
+    clientNode: 'http://jitsi.org/jitsimeet'
 };
 
 const confOptions = {
@@ -45,7 +47,8 @@ function onLocalTracks(tracks) {
             $('body').append(`<video autoplay='1' id='localVideo${i}' />`);
             localTracks[i].attach($(`#localVideo${i}`)[0]);
         } else {
-            $('body').append(`<audio autoplay='1' muted='true' id='localAudio${i}' />`);
+            $('body').append(
+                `<audio autoplay='1' muted='true' id='localAudio${i}' />`);
             localTracks[i].attach($(`#localAudio${i}`)[0]);
         }
         if (isJoined) {
@@ -85,9 +88,11 @@ function onRemoteTrack(track) {
     const id = participant + track.getType() + idx;
 
     if (track.getType() == 'video') {
-        $('body').append(`<video autoplay='1' id='${participant}video${idx}' />`);
+        $('body').append(
+            `<video autoplay='1' id='${participant}video${idx}' />`);
     } else {
-        $('body').append(`<audio autoplay='1' id='${participant}audio${idx}' />`);
+        $('body').append(
+            `<audio autoplay='1' id='${participant}audio${idx}' />`);
     }
     track.attach($(`#${id}`)[0]);
 }
@@ -124,7 +129,9 @@ function onConnectionSuccess() {
     room.on(JitsiMeetJS.events.conference.TRACK_REMOVED, track => {
         console.log(`track removed!!!${track}`);
     });
-    room.on(JitsiMeetJS.events.conference.CONFERENCE_JOINED, onConferenceJoined);
+    room.on(
+        JitsiMeetJS.events.conference.CONFERENCE_JOINED,
+        onConferenceJoined);
     room.on(JitsiMeetJS.events.conference.USER_JOINED, id => {
         console.log('user join');
         remoteTracks[id] = [];
@@ -133,9 +140,9 @@ function onConnectionSuccess() {
     room.on(JitsiMeetJS.events.conference.TRACK_MUTE_CHANGED, track => {
         console.log(`${track.getType()} - ${track.isMuted()}`);
     });
-    room.on(JitsiMeetJS.events.conference.DISPLAY_NAME_CHANGED, (userID, displayName) => {
-        console.log(`${userID} - ${displayName}`);
-    });
+    room.on(
+        JitsiMeetJS.events.conference.DISPLAY_NAME_CHANGED,
+        (userID, displayName) => console.log(`${userID} - ${displayName}`));
     room.on(JitsiMeetJS.events.conference.TRACK_AUDIO_LEVEL_CHANGED,
       (userID, audioLevel) => {
           console.log(`${userID} - ${audioLevel}`);
@@ -172,9 +179,15 @@ function onDeviceListChanged(devices) {
  */
 function disconnect() {
     console.log('disconnect!');
-    connection.removeEventListener(JitsiMeetJS.events.connection.CONNECTION_ESTABLISHED, onConnectionSuccess);
-    connection.removeEventListener(JitsiMeetJS.events.connection.CONNECTION_FAILED, onConnectionFailed);
-    connection.removeEventListener(JitsiMeetJS.events.connection.CONNECTION_DISCONNECTED, disconnect);
+    connection.removeEventListener(
+        JitsiMeetJS.events.connection.CONNECTION_ESTABLISHED,
+        onConnectionSuccess);
+    connection.removeEventListener(
+        JitsiMeetJS.events.connection.CONNECTION_FAILED,
+        onConnectionFailed);
+    connection.removeEventListener(
+        JitsiMeetJS.events.connection.CONNECTION_DISCONNECTED,
+        disconnect);
 }
 
 function unload() {
@@ -193,17 +206,16 @@ function switchVideo() { // eslint-disable-line no-unused-vars
         localTracks[1].dispose();
         localTracks.pop();
     }
-    JitsiMeetJS.createLocalTracks({ devices: isVideo ? [ 'video' ] : [ 'desktop' ] })
+    JitsiMeetJS.createLocalTracks(
+            { devices: isVideo ? [ 'video' ] : [ 'desktop' ] })
         .then(tracks => {
             localTracks.push(tracks[0]);
-            localTracks[1].addEventListener(JitsiMeetJS.events.track.TRACK_MUTE_CHANGED,
-                () => {
-                    console.log('local track muted');
-                });
-            localTracks[1].addEventListener(JitsiMeetJS.events.track.LOCAL_TRACK_STOPPED,
-                () => {
-                    console.log('local track stoped');
-                });
+            localTracks[1].addEventListener(
+                JitsiMeetJS.events.track.TRACK_MUTE_CHANGED,
+                () => console.log('local track muted'));
+            localTracks[1].addEventListener(
+                JitsiMeetJS.events.track.LOCAL_TRACK_STOPPED,
+                () => console.log('local track stoped'));
             localTracks[1].attach($('#localVideo1')[0]);
             room.addTrack(localTracks[1]);
         })
@@ -281,12 +293,16 @@ JitsiMeetJS.init(initOptions)
 
 if (JitsiMeetJS.mediaDevices.isDeviceChangeAvailable('output')) {
     JitsiMeetJS.mediaDevices.enumerateDevices(devices => {
-        const audioOutputDevices = devices.filter(d => d.kind === 'audiooutput');
+        const audioOutputDevices
+            = devices.filter(d => d.kind === 'audiooutput');
 
         if (audioOutputDevices.length > 1) {
             $('#audioOutputSelect').html(
-                audioOutputDevices.map(d => `<option value="${d.deviceId}">${d.label}</option>`).join('\n')
-            );
+                audioOutputDevices
+                    .map(
+                        d =>
+                            `<option value="${d.deviceId}">${d.label}</option>`)
+                    .join('\n'));
 
             $('#audioOutputSelectWrapper').show();
         }

modules/RTC/DataChannels.js

@@ -16,10 +16,10 @@ function DataChannels(peerConnection, emitter) {
     this._dataChannels = [];
 
     // Sample code for opening new data channel from Jitsi Meet to the bridge.
-    // Although it's not a requirement to open separate channels from both bridge
-    // and peer as single channel can be used for sending and receiving data.
-    // So either channel opened by the bridge or the one opened here is enough
-    // for communication with the bridge.
+    // Although it's not a requirement to open separate channels from both
+    // bridge and peer as single channel can be used for sending and receiving
+    // data. So either channel opened by the bridge or the one opened here is
+    // enough for communication with the bridge.
     /* var dataChannelOptions =
      {
      reliable: true
@@ -141,7 +141,8 @@ DataChannels.prototype.onDataChannel = function(event) {
                 self.eventEmitter.emit(
                     RTCEvents.ENDPOINT_MESSAGE_RECEIVED, obj.from,
                     obj.msgPayload);
-            } else if (colibriClass === 'EndpointConnectivityStatusChangeEvent') {
+            } else if (colibriClass
+                    === 'EndpointConnectivityStatusChangeEvent') {
                 const endpoint = obj.endpoint;
                 const isActive = obj.active === 'true';
 

modules/RTC/JitsiLocalTrack.js

@@ -328,7 +328,8 @@ JitsiLocalTrack.prototype._setMute = function(mute) {
         promise = RTCUtils.obtainAudioAndVideoPermissions(streamOptions)
             .then(streamsInfo => {
                 const mediaType = self.getType();
-                const streamInfo = streamsInfo.find(info => info.mediaType === mediaType);
+                const streamInfo
+                    = streamsInfo.find(info => info.mediaType === mediaType);
 
                 if (streamInfo) {
                     self._setStream(streamInfo.stream);
@@ -347,7 +348,9 @@ JitsiLocalTrack.prototype._setMute = function(mute) {
                         JitsiTrackErrors.TRACK_NO_STREAM_FOUND);
                 }
 
-                self.containers = self.containers.map(cont => RTCUtils.attachMediaStream(cont, self.stream));
+                self.containers
+                    = self.containers.map(
+                        cont => RTCUtils.attachMediaStream(cont, self.stream));
 
                 return self._addStreamToConferenceAsUnmute();
             });

modules/RTC/JitsiTrack.js

@@ -67,8 +67,14 @@ function addMediaStreamInactiveHandler(mediaStream, handler) {
  * @param videoType the VideoType for this track if any
  * @param ssrc the SSRC of this track if known
  */
-function JitsiTrack(conference, stream, track, streamInactiveHandler, trackMediaType,
-                    videoType, ssrc) {
+function JitsiTrack(
+        conference,
+        stream,
+        track,
+        streamInactiveHandler,
+        trackMediaType,
+        videoType,
+        ssrc) {
     /**
      * Array with the HTML elements that are displaying the streams.
      * @type {Array}
@@ -418,18 +424,25 @@ JitsiTrack.prototype.setAudioOutput = function(audioOutputDeviceId) {
         return Promise.resolve();
     }
 
-    return Promise.all(this.containers.map(element => element.setSinkId(audioOutputDeviceId)
-            .catch(error => {
-                logger.warn(
-                    'Failed to change audio output device on element. Default'
-                    + ' or previously set audio output device will be used.',
-                    element, error);
-                throw error;
-            })))
-    .then(() => {
-        self.eventEmitter.emit(JitsiTrackEvents.TRACK_AUDIO_OUTPUT_CHANGED,
-            audioOutputDeviceId);
-    });
+    return (
+        Promise.all(
+                this.containers.map(
+                    element =>
+                        element.setSinkId(audioOutputDeviceId)
+                            .catch(error => {
+                                logger.warn(
+                                    'Failed to change audio output device on'
+                                        + ' element. Default or previously set'
+                                        + ' audio output device will be used.',
+                                    element,
+                                    error);
+                                throw error;
+                            })))
+            .then(() => {
+                self.eventEmitter.emit(
+                    JitsiTrackEvents.TRACK_AUDIO_OUTPUT_CHANGED,
+                    audioOutputDeviceId);
+            }));
 };
 
 module.exports = JitsiTrack;

modules/RTC/RTC.js

@@ -66,12 +66,12 @@ export default class RTC extends Listenable {
 
         // A flag whether we had received that the data channel had opened
         // we can get this flag out of sync if for some reason data channel got
-        // closed from server, a desired behaviour so we can see errors when this
-        // happen
+        // closed from server, a desired behaviour so we can see errors when
+        // this happen
         this.dataChannelsOpen = false;
 
-        // Switch audio output device on all remote audio tracks. Local audio tracks
-        // handle this event by themselves.
+        // Switch audio output device on all remote audio tracks. Local audio
+        // tracks handle this event by themselves.
         if (RTCUtils.isDeviceChangeAvailable('output')) {
             RTCUtils.addListener(RTCEvents.AUDIO_OUTPUT_DEVICE_CHANGED,
                 deviceId => {
@@ -90,11 +90,10 @@ export default class RTC extends Listenable {
      * @param {Object} [options] optional parameters
      * @param {Array} options.devices the devices that will be requested
      * @param {string} options.resolution resolution constraints
-     * @param {bool} options.dontCreateJitsiTrack if <tt>true</tt> objects with the
-     * following structure {stream: the Media Stream,
-     * type: "audio" or "video", videoType: "camera" or "desktop"}
-     * will be returned trough the Promise, otherwise JitsiTrack objects will be
-     * returned.
+     * @param {bool} options.dontCreateJitsiTrack if <tt>true</tt> objects with
+     * the following structure {stream: the Media Stream, type: "audio" or
+     * "video", videoType: "camera" or "desktop"} will be returned trough the
+     * Promise, otherwise JitsiTrack objects will be returned.
      * @param {string} options.cameraDeviceId
      * @param {string} options.micDeviceId
      * @returns {*} Promise object that will receive the new JitsiTracks
@@ -186,7 +185,8 @@ export default class RTC extends Listenable {
      * order to always receive video for this participant (even when last n is
      * enabled).
      * @param id {string} the user id
-     * @throws NetworkError or InvalidStateError or Error if the operation fails.
+     * @throws NetworkError or InvalidStateError or Error if the operation
+     * fails.
      */
     pinEndpoint(id) {
         if (this.dataChannels) {
@@ -364,8 +364,8 @@ export default class RTC extends Listenable {
     }
 
     /**
-     * Gets JitsiRemoteTrack for AUDIO MediaType associated with given MUC nickname
-     * (resource part of the JID).
+     * Gets JitsiRemoteTrack for AUDIO MediaType associated with given MUC
+     * nickname (resource part of the JID).
      * @param resource the resource part of the MUC JID
      * @returns {JitsiRemoteTrack|null}
      */
@@ -374,8 +374,8 @@ export default class RTC extends Listenable {
     }
 
     /**
-     * Gets JitsiRemoteTrack for VIDEO MediaType associated with given MUC nickname
-     * (resource part of the JID).
+     * Gets JitsiRemoteTrack for VIDEO MediaType associated with given MUC
+     * nickname (resource part of the JID).
      * @param resource the resource part of the MUC JID
      * @returns {JitsiRemoteTrack|null}
      */
@@ -699,7 +699,7 @@ export default class RTC extends Listenable {
      * matches given SSRC or <tt>undefined</tt> if no such track was found.
      */
     getRemoteTrackBySSRC(ssrc) {
-        return this.getRemoteTracks().find(remoteTrack => ssrc == remoteTrack.getSSRC());
+        return this.getRemoteTracks().find(t => ssrc == t.getSSRC());
     }
 
     /**

modules/RTC/RTCBrowserType.js

@@ -114,7 +114,8 @@ const RTCBrowserType = {
      */
     isTemasysPluginUsed() {
         // Temasys do not support Microsoft Edge:
-        // http://support.temasys.com.sg/support/solutions/articles/5000654345-can-the-temasys-webrtc-plugin-be-used-with-microsoft-edge-
+        // http://support.temasys.com.sg/support/solutions/articles/
+        // 5000654345-can-the-temasys-webrtc-plugin-be-used-with-microsoft-edge-
         if (RTCBrowserType.isIExplorer()
                 && RTCBrowserType.getIExplorerVersion() < 12) {
             return true;

modules/RTC/RTCUtils.js

@@ -193,7 +193,8 @@ function getConstraints(um, options) {
             // for some cameras it might be necessary to request 30fps
             // so they choose 30fps mjpg over 10fps yuy2
             if (options.minFps || options.fps) {
-                options.minFps = options.minFps || options.fps; // Fall back to options.fps for backwards compatibility
+                // Fall back to options.fps for backwards compatibility
+                options.minFps = options.minFps || options.fps;
                 constraints.video.mandatory.minFrameRate = options.minFps;
             }
             if (options.maxFps) {
@@ -303,9 +304,10 @@ function getConstraints(um, options) {
         constraints.video.optional.push({ bandwidth: options.bandwidth });
     }
 
-    // we turn audio for both audio and video tracks, the fake audio & video seems to work
-    // only when enabled in one getUserMedia call, we cannot get fake audio separate by fake video
-    // this later can be a problem with some of the tests
+    // we turn audio for both audio and video tracks, the fake audio & video
+    // seems to work only when enabled in one getUserMedia call, we cannot get
+    // fake audio separate by fake video this later can be a problem with some
+    // of the tests
     if (RTCBrowserType.isFirefox() && options.firefox_fake_device) {
         // seems to be fixed now, removing this experimental fix, as having
         // multiple audio tracks brake the tests
@@ -401,7 +403,9 @@ function pollForAvailableMediaDevices() {
  */
 function onMediaDevicesListChanged(devicesReceived) {
     currentlyAvailableMediaDevices = devicesReceived.slice(0);
-    logger.info('list of media devices has changed:', currentlyAvailableMediaDevices);
+    logger.info(
+        'list of media devices has changed:',
+        currentlyAvailableMediaDevices);
 
     const videoInputDevices
         = currentlyAvailableMediaDevices.filter(d => d.kind === 'videoinput');
@@ -413,12 +417,14 @@ function onMediaDevicesListChanged(devicesReceived) {
         = audioInputDevices.filter(d => d.label === '');
 
     if (videoInputDevices.length
-        && videoInputDevices.length === videoInputDevicesWithEmptyLabels.length) {
+            && videoInputDevices.length
+                === videoInputDevicesWithEmptyLabels.length) {
         devices.video = false;
     }
 
     if (audioInputDevices.length
-        && audioInputDevices.length === audioInputDevicesWithEmptyLabels.length) {
+            && audioInputDevices.length
+                === audioInputDevicesWithEmptyLabels.length) {
         devices.audio = false;
     }
 
@@ -682,7 +688,8 @@ function wrapAttachMediaStream(origAttachMediaStream) {
                 && audioOutputChanged) {
             element.setSinkId(rtcUtils.getAudioOutputDevice())
                 .catch(function(ex) {
-                    const err = new JitsiTrackError(ex, null, [ 'audiooutput' ]);
+                    const err
+                        = new JitsiTrackError(ex, null, [ 'audiooutput' ]);
 
                     GlobalOnErrorHandler.callUnhandledRejectionHandler(
                         { promise: this,
@@ -780,28 +787,33 @@ class RTCUtils extends Listenable {
                     return;
                 }
                 this.peerconnection = mozRTCPeerConnection;
-                this.getUserMedia = wrapGetUserMedia(navigator.mozGetUserMedia.bind(navigator));
-                this.enumerateDevices = wrapEnumerateDevices(
-                    navigator.mediaDevices.enumerateDevices.bind(navigator.mediaDevices)
-                );
+                this.getUserMedia
+                    = wrapGetUserMedia(
+                        navigator.mozGetUserMedia.bind(navigator));
+                this.enumerateDevices
+                    = wrapEnumerateDevices(
+                        navigator.mediaDevices.enumerateDevices.bind(
+                            navigator.mediaDevices));
                 this.pc_constraints = {};
-                this.attachMediaStream = wrapAttachMediaStream((element, stream) => {
-                    //  srcObject is being standardized and FF will eventually
-                    //  support that unprefixed. FF also supports the
-                    //  "element.src = URL.createObjectURL(...)" combo, but that
-                    //  will be deprecated in favour of srcObject.
-                    //
-                    // https://groups.google.com/forum/#!topic/mozilla.dev.media/pKOiioXonJg
-                    // https://github.com/webrtc/samples/issues/302
-                    if (element) {
-                        defaultSetVideoSrc(element, stream);
-                        if (stream) {
-                            element.play();
+                this.attachMediaStream
+                    = wrapAttachMediaStream((element, stream) => {
+                        // srcObject is being standardized and FF will
+                        // eventually support that unprefixed. FF also supports
+                        // the "element.src = URL.createObjectURL(...)" combo,
+                        // but that will be deprecated in favour of srcObject.
+                        //
+                        // https://groups.google.com/forum/#!topic/
+                        // mozilla.dev.media/pKOiioXonJg
+                        // https://github.com/webrtc/samples/issues/302
+                        if (element) {
+                            defaultSetVideoSrc(element, stream);
+                            if (stream) {
+                                element.play();
+                            }
                         }
-                    }
 
-                    return element;
-                });
+                        return element;
+                    });
                 this.getStreamID = function(stream) {
                     let id = stream.id;
 
@@ -816,8 +828,13 @@ class RTCUtils extends Listenable {
 
                     return SDPUtil.filter_special_chars(id);
                 };
-                RTCSessionDescription = mozRTCSessionDescription; // eslint-disable-line
-                RTCIceCandidate = mozRTCIceCandidate;             // eslint-disable-line
+
+                /* eslint-disable no-global-assign, no-native-reassign */
+
+                RTCSessionDescription = mozRTCSessionDescription;
+                RTCIceCandidate = mozRTCIceCandidate;
+
+                /* eslint-enable no-global-assign, no-native-reassign */
             } else if (RTCBrowserType.isChrome()
                     || RTCBrowserType.isOpera()
                     || RTCBrowserType.isNWJS()
@@ -825,22 +842,26 @@ class RTCUtils extends Listenable {
                     || RTCBrowserType.isReactNative()) {
 
                 this.peerconnection = webkitRTCPeerConnection;
-                const getUserMedia = navigator.webkitGetUserMedia.bind(navigator);
+                const getUserMedia
+                    = navigator.webkitGetUserMedia.bind(navigator);
 
                 if (navigator.mediaDevices) {
                     this.getUserMedia = wrapGetUserMedia(getUserMedia);
-                    this.enumerateDevices = wrapEnumerateDevices(
-                        navigator.mediaDevices.enumerateDevices.bind(navigator.mediaDevices)
-                    );
+                    this.enumerateDevices
+                        = wrapEnumerateDevices(
+                            navigator.mediaDevices.enumerateDevices.bind(
+                                navigator.mediaDevices));
                 } else {
                     this.getUserMedia = getUserMedia;
-                    this.enumerateDevices = enumerateDevicesThroughMediaStreamTrack;
+                    this.enumerateDevices
+                      = enumerateDevicesThroughMediaStreamTrack;
                 }
-                this.attachMediaStream = wrapAttachMediaStream((element, stream) => {
-                    defaultSetVideoSrc(element, stream);
+                this.attachMediaStream
+                    = wrapAttachMediaStream((element, stream) => {
+                        defaultSetVideoSrc(element, stream);
 
-                    return element;
-                });
+                        return element;
+                    });
                 this.getStreamID = function(stream) {
                     // A. MediaStreams from FF endpoints have the characters '{'
                     // and '}' that make jQuery choke.
@@ -889,33 +910,37 @@ class RTCUtils extends Listenable {
                 const webRTCReadyCb = () => {
                     this.peerconnection = RTCPeerConnection;
                     this.getUserMedia = window.getUserMedia;
-                    this.enumerateDevices = enumerateDevicesThroughMediaStreamTrack;
-                    this.attachMediaStream = wrapAttachMediaStream((element, stream) => {
-                        if (stream) {
-                            if (stream.id === 'dummyAudio'
-                                    || stream.id === 'dummyVideo') {
-                                return;
-                            }
+                    this.enumerateDevices
+                        = enumerateDevicesThroughMediaStreamTrack;
+                    this.attachMediaStream
+                        = wrapAttachMediaStream((element, stream) => {
+                            if (stream) {
+                                if (stream.id === 'dummyAudio'
+                                        || stream.id === 'dummyVideo') {
+                                    return;
+                                }
 
-                            // The container must be visible in order to play or
-                            // attach the stream when Temasys plugin is in use
-                            const containerSel = $(element);
+                                // The container must be visible in order to
+                                // play or attach the stream when Temasys plugin
+                                // is in use
+                                const containerSel = $(element);
 
-                            if (RTCBrowserType.isTemasysPluginUsed()
-                                    && !containerSel.is(':visible')) {
-                                containerSel.show();
-                            }
-                            const video = stream.getVideoTracks().length > 0;
+                                if (RTCBrowserType.isTemasysPluginUsed()
+                                        && !containerSel.is(':visible')) {
+                                    containerSel.show();
+                                }
+                                const video
+                                    = stream.getVideoTracks().length > 0;
 
-                            if (video && !$(element).is(':visible')) {
-                                throw new Error(
-                                    'video element must be visible to attach'
-                                        + ' video stream');
+                                if (video && !$(element).is(':visible')) {
+                                    throw new Error(
+                                        'video element must be visible to'
+                                            + ' attach video stream');
+                                }
                             }
-                        }
 
-                        return attachMediaStream(element, stream);
-                    });
+                            return attachMediaStream(element, stream);
+                        });
                     this.getStreamID
                         = stream => SDPUtil.filter_special_chars(stream.label);
 
@@ -975,7 +1000,11 @@ class RTCUtils extends Listenable {
     * @param {string} options.cameraDeviceId
     * @param {string} options.micDeviceId
     **/
-    getUserMediaWithConstraints(um, success_callback, failure_callback, options) {
+    getUserMediaWithConstraints(
+            um,
+            success_callback,
+            failure_callback,
+            options) {
         options = options || {};
         const constraints = getConstraints(um, options);
 
@@ -1013,9 +1042,10 @@ class RTCUtils extends Listenable {
      * @param {Object} [options] optional parameters
      * @param {Array} options.devices the devices that will be requested
      * @param {string} options.resolution resolution constraints
-     * @param {bool} options.dontCreateJitsiTrack if <tt>true</tt> objects with the following structure {stream: the Media Stream,
-     * type: "audio" or "video", videoType: "camera" or "desktop"}
-     * will be returned trough the Promise, otherwise JitsiTrack objects will be returned.
+     * @param {bool} options.dontCreateJitsiTrack if <tt>true</tt> objects with
+     * the following structure {stream: the Media Stream, type: "audio" or
+     * "video", videoType: "camera" or "desktop"} will be returned trough the
+     * Promise, otherwise JitsiTrack objects will be returned.
      * @param {string} options.cameraDeviceId
      * @param {string} options.micDeviceId
      * @returns {*} Promise object that will receive the new JitsiTracks
@@ -1062,14 +1092,14 @@ class RTCUtils extends Listenable {
                         dsOptions);
                 }
 
-                // With FF/IE we can't split the stream into audio and video because FF
-                // doesn't support media stream constructors. So, we need to get the
-                // audio stream separately from the video stream using two distinct GUM
-                // calls. Not very user friendly :-( but we don't have many other
-                // options neither.
+                // With FF/IE we can't split the stream into audio and video
+                // because FF doesn't support media stream constructors. So, we
+                // need to get the audio stream separately from the video stream
+                // using two distinct GUM calls. Not very user friendly :-( but
+                // we don't have many other options neither.
                 //
-                // Note that we pack those 2 streams in a single object and pass it to
-                // the successCallback method.
+                // Note that we pack those 2 streams in a single object and pass
+                // it to the successCallback method.
                 obtainDevices({
                     devices: options.devices,
                     streams: [],
@@ -1081,7 +1111,9 @@ class RTCUtils extends Listenable {
                 const hasDesktop = options.devices.indexOf('desktop') > -1;
 
                 if (hasDesktop) {
-                    options.devices.splice(options.devices.indexOf('desktop'), 1);
+                    options.devices.splice(
+                        options.devices.indexOf('desktop'),
+                        1);
                 }
                 options.resolution = options.resolution || '360';
                 if (options.devices.length) {
@@ -1098,7 +1130,8 @@ class RTCUtils extends Listenable {
                                 = stream.getVideoTracks().length > 0;
 
                             if ((audioDeviceRequested && !audioTracksReceived)
-                                || (videoDeviceRequested && !videoTracksReceived)) {
+                                    || (videoDeviceRequested
+                                        && !videoTracksReceived)) {
                                 self.stopMediaStream(stream);
 
                                 // We are getting here in case if we requested
@@ -1108,11 +1141,13 @@ class RTCUtils extends Listenable {
                                 // this happened, so reject with general error.
                                 const devices = [];
 
-                                if (audioDeviceRequested && !audioTracksReceived) {
+                                if (audioDeviceRequested
+                                        && !audioTracksReceived) {
                                     devices.push('audio');
                                 }
 
-                                if (videoDeviceRequested && !videoTracksReceived) {
+                                if (videoDeviceRequested
+                                        && !videoTracksReceived) {
                                     devices.push('video');
                                 }
 
@@ -1135,7 +1170,8 @@ class RTCUtils extends Listenable {
                                         reject(new JitsiTrackError(
                                             { name: 'UnknownError' },
                                             getConstraints(
-                                                options.devices, options),
+                                                options.devices,
+                                                options),
                                             devices)
                                         );
                                     },

modules/RTC/TraceablePeerConnection.js

@@ -228,7 +228,8 @@ const dumpSDP = function(description) {
 TraceablePeerConnection.prototype._remoteStreamAdded = function(stream) {
     if (!RTC.isUserStream(stream)) {
         logger.info(
-            'Ignored remote \'stream added\' event for non-user stream', stream);
+            'Ignored remote \'stream added\' event for non-user stream',
+            stream);
 
         return;
     }
@@ -545,7 +546,8 @@ const normalizePlanB = function(desc) {
     const session = transform.parse(desc.sdp);
 
     if (typeof session !== 'undefined'
-        && typeof session.media !== 'undefined' && Array.isArray(session.media)) {
+            && typeof session.media !== 'undefined'
+            && Array.isArray(session.media)) {
         session.media.forEach(mLine => {
 
             // Chrome appears to be picky about the order in which a=ssrc lines
@@ -696,9 +698,11 @@ TraceablePeerConnection.prototype.createDataChannel = function(label, opts) {
 
 TraceablePeerConnection.prototype.setLocalDescription
         = function(description, successCallback, failureCallback) {
-            this.trace('setLocalDescription::preTransform', dumpSDP(description));
+            this.trace(
+                'setLocalDescription::preTransform',
+                dumpSDP(description));
 
-    // if we're running on FF, transform to Plan A first.
+            // if we're running on FF, transform to Plan A first.
             if (RTCBrowserType.usesUnifiedPlan()) {
                 description = this.interop.toUnifiedPlan(description);
                 this.trace('setLocalDescription::postTransform (Plan A)',
@@ -724,26 +728,31 @@ TraceablePeerConnection.prototype.setLocalDescription
 
 TraceablePeerConnection.prototype.setRemoteDescription
         = function(description, successCallback, failureCallback) {
-            this.trace('setRemoteDescription::preTransform', dumpSDP(description));
+            this.trace(
+                'setRemoteDescription::preTransform',
+                dumpSDP(description));
 
-    // TODO the focus should squeze or explode the remote simulcast
+            // TODO the focus should squeze or explode the remote simulcast
             description = this.simulcast.mungeRemoteDescription(description);
             this.trace(
-        'setRemoteDescription::postTransform (simulcast)',
-        dumpSDP(description));
+                'setRemoteDescription::postTransform (simulcast)',
+                dumpSDP(description));
 
             if (this.options.preferH264) {
                 const parsedSdp = transform.parse(description.sdp);
-                const videoMLine = parsedSdp.media.find(m => m.type === 'video');
+                const videoMLine
+                    = parsedSdp.media.find(m => m.type === 'video');
 
                 SDPUtil.preferVideoCodec(videoMLine, 'h264');
                 description.sdp = transform.write(parsedSdp);
             }
 
-    // if we're running on FF, transform to Plan A first.
+            // if we're running on FF, transform to Plan A first.
             if (RTCBrowserType.usesUnifiedPlan()) {
                 description.sdp = this.rtxModifier.stripRtx(description.sdp);
-                this.trace('setRemoteDescription::postTransform (stripRtx)', dumpSDP(description));
+                this.trace(
+                    'setRemoteDescription::postTransform (stripRtx)',
+                    dumpSDP(description));
                 description = this.interop.toUnifiedPlan(description);
                 this.trace(
             'setRemoteDescription::postTransform (Plan A)',
@@ -900,7 +909,8 @@ TraceablePeerConnection.prototype.createAnswer
                         = this.sdpConsistency.makeVideoPrimarySsrcsConsistent(
                             answer.sdp);
                     this.trace(
-                        'createAnswerOnSuccess::postTransform (make primary video ssrcs consistent)',
+                        'createAnswerOnSuccess::postTransform (make primary'
+                          + ' video ssrcs consistent)',
                         dumpSDP(answer));
                 }
 

modules/statistics/RTPStatsCollector.js

@@ -66,7 +66,8 @@ KEYS_BY_BROWSER_TYPE[RTCBrowserType.RTC_BROWSER_REACT_NATIVE]
  * @returns {number} packet loss percent
  */
 function calculatePacketLoss(lostPackets, totalPackets) {
-    if (!totalPackets || totalPackets <= 0 || !lostPackets || lostPackets <= 0) {
+    if (!totalPackets || totalPackets <= 0
+            || !lostPackets || lostPackets <= 0) {
         return 0;
     }
 
@@ -517,7 +518,8 @@ StatsCollector.prototype.processStatsReport = function() {
 
         const bytesReceivedNow = getNonNegativeStat(now, 'bytesReceived');
         const bytesReceivedBefore = getNonNegativeStat(before, 'bytesReceived');
-        const bytesReceived = Math.max(0, bytesReceivedNow - bytesReceivedBefore);
+        const bytesReceived
+            = Math.max(0, bytesReceivedNow - bytesReceivedBefore);
 
         let bytesSent = 0;
 

modules/statistics/statistics.js

@@ -168,7 +168,9 @@ Statistics.prototype.addConnectionStatsListener = function(listener) {
 };
 
 Statistics.prototype.removeConnectionStatsListener = function(listener) {
-    this.eventEmitter.removeListener(StatisticsEvents.CONNECTION_STATS, listener);
+    this.eventEmitter.removeListener(
+        StatisticsEvents.CONNECTION_STATS,
+        listener);
 };
 
 Statistics.prototype.addByteSentStatsListener = function(listener) {

modules/xmpp/ChatRoom.js

@@ -93,9 +93,11 @@ export default class ChatRoom extends Listenable {
         this.focusMucJid = null;
         this.noBridgeAvailable = false;
         this.options = options || {};
-        this.moderator = new Moderator(this.roomjid, this.xmpp, this.eventEmitter,
-            { connection: this.xmpp.options,
-                conference: this.options });
+        this.moderator
+            = new Moderator(this.roomjid, this.xmpp, this.eventEmitter, {
+                connection: this.xmpp.options,
+                conference: this.options
+            });
         this.initPresenceMap();
         this.lastPresences = {};
         this.phoneNumber = null;
@@ -116,8 +118,9 @@ export default class ChatRoom extends Listenable {
             'attributes': { xmlns: 'http://jitsi.org/jitmeet/user-agent' }
         });
 
-        // We need to broadcast 'videomuted' status from the beginning, cause Jicofo
-        // makes decisions based on that. Initialize it with 'false' here.
+        // We need to broadcast 'videomuted' status from the beginning, cause
+        // Jicofo makes decisions based on that. Initialize it with 'false'
+        // here.
         this.addVideoInfoToPresence(false);
     }
 
@@ -153,10 +156,10 @@ export default class ChatRoom extends Listenable {
         const pres = $pres({ to });
 
         // xep-0045 defines: "including in the initial presence stanza an empty
-        // <x/> element qualified by the 'http://jabber.org/protocol/muc' namespace"
-        // and subsequent presences should not include that or it can be considered
-        // as joining, and server can send us the message history for the room on
-        // every presence
+        // <x/> element qualified by the 'http://jabber.org/protocol/muc'
+        // namespace" and subsequent presences should not include that or it can
+        // be considered as joining, and server can send us the message history
+        // for the room on every presence
         if (fromJoin) {
             pres.c('x', { xmlns: this.presMap.xns });
 
@@ -171,8 +174,8 @@ export default class ChatRoom extends Listenable {
         if (fromJoin) {
             // XXX We're pressed for time here because we're beginning a complex
             // and/or lengthy conference-establishment process which supposedly
-            // involves multiple RTTs. We don't have the time to wait for Strophe to
-            // decide to send our IQ.
+            // involves multiple RTTs. We don't have the time to wait for
+            // Strophe to decide to send our IQ.
             this.connection.flush();
         }
     }
@@ -189,16 +192,17 @@ export default class ChatRoom extends Listenable {
         this.presMap.length = 0;
 
         // XXX Strophe is asynchronously sending by default. Unfortunately, that
-        // means that there may not be enough time to send the unavailable presence.
-        // Switching Strophe to synchronous sending is not much of an option because
-        // it may lead to a noticeable delay in navigating away from the current
-        // location. As a compromise, we will try to increase the chances of sending
-        // the unavailable presence within the short time span that we have upon
-        // unloading by invoking flush() on the connection. We flush() once before
-        // sending/queuing the unavailable presence in order to attemtp to have the
-        // unavailable presence at the top of the send queue. We flush() once more
-        // after sending/queuing the unavailable presence in order to attempt to
-        // have it sent as soon as possible.
+        // means that there may not be enough time to send the unavailable
+        // presence. Switching Strophe to synchronous sending is not much of an
+        // option because it may lead to a noticeable delay in navigating away
+        // from the current location. As a compromise, we will try to increase
+        // the chances of sending the unavailable presence within the short time
+        // span that we have upon unloading by invoking flush() on the
+        // connection. We flush() once before sending/queuing the unavailable
+        // presence in order to attemtp to have the unavailable presence at the
+        // top of the send queue. We flush() once more after sending/queuing the
+        // unavailable presence in order to attempt to have it sent as soon as
+        // possible.
         this.connection.flush();
         this.connection.send(pres);
         this.connection.flush();
@@ -323,11 +327,14 @@ export default class ChatRoom extends Listenable {
         }
 
         if (from == this.myroomjid) {
-            const newRole = member.affiliation == 'owner' ? member.role : 'none';
+            const newRole
+                = member.affiliation == 'owner' ? member.role : 'none';
 
             if (this.role !== newRole) {
                 this.role = newRole;
-                this.eventEmitter.emit(XMPPEvents.LOCAL_ROLE_CHANGED, this.role);
+                this.eventEmitter.emit(
+                    XMPPEvents.LOCAL_ROLE_CHANGED,
+                    this.role);
             }
             if (!this.joined) {
                 this.joined = true;
@@ -367,13 +374,14 @@ export default class ChatRoom extends Listenable {
 
             if (member.isFocus) {
                 // From time to time first few presences of the focus are not
-                // containing it's jid. That way we can mark later the focus member
-                // instead of not marking it at all and not starting the conference.
-                // FIXME: Maybe there is a better way to handle this issue. It seems
-                // there is some period of time in prosody that the configuration
-                // form is received but not applied. And if any participant joins
-                // during that period of time the first presence from the focus
-                // won't conain <item jid="focus..." />.
+                // containing it's jid. That way we can mark later the focus
+                // member instead of not marking it at all and not starting the
+                // conference.
+                // FIXME: Maybe there is a better way to handle this issue. It
+                // seems there is some period of time in prosody that the
+                // configuration form is received but not applied. And if any
+                // participant joins during that period of time the first
+                // presence from the focus won't conain <item jid="focus..." />.
                 memberOfThis.isFocus = true;
                 this._initFocus(from, jid);
             }
@@ -397,7 +405,9 @@ export default class ChatRoom extends Listenable {
 
                     if (displayName && displayName.length > 0) {
                         this.eventEmitter.emit(
-                                XMPPEvents.DISPLAY_NAME_CHANGED, from, displayName);
+                            XMPPEvents.DISPLAY_NAME_CHANGED,
+                            from,
+                            displayName);
                     }
                 }
                 break;
@@ -461,8 +471,8 @@ export default class ChatRoom extends Listenable {
     }
 
     /**
-     * Sets the special listener to be used for "command"s whose name starts with
-     * "jitsi_participant_".
+     * Sets the special listener to be used for "command"s whose name starts
+     * with "jitsi_participant_".
      */
     setParticipantPropertyListener(listener) {
         this.participantPropertyListener = listener;
@@ -493,7 +503,10 @@ export default class ChatRoom extends Listenable {
 
         msg.c('body', body).up();
         if (nickname) {
-            msg.c('nick', { xmlns: 'http://jabber.org/protocol/nick' }).t(nickname).up().up();
+            msg.c('nick', { xmlns: 'http://jabber.org/protocol/nick' })
+                .t(nickname)
+                .up()
+                .up();
         }
         this.connection.send(msg);
         this.eventEmitter.emit(XMPPEvents.SENDING_CHAT_MESSAGE, body);
@@ -548,12 +561,20 @@ export default class ChatRoom extends Listenable {
         }
 
         // Status code 110 indicates that this notification is "self-presence".
-        const isSelfPresence = $(pres).find(
-                '>x[xmlns="http://jabber.org/protocol/muc#user"]>status[code="110"]'
-            ).length !== 0;
-        const isKick = $(pres).find(
-                '>x[xmlns="http://jabber.org/protocol/muc#user"]>status[code="307"]'
-            ).length !== 0;
+        const isSelfPresence
+            = $(pres)
+                    .find(
+                        '>x[xmlns="http://jabber.org/protocol/muc#user"]>'
+                            + 'status[code="110"]')
+                    .length
+                !== 0;
+        const isKick
+            = $(pres)
+                    .find(
+                        '>x[xmlns="http://jabber.org/protocol/muc#user"]'
+                            + '>status[code="307"]')
+                    .length
+                !== 0;
         const membersKeys = Object.keys(this.members);
 
         if (!isSelfPresence) {
@@ -620,13 +641,19 @@ export default class ChatRoom extends Listenable {
 
             if (stamp) {
                 // the format is CCYYMMDDThh:mm:ss
-                const dateParts = stamp.match(/(\d{4})(\d{2})(\d{2}T\d{2}:\d{2}:\d{2})/);
+                const dateParts
+                    = stamp.match(/(\d{4})(\d{2})(\d{2}T\d{2}:\d{2}:\d{2})/);
 
                 stamp = `${dateParts[1]}-${dateParts[2]}-${dateParts[3]}Z`;
             }
         }
 
-        if (from == this.roomjid && $(msg).find('>x[xmlns="http://jabber.org/protocol/muc#user"]>status[code="104"]').length) {
+        if (from == this.roomjid
+                && $(msg)
+                    .find(
+                        '>x[xmlns="http://jabber.org/protocol/muc#user"]'
+                            + '>status[code="104"]')
+                    .length) {
             this.discoRoomInfo();
         }
 
@@ -638,11 +665,20 @@ export default class ChatRoom extends Listenable {
     }
 
     onPresenceError(pres, from) {
-        if ($(pres).find('>error[type="auth"]>not-authorized[xmlns="urn:ietf:params:xml:ns:xmpp-stanzas"]').length) {
+        if ($(pres)
+                .find(
+                    '>error[type="auth"]'
+                        + '>not-authorized['
+                        + 'xmlns="urn:ietf:params:xml:ns:xmpp-stanzas"]')
+                .length) {
             logger.log('on password required', from);
             this.eventEmitter.emit(XMPPEvents.PASSWORD_REQUIRED);
-        } else if ($(pres).find(
-            '>error[type="cancel"]>not-allowed[xmlns="urn:ietf:params:xml:ns:xmpp-stanzas"]').length) {
+        } else if ($(pres)
+                .find(
+                    '>error[type="cancel"]'
+                        + '>not-allowed['
+                        + 'xmlns="urn:ietf:params:xml:ns:xmpp-stanzas"]')
+                .length) {
             const toDomain = Strophe.getDomainFromJid(pres.getAttribute('to'));
 
             if (toDomain === this.xmpp.options.hosts.anonymousdomain) {
@@ -654,7 +690,8 @@ export default class ChatRoom extends Listenable {
 
             } else {
                 logger.warn('onPresError ', pres);
-                this.eventEmitter.emit(XMPPEvents.ROOM_CONNECT_NOT_ALLOWED_ERROR);
+                this.eventEmitter.emit(
+                    XMPPEvents.ROOM_CONNECT_NOT_ALLOWED_ERROR);
             }
         } else if ($(pres).find('>error>service-unavailable').length) {
             logger.warn('Maximum users limit for the room has been reached',
@@ -683,22 +720,51 @@ export default class ChatRoom extends Listenable {
     lockRoom(key, onSuccess, onError, onNotSupported) {
         // http://xmpp.org/extensions/xep-0045.html#roomconfig
         this.connection.sendIQ(
-            $iq({ to: this.roomjid,
-                type: 'get' }).c('query', { xmlns: 'http://jabber.org/protocol/muc#owner' }),
+            $iq({
+                to: this.roomjid,
+                type: 'get'
+            })
+                .c('query', { xmlns: 'http://jabber.org/protocol/muc#owner' }),
             res => {
-                if ($(res).find('>query>x[xmlns="jabber:x:data"]>field[var="muc#roomconfig_roomsecret"]').length) {
+                if ($(res)
+                        .find(
+                            '>query>x[xmlns="jabber:x:data"]'
+                                + '>field[var="muc#roomconfig_roomsecret"]')
+                        .length) {
                     const formsubmit
-                        = $iq({ to: this.roomjid,
-                            type: 'set' })
-                            .c('query', { xmlns: 'http://jabber.org/protocol/muc#owner' });
-
-                    formsubmit.c('x', { xmlns: 'jabber:x:data',
-                        type: 'submit' });
-                    formsubmit.c('field', { 'var': 'FORM_TYPE' }).c('value').t('http://jabber.org/protocol/muc#roomconfig').up().up();
-                    formsubmit.c('field', { 'var': 'muc#roomconfig_roomsecret' }).c('value').t(key).up().up();
-
-                    // Fixes a bug in prosody 0.9.+ https://code.google.com/p/lxmppd/issues/detail?id=373
-                    formsubmit.c('field', { 'var': 'muc#roomconfig_whois' }).c('value').t('anyone').up().up();
+                        = $iq({
+                            to: this.roomjid,
+                            type: 'set'
+                        })
+                            .c('query', {
+                                xmlns: 'http://jabber.org/protocol/muc#owner'
+                            });
+
+                    formsubmit.c('x', {
+                        xmlns: 'jabber:x:data',
+                        type: 'submit'
+                    });
+                    formsubmit
+                        .c('field', { 'var': 'FORM_TYPE' })
+                        .c('value')
+                        .t('http://jabber.org/protocol/muc#roomconfig')
+                        .up()
+                        .up();
+                    formsubmit
+                        .c('field', { 'var': 'muc#roomconfig_roomsecret' })
+                        .c('value')
+                        .t(key)
+                        .up()
+                        .up();
+
+                    // Fixes a bug in prosody 0.9.+
+                    // https://code.google.com/p/lxmppd/issues/detail?id=373
+                    formsubmit
+                        .c('field', { 'var': 'muc#roomconfig_whois' })
+                        .c('value')
+                        .t('anyone')
+                        .up()
+                        .up();
 
                     // FIXME: is muc#roomconfig_passwordprotectedroom required?
                     this.connection.sendIQ(formsubmit, onSuccess, onError);
@@ -733,8 +799,8 @@ export default class ChatRoom extends Listenable {
      * Checks if the user identified by given <tt>mucJid</tt> is the conference
      * focus.
      * @param mucJid the full MUC address of the user to be checked.
-     * @returns {boolean|null} <tt>true</tt> if MUC user is the conference focus or
-     * <tt>false</tt> if is not. When given <tt>mucJid</tt> does not exist in
+     * @returns {boolean|null} <tt>true</tt> if MUC user is the conference focus
+     * or <tt>false</tt> if is not. When given <tt>mucJid</tt> does not exist in
      * the MUC then <tt>null</tt> is returned.
      */
     isFocus(mucJid) {
@@ -861,8 +927,8 @@ export default class ChatRoom extends Listenable {
     }
 
     /**
-     * Returns null if the recording is not supported, "on" if the recording started
-     * and "off" if the recording is not started.
+     * Returns null if the recording is not supported, "on" if the recording
+     * started and "off" if the recording is not started.
      */
     getRecordingState() {
         return this.recording ? this.recording.getState() : undefined;
@@ -878,7 +944,8 @@ export default class ChatRoom extends Listenable {
     /**
      * Starts/stops the recording
      * @param token token for authentication
-     * @param statusChangeHandler {function} receives the new status as argument.
+     * @param statusChangeHandler {function} receives the new status as
+     * argument.
      */
     toggleRecording(options, statusChangeHandler) {
         if (this.recording) {
@@ -967,7 +1034,9 @@ export default class ChatRoom extends Listenable {
         if (mute.length) {
             const doMuteAudio = mute.text() === 'true';
 
-            this.eventEmitter.emit(XMPPEvents.AUDIO_MUTED_BY_FOCUS, doMuteAudio);
+            this.eventEmitter.emit(
+                XMPPEvents.AUDIO_MUTED_BY_FOCUS,
+                doMuteAudio);
         }
 
         return true;
@@ -975,8 +1044,8 @@ export default class ChatRoom extends Listenable {
 
     /**
      * Leaves the room. Closes the jingle session.
-     * @returns {Promise} which is resolved if XMPPEvents.MUC_LEFT is received less
-     * than 5s after sending presence unavailable. Otherwise the promise is
+     * @returns {Promise} which is resolved if XMPPEvents.MUC_LEFT is received
+     * less than 5s after sending presence unavailable. Otherwise the promise is
      * rejected.
      */
     leave() {

modules/xmpp/JingleSessionPC.js

@@ -779,7 +779,8 @@ export default class JingleSessionPC extends JingleSession {
                     finishedCallback();
                 }, error => {
                     logger.error(
-                        `Error renegotiating after processing remote source-add: ${error}`);
+                        'Error renegotiating after processing remote'
+                            + ` source-add: ${error}`);
                     finishedCallback(error);
                 });
         };
@@ -823,7 +824,8 @@ export default class JingleSessionPC extends JingleSession {
                     finishedCallback();
                 }, error => {
                     logger.error(
-                        `Error renegotiating after processing remote source-remove: ${error}`);
+                        'Error renegotiating after processing remote'
+                          + ` source-remove: ${error}`);
                     finishedCallback(error);
                 });
         };
@@ -954,7 +956,9 @@ export default class JingleSessionPC extends JingleSession {
                 remoteDescription,
                 () => {
                     if (this.signalingState === 'closed') {
-                        reject('Attempted to setRemoteDescription in state closed');
+                        reject(
+                            'Attempted to setRemoteDescription in state'
+                                + ' closed');
 
                         return;
                     }
@@ -1165,7 +1169,8 @@ export default class JingleSessionPC extends JingleSession {
         const workFunction = finishedCallback => {
             if (!this.peerconnection) {
                 finishedCallback(
-                    'Error: tried adding stream with no active peer connection');
+                    'Error: tried adding stream with no active peer'
+                        + ' connection');
 
                 return;
             }

modules/xmpp/SDP.js

@@ -51,7 +51,10 @@ SDP.prototype.getMediaSsrcMap = function() {
 
     for (let mediaindex = 0; mediaindex < self.media.length; mediaindex++) {
         tmp = SDPUtil.find_lines(self.media[mediaindex], 'a=ssrc:');
-        const mid = SDPUtil.parse_mid(SDPUtil.find_line(self.media[mediaindex], 'a=mid:'));
+        const mid
+            = SDPUtil.parse_mid(
+                SDPUtil.find_line(self.media[mediaindex],
+                'a=mid:'));
         const media = {
             mediaindex,
             mid,
@@ -223,11 +226,18 @@ SDP.prototype.toJingle = function(elem, thecreator) {
                 elem.attrs({ ssrc });
             }
             for (j = 0; j < mline.fmt.length; j++) {
-                rtpmap = SDPUtil.find_line(this.media[i], `a=rtpmap:${mline.fmt[j]}`);
+                rtpmap
+                    = SDPUtil.find_line(
+                        this.media[i],
+                        `a=rtpmap:${mline.fmt[j]}`);
                 elem.c('payload-type', SDPUtil.parse_rtpmap(rtpmap));
 
-                // put any 'a=fmtp:' + mline.fmt[j] lines into <param name=foo value=bar/>
-                const afmtpline = SDPUtil.find_line(this.media[i], `a=fmtp:${mline.fmt[j]}`);
+                // put any 'a=fmtp:' + mline.fmt[j] lines into <param name=foo
+                // value=bar/>
+                const afmtpline
+                    = SDPUtil.find_line(
+                        this.media[i],
+                        `a=fmtp:${mline.fmt[j]}`);
 
                 if (afmtpline) {
                     tmp = SDPUtil.parse_fmtp(afmtpline);
@@ -235,11 +245,14 @@ SDP.prototype.toJingle = function(elem, thecreator) {
                         elem.c('parameter', tmp[k]).up();
                     }
                 }
-                this.rtcpFbToJingle(i, elem, mline.fmt[j]); // XEP-0293 -- map a=rtcp-fb
+
+                // XEP-0293 -- map a=rtcp-fb
+                this.rtcpFbToJingle(i, elem, mline.fmt[j]);
 
                 elem.up();
             }
-            const crypto = SDPUtil.find_lines(this.media[i], 'a=crypto:', this.session);
+            const crypto
+                = SDPUtil.find_lines(this.media[i], 'a=crypto:', this.session);
 
             if (crypto.length) {
                 elem.c('encryption', { required: 1 });
@@ -327,7 +340,8 @@ SDP.prototype.toJingle = function(elem, thecreator) {
                 elem.up();
 
                 // XEP-0339 handle ssrc-group attributes
-                const ssrc_group_lines = SDPUtil.find_lines(this.media[i], 'a=ssrc-group:');
+                const ssrc_group_lines
+                    = SDPUtil.find_lines(this.media[i], 'a=ssrc-group:');
 
                 ssrc_group_lines.forEach(line => {
                     const idx = line.indexOf(' ');
@@ -355,9 +369,11 @@ SDP.prototype.toJingle = function(elem, thecreator) {
             if (lines.length) {
                 for (j = 0; j < lines.length; j++) {
                     tmp = SDPUtil.parse_extmap(lines[j]);
-                    elem.c('rtp-hdrext', { xmlns: 'urn:xmpp:jingle:apps:rtp:rtp-hdrext:0',
+                    elem.c('rtp-hdrext', {
+                        xmlns: 'urn:xmpp:jingle:apps:rtp:rtp-hdrext:0',
                         uri: tmp.uri,
-                        id: tmp.value });
+                        id: tmp.value
+                    });
                     if (tmp.hasOwnProperty('direction')) {
                         switch (tmp.direction) {
                         case 'sendonly':
@@ -385,13 +401,15 @@ SDP.prototype.toJingle = function(elem, thecreator) {
         // map ice-ufrag/pwd, dtls fingerprint, candidates
         this.transportToJingle(i, elem);
 
-        if (SDPUtil.find_line(this.media[i], 'a=sendrecv', this.session)) {
+        const m = this.media[i];
+
+        if (SDPUtil.find_line(m, 'a=sendrecv', this.session)) {
             elem.attrs({ senders: 'both' });
-        } else if (SDPUtil.find_line(this.media[i], 'a=sendonly', this.session)) {
+        } else if (SDPUtil.find_line(m, 'a=sendonly', this.session)) {
             elem.attrs({ senders: 'initiator' });
-        } else if (SDPUtil.find_line(this.media[i], 'a=recvonly', this.session)) {
+        } else if (SDPUtil.find_line(m, 'a=recvonly', this.session)) {
             elem.attrs({ senders: 'responder' });
-        } else if (SDPUtil.find_line(this.media[i], 'a=inactive', this.session)) {
+        } else if (SDPUtil.find_line(m, 'a=inactive', this.session)) {
             elem.attrs({ senders: 'none' });
         }
         if (mline.port == '0') {
@@ -433,14 +451,21 @@ SDP.prototype.transportToJingle = function(mediaindex, elem) {
 
     // XEP-0320
     const fingerprints
-        = SDPUtil.find_lines(this.media[mediaindex], 'a=fingerprint:', this.session);
+        = SDPUtil.find_lines(
+            this.media[mediaindex],
+            'a=fingerprint:',
+            this.session);
 
     fingerprints.forEach(line => {
         tmp = SDPUtil.parse_fingerprint(line);
         tmp.xmlns = 'urn:xmpp:jingle:apps:dtls:0';
         elem.c('fingerprint').t(tmp.fingerprint);
         delete tmp.fingerprint;
-        line = SDPUtil.find_line(self.media[mediaindex], 'a=setup:', self.session);
+        line
+            = SDPUtil.find_line(
+                self.media[mediaindex],
+                'a=setup:',
+                self.session);
         if (line) {
             tmp.setup = line.substr(8);
         }
@@ -453,18 +478,23 @@ SDP.prototype.transportToJingle = function(mediaindex, elem) {
         elem.attrs(tmp);
 
         // XEP-0176
-        if (SDPUtil.find_line(this.media[mediaindex], 'a=candidate:', this.session)) { // add any a=candidate lines
-            const lines = SDPUtil.find_lines(this.media[mediaindex], 'a=candidate:', this.session);
+        const lines
+            = SDPUtil.find_lines(
+                this.media[mediaindex],
+                'a=candidate:',
+                this.session);
 
+        if (lines.length) { // add any a=candidate lines
             lines.forEach(line => {
                 const candidate = SDPUtil.candidateToJingle(line);
 
                 if (self.failICE) {
                     candidate.ip = '1.1.1.1';
                 }
-                const protocol = candidate
-                        && typeof candidate.protocol === 'string'
-                    ? candidate.protocol.toLowerCase() : '';
+                const protocol
+                    = candidate && typeof candidate.protocol === 'string'
+                        ? candidate.protocol.toLowerCase()
+                        : '';
 
                 if ((self.removeTcpCandidates
                         && (protocol === 'tcp' || protocol === 'ssltcp'))
@@ -478,19 +508,27 @@ SDP.prototype.transportToJingle = function(mediaindex, elem) {
     elem.up(); // end of transport
 };
 
-SDP.prototype.rtcpFbToJingle = function(mediaindex, elem, payloadtype) { // XEP-0293
-    const lines = SDPUtil.find_lines(this.media[mediaindex], `a=rtcp-fb:${payloadtype}`);
+// XEP-0293
+SDP.prototype.rtcpFbToJingle = function(mediaindex, elem, payloadtype) {
+    const lines
+        = SDPUtil.find_lines(
+            this.media[mediaindex],
+            `a=rtcp-fb:${payloadtype}`);
 
     lines.forEach(line => {
         const tmp = SDPUtil.parse_rtcpfb(line);
 
         if (tmp.type == 'trr-int') {
-            elem.c('rtcp-fb-trr-int', { xmlns: 'urn:xmpp:jingle:apps:rtp:rtcp-fb:0',
-                value: tmp.params[0] });
+            elem.c('rtcp-fb-trr-int', {
+                xmlns: 'urn:xmpp:jingle:apps:rtp:rtcp-fb:0',
+                value: tmp.params[0]
+            });
             elem.up();
         } else {
-            elem.c('rtcp-fb', { xmlns: 'urn:xmpp:jingle:apps:rtp:rtcp-fb:0',
-                type: tmp.type });
+            elem.c('rtcp-fb', {
+                xmlns: 'urn:xmpp:jingle:apps:rtp:rtcp-fb:0',
+                type: tmp.type
+            });
             if (tmp.params.length > 0) {
                 elem.attrs({ 'subtype': tmp.params[0] });
             }
@@ -501,7 +539,9 @@ SDP.prototype.rtcpFbToJingle = function(mediaindex, elem, payloadtype) { // XEP-
 
 SDP.prototype.rtcpFbFromJingle = function(elem, payloadtype) { // XEP-0293
     let media = '';
-    let tmp = elem.find('>rtcp-fb-trr-int[xmlns="urn:xmpp:jingle:apps:rtp:rtcp-fb:0"]');
+    let tmp
+        = elem.find(
+            '>rtcp-fb-trr-int[xmlns="urn:xmpp:jingle:apps:rtp:rtcp-fb:0"]');
 
     if (tmp.length) {
         media += 'a=rtcp-fb:* trr-int ';
@@ -533,7 +573,8 @@ SDP.prototype.fromJingle = function(jingle) {
         + 's=-\r\n'
         + 't=0 0\r\n';
 
-    // http://tools.ietf.org/html/draft-ietf-mmusic-sdp-bundle-negotiation-04#section-8
+    // http://tools.ietf.org/html/draft-ietf-mmusic-sdp-bundle-negotiation-04
+    // #section-8
     const groups
         = $(jingle).find('>group[xmlns="urn:xmpp:jingle:apps:grouping:0"]');
 
@@ -547,7 +588,10 @@ SDP.prototype.fromJingle = function(jingle) {
 
             if (contents.length > 0) {
                 self.raw
-                    += `a=group:${group.getAttribute('semantics') || group.getAttribute('type')} ${contents.join(' ')}\r\n`;
+                    += `a=group:${
+                        group.getAttribute('semantics')
+                            || group.getAttribute('type')} ${
+                        contents.join(' ')}\r\n`;
             }
         });
     }
@@ -617,7 +661,9 @@ SDP.prototype.jingle2media = function(content) {
     if (!sctp.length) {
         media += 'a=rtcp:1 IN IP4 0.0.0.0\r\n';
     }
-    tmp = content.find('>transport[xmlns="urn:xmpp:jingle:transports:ice-udp:1"]');
+    tmp
+        = content.find(
+            '>transport[xmlns="urn:xmpp:jingle:transports:ice-udp:1"]');
     if (tmp.length) {
         if (tmp.attr('ufrag')) {
             media += `${SDPUtil.build_iceufrag(tmp.attr('ufrag'))}\r\n`;
@@ -652,7 +698,8 @@ SDP.prototype.jingle2media = function(content) {
     media += `a=mid:${content.attr('name')}\r\n`;
 
     // <description><rtcp-mux/></description>
-    // see http://code.google.com/p/libjingle/issues/detail?id=309 -- no spec though
+    // see http://code.google.com/p/libjingle/issues/detail?id=309 -- no spec
+    // though
     // and http://mail.jabber.org/pipermail/jingle/2011-December/001761.html
     if (desc.find('rtcp-mux').length) {
         media += 'a=rtcp-mux\r\n';
@@ -694,44 +741,57 @@ SDP.prototype.jingle2media = function(content) {
     media += self.rtcpFbFromJingle(desc, '*');
 
     // xep-0294
-    tmp = desc.find('>rtp-hdrext[xmlns="urn:xmpp:jingle:apps:rtp:rtp-hdrext:0"]');
+    tmp
+        = desc.find(
+            '>rtp-hdrext[xmlns="urn:xmpp:jingle:apps:rtp:rtp-hdrext:0"]');
     tmp.each(function() {
-        media += `a=extmap:${this.getAttribute('id')} ${this.getAttribute('uri')}\r\n`;
+        media
+            += `a=extmap:${this.getAttribute('id')} ${
+                this.getAttribute('uri')}\r\n`;
     });
 
-    content.find('>transport[xmlns="urn:xmpp:jingle:transports:ice-udp:1"]>candidate').each(function() {
-        let protocol = this.getAttribute('protocol');
-
-        protocol = typeof protocol === 'string' ? protocol.toLowerCase() : '';
-
-        if ((self.removeTcpCandidates
-                && (protocol === 'tcp' || protocol === 'ssltcp'))
-            || (self.removeUdpCandidates && protocol === 'udp')) {
-            return;
-        } else if (self.failICE) {
-            this.setAttribute('ip', '1.1.1.1');
-        }
+    content
+        .find(
+            '>transport[xmlns="urn:xmpp:jingle:transports:ice-udp:1"]'
+                + '>candidate')
+        .each(function() {
+            let protocol = this.getAttribute('protocol');
+
+            protocol
+                = typeof protocol === 'string' ? protocol.toLowerCase() : '';
+
+            if ((self.removeTcpCandidates
+                    && (protocol === 'tcp' || protocol === 'ssltcp'))
+                || (self.removeUdpCandidates && protocol === 'udp')) {
+                return;
+            } else if (self.failICE) {
+                this.setAttribute('ip', '1.1.1.1');
+            }
 
-        media += SDPUtil.candidateFromJingle(this);
-    });
+            media += SDPUtil.candidateFromJingle(this);
+        });
 
     // XEP-0339 handle ssrc-group attributes
-    content.find('description>ssrc-group[xmlns="urn:xmpp:jingle:apps:rtp:ssma:0"]').each(function() {
-        const semantics = this.getAttribute('semantics');
-        const ssrcs
-            = $(this)
-                .find('>source')
-                .map(function() {
-                    return this.getAttribute('ssrc');
-                })
-                .get();
+    content
+        .find('description>ssrc-group[xmlns="urn:xmpp:jingle:apps:rtp:ssma:0"]')
+        .each(function() {
+            const semantics = this.getAttribute('semantics');
+            const ssrcs
+                = $(this)
+                    .find('>source')
+                    .map(function() {
+                        return this.getAttribute('ssrc');
+                    })
+                    .get();
 
-        if (ssrcs.length) {
-            media += `a=ssrc-group:${semantics} ${ssrcs.join(' ')}\r\n`;
-        }
-    });
+            if (ssrcs.length) {
+                media += `a=ssrc-group:${semantics} ${ssrcs.join(' ')}\r\n`;
+            }
+        });
 
-    tmp = content.find('description>source[xmlns="urn:xmpp:jingle:apps:rtp:ssma:0"]');
+    tmp
+        = content.find(
+            'description>source[xmlns="urn:xmpp:jingle:apps:rtp:ssma:0"]');
     tmp.each(function() {
         const ssrc = this.getAttribute('ssrc');
 

modules/xmpp/SDPUtil.js

@@ -16,7 +16,11 @@ const SDPUtil = {
         let pwd, ufrag;
 
         if ((ufrag = SDPUtil.find_line(mediadesc, 'a=ice-ufrag:', sessiondesc))
-                && (pwd = SDPUtil.find_line(mediadesc, 'a=ice-pwd:', sessiondesc))) {
+                && (pwd
+                    = SDPUtil.find_line(
+                        mediadesc,
+                        'a=ice-pwd:',
+                        sessiondesc))) {
             data = {
                 ufrag: SDPUtil.parse_iceufrag(ufrag),
                 pwd: SDPUtil.parse_icepwd(pwd)
@@ -55,7 +59,9 @@ const SDPUtil = {
         return data;
     },
     build_mline(mline) {
-        return `m=${mline.media} ${mline.port} ${mline.proto} ${mline.fmt.join(' ')}`;
+        return (
+            `m=${mline.media} ${mline.port} ${mline.proto} ${
+                mline.fmt.join(' ')}`);
     },
     parse_rtpmap(line) {
         const data = {};
@@ -87,7 +93,9 @@ const SDPUtil = {
         return [ sctpPort, protocol, streamCount ];// SCTP port
     },
     build_rtpmap(el) {
-        let line = `a=rtpmap:${el.getAttribute('id')} ${el.getAttribute('name')}/${el.getAttribute('clockrate')}`;
+        let line
+            = `a=rtpmap:${el.getAttribute('id')} ${el.getAttribute('name')}/${
+                el.getAttribute('clockrate')}`;
 
         if (el.getAttribute('channels') && el.getAttribute('channels') != '1') {
             line += `/${el.getAttribute('channels')}`;
@@ -173,7 +181,9 @@ const SDPUtil = {
                 candidate.tcptype = elems[i + 1];
                 break;
             default: // TODO
-                logger.log(`parse_icecandidate not translating "${elems[i]}" = "${elems[i + 1]}"`);
+                logger.log(
+                    `parse_icecandidate not translating "${
+                        elems[i]}" = "${elems[i + 1]}"`);
             }
         }
         candidate.network = '1';
@@ -185,14 +195,24 @@ const SDPUtil = {
         return candidate;
     },
     build_icecandidate(cand) {
-        let line = [ `a=candidate:${cand.foundation}`, cand.component, cand.protocol, cand.priority, cand.ip, cand.port, 'typ', cand.type ].join(' ');
+        let line = [
+            `a=candidate:${cand.foundation}`,
+            cand.component,
+            cand.protocol,
+            cand.priority,
+            cand.ip,
+            cand.port,
+            'typ',
+            cand.type
+        ].join(' ');
 
         line += ' ';
         switch (cand.type) {
         case 'srflx':
         case 'prflx':
         case 'relay':
-            if (cand.hasOwnAttribute('rel-addr') && cand.hasOwnAttribute('rel-port')) {
+            if (cand.hasOwnAttribute('rel-addr')
+                    && cand.hasOwnAttribute('rel-port')) {
                 line += 'raddr';
                 line += ' ';
                 line += cand['rel-addr'];
@@ -218,8 +238,8 @@ const SDPUtil = {
     },
     parse_ssrc(desc) {
         // proprietary mapping of a=ssrc lines
-        // TODO: see "Jingle RTP Source Description" by Juberti and P. Thatcher on google docs
-        // and parse according to that
+        // TODO: see "Jingle RTP Source Description" by Juberti and P. Thatcher
+        // on google docs and parse according to that
         const data = {};
         const lines = desc.split('\r\n');
 
@@ -227,7 +247,8 @@ const SDPUtil = {
             if (lines[i].substring(0, 7) == 'a=ssrc:') {
                 const idx = lines[i].indexOf(' ');
 
-                data[lines[i].substr(idx + 1).split(':', 2)[0]] = lines[i].substr(idx + 1).split(':', 2)[1];
+                data[lines[i].substr(idx + 1).split(':', 2)[0]]
+                    = lines[i].substr(idx + 1).split(':', 2)[1];
             }
         }
 
@@ -305,12 +326,16 @@ const SDPUtil = {
         return needles;
     },
     candidateToJingle(line) {
-        // a=candidate:2979166662 1 udp 2113937151 192.168.2.100 57698 typ host generation 0
-        //      <candidate component=... foundation=... generation=... id=... ip=... network=... port=... priority=... protocol=... type=.../>
+        // a=candidate:2979166662 1 udp 2113937151 192.168.2.100 57698 typ host
+        // generation 0
+        //      <candidate component=... foundation=... generation=... id=...
+        // ip=... network=... port=... priority=... protocol=... type=.../>
         if (line.indexOf('candidate:') === 0) {
             line = `a=${line}`;
         } else if (line.substring(0, 12) != 'a=candidate:') {
-            logger.log('parseCandidate called with a line that is not a candidate line');
+            logger.log(
+                'parseCandidate called with a line that is not a candidate'
+                    + ' line');
             logger.log(line);
 
             return null;
@@ -395,7 +420,8 @@ const SDPUtil = {
         case 'srflx':
         case 'prflx':
         case 'relay':
-            if (cand.getAttribute('rel-addr') && cand.getAttribute('rel-port')) {
+            if (cand.getAttribute('rel-addr')
+                    && cand.getAttribute('rel-port')) {
                 line += 'raddr';
                 line += ' ';
                 line += cand.getAttribute('rel-addr');
@@ -540,7 +566,8 @@ const SDPUtil = {
             }
         }
         if (payloadType) {
-            const payloadTypes = videoMLine.payloads.split(' ').map(p => parseInt(p));
+            const payloadTypes
+                = videoMLine.payloads.split(' ').map(p => parseInt(p));
             const payloadIndex = payloadTypes.indexOf(payloadType);
 
             payloadTypes.splice(payloadIndex, 1);

modules/xmpp/strophe.jingle.js

@@ -55,8 +55,12 @@ class JingleConnectionPlugin extends ConnectionPlugin {
                 ack.attrs({ type: 'error' });
                 ack.c('error', { type: 'cancel' })
                     .c('item-not-found', {
-                        xmlns: 'urn:ietf:params:xml:ns:xmpp-stanzas' }).up()
-                    .c('unknown-session', { xmlns: 'urn:xmpp:jingle:errors:1' });
+                        xmlns: 'urn:ietf:params:xml:ns:xmpp-stanzas'
+                    })
+                    .up()
+                    .c('unknown-session', {
+                        xmlns: 'urn:xmpp:jingle:errors:1'
+                    });
                 logger.warn('invalid session id', iq);
                 this.connection.send(ack);
 
@@ -69,18 +73,26 @@ class JingleConnectionPlugin extends ConnectionPlugin {
                     'jid mismatch for session id', sid, sess.peerjid, iq);
                 ack.attrs({ type: 'error' });
                 ack.c('error', { type: 'cancel' })
-                    .c('item-not-found', { xmlns: 'urn:ietf:params:xml:ns:xmpp-stanzas' }).up()
-                    .c('unknown-session', { xmlns: 'urn:xmpp:jingle:errors:1' });
+                    .c('item-not-found', {
+                        xmlns: 'urn:ietf:params:xml:ns:xmpp-stanzas'
+                    })
+                    .up()
+                    .c('unknown-session', {
+                        xmlns: 'urn:xmpp:jingle:errors:1'
+                    });
                 this.connection.send(ack);
 
                 return true;
             }
         } else if (sess !== undefined) {
-            // existing session with same session id
-            // this might be out-of-order if the sess.peerjid is the same as from
+            // Existing session with same session id. This might be out-of-order
+            // if the sess.peerjid is the same as from.
             ack.attrs({ type: 'error' });
             ack.c('error', { type: 'cancel' })
-                .c('service-unavailable', { xmlns: 'urn:ietf:params:xml:ns:xmpp-stanzas' }).up();
+                .c('service-unavailable', {
+                    xmlns: 'urn:ietf:params:xml:ns:xmpp-stanzas'
+                })
+                .up();
             logger.warn('duplicate session id', sid, iq);
             this.connection.send(ack);
 
@@ -188,11 +200,13 @@ class JingleConnectionPlugin extends ConnectionPlugin {
         // uses time-limited credentials as described in
         // http://tools.ietf.org/html/draft-uberti-behave-turn-rest-00
         //
-        // see https://code.google.com/p/prosody-modules/source/browse/mod_turncredentials/mod_turncredentials.lua
-        // for a prosody module which implements this
+        // See https://code.google.com/p/prosody-modules/source/browse/
+        // mod_turncredentials/mod_turncredentials.lua
+        // for a prosody module which implements this.
         //
-        // currently, this doesn't work with updateIce and therefore credentials with a long
-        // validity have to be fetched before creating the peerconnection
+        // Currently, this doesn't work with updateIce and therefore credentials
+        // with a long validity have to be fetched before creating the
+        // peerconnection.
         // TODO: implement refresh via updateIce as described in
         //      https://code.google.com/p/webrtc/issues/detail?id=1650
         this.connection.sendIQ(
@@ -221,7 +235,8 @@ class JingleConnectionPlugin extends ConnectionPlugin {
                         dict.url = `${type}:`;
                         const username = el.attr('username');
 
-                            // https://code.google.com/p/webrtc/issues/detail?id=1508
+                        // https://code.google.com/p/webrtc/issues/detail
+                        // ?id=1508
 
                         if (username) {
                             if (navigator.userAgent.match(

modules/xmpp/xmpp.js

@@ -47,14 +47,15 @@ export default class XMPP extends Listenable {
 
         // Setup a disconnect on unload as a way to facilitate API consumers. It
         // sounds like they would want that. A problem for them though may be if
-        // they wanted to utilize the connected connection in an unload handler of
-        // their own. However, it should be fairly easy for them to do that by
-        // registering their unload handler before us.
+        // they wanted to utilize the connected connection in an unload handler
+        // of their own. However, it should be fairly easy for them to do that
+        // by registering their unload handler before us.
         $(window).on('beforeunload unload', this.disconnect.bind(this));
     }
 
     /**
-     * Initializes the list of feature advertised through the disco-info mechanism
+     * Initializes the list of feature advertised through the disco-info
+     * mechanism.
      */
     initFeaturesList() {
         // http://xmpp.org/extensions/xep-0167.html#support
@@ -237,9 +238,9 @@ export default class XMPP extends Listenable {
     }
 
     /**
-     * Attach to existing connection. Can be used for optimizations. For example:
-     * if the connection is created on the server we can attach to it and start
-     * using it.
+     * Attach to existing connection. Can be used for optimizations. For
+     * example: if the connection is created on the server we can attach to it
+     * and start using it.
      *
      * @param options {object} connecting options - rid, sid, jid and password.
      */
@@ -339,8 +340,8 @@ export default class XMPP extends Listenable {
     /**
      * Disconnects this from the XMPP server (if this is connected).
      *
-     * @param ev optionally, the event which triggered the necessity to disconnect
-     * from the XMPP server (e.g. beforeunload, unload)
+     * @param ev optionally, the event which triggered the necessity to
+     * disconnect from the XMPP server (e.g. beforeunload, unload).
      */
     disconnect(ev) {
         if (this.disconnectInProgress
@@ -354,16 +355,17 @@ export default class XMPP extends Listenable {
         this.disconnectInProgress = true;
 
         // XXX Strophe is asynchronously sending by default. Unfortunately, that
-        // means that there may not be enough time to send an unavailable presence
-        // or disconnect at all. Switching Strophe to synchronous sending is not
-        // much of an option because it may lead to a noticeable delay in navigating
-        // away from the current location. As a compromise, we will try to increase
-        // the chances of sending an unavailable presence and/or disconecting within
-        // the short time span that we have upon unloading by invoking flush() on
-        // the connection. We flush() once before disconnect() in order to attemtp
-        // to have its unavailable presence at the top of the send queue. We flush()
-        // once more after disconnect() in order to attempt to have its unavailable
-        // presence sent as soon as possible.
+        // means that there may not be enough time to send an unavailable
+        // presence or disconnect at all. Switching Strophe to synchronous
+        // sending is not much of an option because it may lead to a noticeable
+        // delay in navigating away from the current location. As a compromise,
+        // we will try to increase the chances of sending an unavailable
+        // presence and/or disconecting within the short time span that we have
+        // upon unloading by invoking flush() on the connection. We flush() once
+        // before disconnect() in order to attemtp to have its unavailable
+        // presence at the top of the send queue. We flush() once more after
+        // disconnect() in order to attempt to have its unavailable presence
+        // sent as soon as possible.
         this.connection.flush();
 
         if (ev !== null && typeof ev !== 'undefined') {
@@ -372,9 +374,9 @@ export default class XMPP extends Listenable {
             if (evType == 'beforeunload' || evType == 'unload') {
                 // XXX Whatever we said above, synchronous sending is the best
                 // (known) way to properly disconnect from the XMPP server.
-                // Consequently, it may be fine to have the source code and comment
-                // it in or out depending on whether we want to run with it for some
-                // time.
+                // Consequently, it may be fine to have the source code and
+                // comment it in or out depending on whether we want to run with
+                // it for some time.
                 this.connection.options.sync = true;
             }
         }

service/RTC/CameraFacingMode.js

@@ -1,9 +1,9 @@
-/* global module */
 /**
  * The possible camera facing modes. For now support only 'user' and
  * 'environment' because 'left' and 'right' are not used anywhere in our
  * projects at the time of this writing. For more information please refer to
- * https://w3c.github.io/mediacapture-main/getusermedia.html#def-constraint-facingMode
+ * https://w3c.github.io/mediacapture-main/getusermedia.html
+ * #def-constraint-facingMode.
  *
  * @enum {string}
  */