[eslint] spaced-comment

.eslintrc.js

@@ -158,6 +158,7 @@ module.exports = {
         'quote-props': 0,
         'semi': [ 'error', 'always' ],
         'space-unary-ops': 2,
+        'spaced-comment': 2,
         'unicode-bom': 0,
         'wrap-regex': 0,
 

JitsiConference.js

@@ -414,12 +414,12 @@ JitsiConference.prototype.setSubject = function (subject) {
 JitsiConference.prototype.getTranscriber = function(){
     if (this.transcriber === undefined){
         this.transcriber = new Transcriber();
-        //add all existing local audio tracks to the transcriber
+        // add all existing local audio tracks to the transcriber
         const localAudioTracks = this.getLocalTracks(MediaType.AUDIO);
         for (const localAudio of localAudioTracks) {
             this.transcriber.addTrack(localAudio);
         }
-        //and all remote audio tracks
+        // and all remote audio tracks
         const remoteAudioTracks = this.rtc.getRemoteTracks(MediaType.AUDIO);
         for (const remoteTrack of remoteAudioTracks){
             this.transcriber.addTrack(remoteTrack);

JitsiConferenceEventManager.js

@@ -20,7 +20,7 @@ const logger = getLogger(__filename);
 function JitsiConferenceEventManager(conference) {
     this.conference = conference;
 
-    //Listeners related to the conference only
+    // Listeners related to the conference only
     conference.on(JitsiConferenceEvents.TRACK_MUTE_CHANGED,
         function (track) {
             if(!track.isLocal() || !conference.statistics) {

JitsiParticipant.js

@@ -157,8 +157,8 @@ export default class JitsiParticipant {
     }
 
     // Gets a link to an etherpad instance advertised by the participant?
-    //getEtherpad() {
-    //}
+    // getEtherpad() {
+    // }
 
     /**
      * @returns {Boolean} Whether this participant has muted their audio.

modules/RTC/DataChannels.js

@@ -20,7 +20,7 @@ function DataChannels(peerConnection, emitter) {
     // 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 =
+    /* var dataChannelOptions =
      {
      reliable: true
      };
@@ -53,9 +53,9 @@ DataChannels.prototype.onDataChannel = function (event) {
 
         // Code sample for sending string and/or binary data
         // Sends String message to the bridge
-        //dataChannel.send("Hello bridge!");
+        // dataChannel.send("Hello bridge!");
         // Sends 12 bytes binary message to the bridge
-        //dataChannel.send(new ArrayBuffer(12));
+        // dataChannel.send(new ArrayBuffer(12));
 
         self.eventEmitter.emit(RTCEvents.DATA_CHANNEL_OPEN);
     };
@@ -64,7 +64,7 @@ DataChannels.prototype.onDataChannel = function (event) {
         // FIXME: this one seems to be generated a bit too often right now
         // so we are temporarily commenting it before we have more clarity
         // on which of the errors we absolutely need to report
-        //GlobalOnErrorHandler.callErrorHandler(
+        // GlobalOnErrorHandler.callErrorHandler(
         //        new Error("Data Channel Error:" + error));
         logger.error("Data Channel Error:", error, dataChannel);
     };

modules/RTC/JitsiLocalTrack.js

@@ -303,7 +303,7 @@ JitsiLocalTrack.prototype._setMute = function (mute) {
             this.dontFireRemoveEvent = true;
             promise = new Promise( (resolve, reject) => {
                 this._removeStreamFromConferenceAsMute(() => {
-                    //FIXME: Maybe here we should set the SRC for the containers
+                    // FIXME: Maybe here we should set the SRC for the containers
                     // to something
                     this._stopMediaStream();
                     this._setStream(null);
@@ -563,7 +563,7 @@ JitsiLocalTrack.prototype._setByteSent = function (bytesSent) {
     if(this._testByteSent && "connected" === iceConnectionState) {
         setTimeout(function () {
             if(this._bytesSent <= 0){
-                //we are not receiving anything from the microphone
+                // we are not receiving anything from the microphone
                 this._fireNoDataFromSourceEvent();
             }
         }.bind(this), 3000);

modules/RTC/JitsiTrack.js

@@ -12,7 +12,7 @@ const logger = getLogger(__filename);
  * Maps our handler types to MediaStreamTrack properties.
  */
 var trackHandler2Prop = {
-    "track_mute": "onmute",//Not supported on FF
+    "track_mute": "onmute",// Not supported on FF
     "track_unmute": "onunmute",
     "track_ended": "onended"
 };

modules/RTC/RTC.js

@@ -54,7 +54,7 @@ export default class RTC extends Listenable {
         this.peerConnectionIdCounter = 1;
 
         this.localTracks = [];
-        //FIXME: We should support multiple streams per jid.
+        // FIXME: We should support multiple streams per jid.
         this.remoteTracks = {};
         this.options = options;
         // A flag whether we had received that the data channel had opened

modules/RTC/RTCUtils.js

@@ -186,7 +186,7 @@ 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
+                options.minFps = options.minFps || options.fps; // Fall back to options.fps for backwards compatibility
                 constraints.video.mandatory.minFrameRate = options.minFps;
             }
             if (options.maxFps) {
@@ -285,7 +285,7 @@ function getConstraints(um, options) {
 
     if (options.bandwidth) {
         if (!constraints.video) {
-            //same behaviour as true
+            // same behaviour as true
             constraints.video = {mandatory: {}, optional: []};
         }
         constraints.video.optional.push({bandwidth: options.bandwidth});
@@ -297,7 +297,7 @@ function getConstraints(um, options) {
     if(RTCBrowserType.isFirefox() && options.firefox_fake_device) {
         // seems to be fixed now, removing this experimental fix, as having
         // multiple audio tracks brake the tests
-        //constraints.audio = true;
+        // constraints.audio = true;
         constraints.fake = true;
     }
 
@@ -728,7 +728,7 @@ function defaultSetVideoSrc(element, stream) {
     element.src = src || '';
 }
 
-//Options parameter is to pass config options. Currently uses only "useIPv6".
+// Options parameter is to pass config options. Currently uses only "useIPv6".
 class RTCUtils extends Listenable {
     constructor() {
         super(eventEmitter);

modules/RTC/TraceablePeerConnection.js

@@ -89,7 +89,7 @@ function TraceablePeerConnection(rtc, id, signalingLayer, ice_config,
 
     // override as desired
     this.trace = function (what, info) {
-        /*logger.warn('WTRACE', what, info);
+        /* logger.warn('WTRACE', what, info);
         if (info && RTCBrowserType.isIExplorer()) {
             if (info.length > 1024) {
                 logger.warn('WTRACE', what, info.substr(1024));
@@ -890,7 +890,7 @@ TraceablePeerConnection.prototype.createAnswer
 TraceablePeerConnection.prototype.addIceCandidate
         // eslint-disable-next-line no-unused-vars
         = function (candidate, successCallback, failureCallback) {
-    //var self = this;
+    // var self = this;
             this.trace('addIceCandidate', JSON.stringify(candidate, null, ' '));
             this.peerconnection.addIceCandidate(candidate);
     /* maybe later

modules/connectivity/ConnectionQuality.js

@@ -371,9 +371,9 @@ export default class ConnectionQuality {
             // send the statistics again after a few seconds, and the error is
             // already logged elsewhere. So just ignore it.
 
-            //let errorMsg = "Failed to broadcast local stats";
-            //logger.error(errorMsg, e);
-            //GlobalOnErrorHandler.callErrorHandler(
+            // let errorMsg = "Failed to broadcast local stats";
+            // logger.error(errorMsg, e);
+            // GlobalOnErrorHandler.callErrorHandler(
             //    new Error(errorMsg + ": " + e));
         }
     }

modules/statistics/CallStats.js

@@ -159,7 +159,7 @@ var CallStats = _try_catch(function(jingleSession, options) {
         this.callStatsSecret = options.callStatsSecret;
 
         CallStats.initializeInProgress = true;
-        //userID is generated or given by the origin server
+        // userID is generated or given by the origin server
         callStats.initialize(this.callStatsID,
             this.callStatsSecret,
             this.userID,

modules/statistics/RTPStatsCollector.js

@@ -276,10 +276,10 @@ StatsCollector.prototype.start = function (startAudioLevelStats) {
                         var results = null;
                         if (!report || !report.result ||
                             typeof report.result != 'function') {
-                            //firefox
+                            // firefox
                             results = report;
                         } else {
-                            //chrome
+                            // chrome
                             results = report.result();
                         }
                         self.currentStatsReport = results;
@@ -406,7 +406,7 @@ StatsCollector.prototype.processStatsReport = function () {
                     "upload": Math.round(sendBandwidth / 1000)
                 };
             }
-        } catch(e){/*not supported*/}
+        } catch(e){/* not supported*/}
 
         if(now.type == 'googCandidatePair') {
             var ip, type, localip, active;
@@ -415,7 +415,7 @@ StatsCollector.prototype.processStatsReport = function () {
                 type = getStatValue(now, "transportType");
                 localip = getStatValue(now, "localAddress");
                 active = getStatValue(now, "activeConnection");
-            } catch(e){/*not supported*/}
+            } catch(e){/* not supported*/}
             if(!ip || !type || !localip || active != "true") {
                 continue;
             }
@@ -536,7 +536,7 @@ StatsCollector.prototype.processStatsReport = function () {
                 resolution.height = height;
                 resolution.width = width;
             }
-        } catch(e){/*not supported*/}
+        } catch(e){/* not supported*/}
 
         if (resolution.height && resolution.width) {
             ssrcStats.setResolution(resolution);
@@ -638,7 +638,7 @@ StatsCollector.prototype.processAudioLevelReport = function () {
             var audioLevel
                 = getStatValue(now, 'audioInputLevel')
                     || getStatValue(now, 'audioOutputLevel');
-        } catch(e) {/*not supported*/
+        } catch(e) {/* not supported*/
             logger.warn("Audio Levels are not available in the statistics.");
             clearInterval(this.audioLevelsIntervalId);
             return;

modules/statistics/statistics.js

@@ -208,7 +208,7 @@ Statistics.prototype.stopRemoteStats = function () {
     this.rtpStats = null;
 };
 
-//CALSTATS METHODS
+// CALSTATS METHODS
 
 /**
  * Initializes the callstats.io API.

modules/transcription/audioRecorder.js

@@ -21,10 +21,10 @@ var TrackRecorder = function(track){
     // The array of data chunks recorded from the stream
     // acts as a buffer until the data is stored on disk
     this.data = null;
-    //the name of the person of the JitsiTrack. This can be undefined and/or
-    //not unique
+    // the name of the person of the JitsiTrack. This can be undefined and/or
+    // not unique
     this.name = null;
-    //the time of the start of the recording
+    // the time of the start of the recording
     this.startTime = null;
 };
 
@@ -71,8 +71,8 @@ function instantiateTrackRecorder(track) {
     // Create the MediaRecorder
     trackRecorder.recorder = new MediaRecorder(stream,
         {mimeType: audioRecorder.fileType});
-    //array for holding the recorder data. Resets it when
-    //audio already has been recorder once
+    // array for holding the recorder data. Resets it when
+    // audio already has been recorder once
     trackRecorder.data = [];
     // function handling a dataEvent, e.g the stream gets new data
     trackRecorder.recorder.ondataavailable = function (dataEvent) {
@@ -110,13 +110,13 @@ var audioRecorder = function(jitsiConference){
     // holds the JitsiTrack, MediaRecorder and recorder data
     this.recorders = [];
 
-    //get which file type is supported by the current browser
+    // get which file type is supported by the current browser
     this.fileType = determineCorrectFileType();
 
-    //boolean flag for active recording
+    // boolean flag for active recording
     this.isRecording = false;
 
-    //the jitsiconference the object is recording
+    // the jitsiconference the object is recording
     this.jitsiConference = jitsiConference;
 };
 
@@ -132,13 +132,14 @@ audioRecorder.determineCorrectFileType = determineCorrectFileType;
  */
 audioRecorder.prototype.addTrack = function (track) {
     if(track.isAudioTrack()) {
-        //create the track recorder
+        // create the track recorder
         var trackRecorder = instantiateTrackRecorder(track);
-        //push it to the local array of all recorders
+        // push it to the local array of all recorders
         this.recorders.push(trackRecorder);
-        //update the name of the trackRecorders
+        // update the name of the trackRecorders
         this.updateNames();
-        //if we're already recording, immediately start recording this new track
+        // If we're already recording, immediately start recording this new
+        // track.
         if(this.isRecording){
             startRecorder(trackRecorder);
         }
@@ -159,7 +160,7 @@ audioRecorder.prototype.removeTrack = function(track){
     if(track.isVideoTrack()){
         return;
     }
-    
+
     var array = this.recorders;
     var i;
     for(i = 0; i < array.length; i++) {
@@ -168,13 +169,13 @@ audioRecorder.prototype.removeTrack = function(track){
             if(this.isRecording){
                 stopRecorder(recorderToRemove);
             } else {
-                //remove the TrackRecorder from the array
+                // remove the TrackRecorder from the array
                 array.splice(i, 1);
             }
         }
     }
 
-    //make sure the names are up to date
+    // make sure the names are up to date
     this.updateNames();
 };
 
@@ -209,11 +210,11 @@ audioRecorder.prototype.start = function () {
     // set boolean isRecording flag to true so if new participants join the
     // conference, that track can instantly start recording as well
     this.isRecording = true;
-    //start all the mediaRecorders
+    // start all the mediaRecorders
     this.recorders.forEach(function(trackRecorder){
         startRecorder(trackRecorder);
     });
-    //log that recording has started
+    // log that recording has started
     console.log("Started the recording of the audio. There are currently " +
         this.recorders.length + " recorders active.");
 };
@@ -222,9 +223,9 @@ audioRecorder.prototype.start = function () {
  * Stops the audio recording of every local and remote track
  */
 audioRecorder.prototype.stop = function() {
-    //set the boolean flag to false
+    // set the boolean flag to false
     this.isRecording = false;
-    //stop all recorders
+    // stop all recorders
     this.recorders.forEach(function(trackRecorder){
         stopRecorder(trackRecorder);
     });
@@ -259,7 +260,7 @@ audioRecorder.prototype.getRecordingResults = function () {
         throw new Error("cannot get blobs because the AudioRecorder is still" +
             "recording!");
     }
-    //make sure the names are up to date before sending them off
+    // make sure the names are up to date before sending them off
     this.updateNames();
 
     var array = [];

modules/transcription/transcriber.js

@@ -7,7 +7,7 @@ var RECORDING_STATE = "recording";
 var TRANSCRIBING_STATE = "transcribing";
 var FINISHED_STATE = "finished";
 
-//the amount of characters each line in the transcription will have
+// the amount of characters each line in the transcription will have
 var MAXIMUM_SENTENCE_LENGTH = 80;
 
 /**
@@ -18,27 +18,27 @@ var MAXIMUM_SENTENCE_LENGTH = 80;
  * @param {AudioRecorder} audioRecorder An audioRecorder recording a conference
  */
 var transcriber = function() {
-    //the object which can record all audio in the conference
+    // the object which can record all audio in the conference
     this.audioRecorder = new AudioRecorder();
-    //this object can send the recorder audio to a speech-to-text service
+    // this object can send the recorder audio to a speech-to-text service
     this.transcriptionService = new SphinxService();
-    //holds a counter to keep track if merging can start
+    // holds a counter to keep track if merging can start
     this.counter = null;
-    //holds the date when transcription started which makes it possible
-    //to calculate the offset between recordings
+    // holds the date when transcription started which makes it possible
+    // to calculate the offset between recordings
     this.startTime = null;
-    //will hold the transcription once it is completed
+    // will hold the transcription once it is completed
     this.transcription = null;
-    //this will be a method which will be called once the transcription is done
-    //with the transcription as parameter
+    // this will be a method which will be called once the transcription is done
+    // with the transcription as parameter
     this.callback = null;
-    //stores all the retrieved speech-to-text results to merge together
-    //this value will store an Array<Word> object
+    // stores all the retrieved speech-to-text results to merge together
+    // this value will store an Array<Word> object
     this.results = [];
     // Stores the current state of the transcription process
     this.state = BEFORE_STATE;
-    //Used in the updateTranscription method to add a new line when the
-    //sentence becomes to long
+    // Used in the updateTranscription method to add a new line when the
+    // sentence becomes to long
     this.lineLength = 0;
 };
 
@@ -70,10 +70,10 @@ transcriber.prototype.stop = function stop(callback) {
             "\"" + RECORDING_STATE + "\" state. It's currently in the " +
             "\"" + this.state + "\" state");
     }
-    //stop the recording
+    // stop the recording
     console.log("stopping recording and sending audio files");
     this.audioRecorder.stop();
-    //and send all recorded audio the the transcription service
+    // and send all recorded audio the the transcription service
     var t = this;
 
     var callBack = blobCallBack.bind(this);
@@ -81,9 +81,9 @@ transcriber.prototype.stop = function stop(callback) {
         t.transcriptionService.send(recordingResult, callBack);
         t.counter++;
     });
-    //set the state to "transcribing" so that maybeMerge() functions correctly
+    // set the state to "transcribing" so that maybeMerge() functions correctly
     this.state = TRANSCRIBING_STATE;
-    //and store the callback for later
+    // and store the callback for later
     this.callback = callback;
 };
 
@@ -100,14 +100,14 @@ transcriber.prototype.stop = function stop(callback) {
 var blobCallBack = function(answer){
     console.log("retrieved an answer from the transcription service. The" +
         " answer has an array of length: " + answer.wordArray.length);
-    //first add the offset between the start of the transcription and
-    //the start of the recording to all start and end times
+    // first add the offset between the start of the transcription and
+    // the start of the recording to all start and end times
     if(answer.wordArray.length > 0) {
         var offset = answer.startTime.getUTCMilliseconds() -
             this.startTime.getUTCMilliseconds();
-        //transcriber time will always be earlier
+        // transcriber time will always be earlier
         if (offset < 0) {
-            offset = 0; //presume 0 if it somehow not earlier
+            offset = 0; // presume 0 if it somehow not earlier
         }
 
         var array = "[";
@@ -118,16 +118,16 @@ var blobCallBack = function(answer){
         });
         array += "]";
         console.log(array);
-        //give a name value to the Array object so that the merging can access
-        //the name value without having to use the whole recordingResult object
-        //in the algorithm
+        // give a name value to the Array object so that the merging can access
+        // the name value without having to use the whole recordingResult object
+        // in the algorithm
         answer.wordArray.name = answer.name;
     }
-    //then store the array and decrease the counter
+    // then store the array and decrease the counter
     this.results.push(answer.wordArray);
     this.counter--;
     console.log("current counter: " + this.counter);
-    //and check if all results have been received.
+    // and check if all results have been received.
     this.maybeMerge();
 };
 
@@ -137,8 +137,8 @@ var blobCallBack = function(answer){
  */
 transcriber.prototype.maybeMerge = function(){
     if(this.state === TRANSCRIBING_STATE && this.counter === 0){
-        //make sure to include the events in the result arrays before
-        //merging starts
+        // make sure to include the events in the result arrays before
+        // merging starts
         this.merge();
     }
 };
@@ -151,47 +151,47 @@ transcriber.prototype.merge = function() {
     console.log("starting merge process!\n The length of the array: " +
         this.results.length);
     this.transcription = "";
-    //the merging algorithm will look over all Word objects who are at pos 0 in
-    //every array. It will then select the one closest in time to the
-    //previously placed word, while removing the selected word from its array
-    //note: words can be skipped the skipped word's begin and end time somehow
-    //end up between the closest word start and end time
+    // the merging algorithm will look over all Word objects who are at pos 0 in
+    // every array. It will then select the one closest in time to the
+    // previously placed word, while removing the selected word from its array
+    // note: words can be skipped the skipped word's begin and end time somehow
+    // end up between the closest word start and end time
     var arrays = this.results;
-    //arrays of Word objects
-    var potentialWords = []; //array of the first Word objects
-    //check if any arrays are already empty and remove them
+    // arrays of Word objects
+    var potentialWords = []; // array of the first Word objects
+    // check if any arrays are already empty and remove them
     hasPopulatedArrays(arrays);
 
-    //populate all the potential Words for a first time
+    // populate all the potential Words for a first time
     arrays.forEach(function (array){
         pushWordToSortedArray(potentialWords, array);
     });
 
-    //keep adding words to transcription until all arrays are exhausted
+    // keep adding words to transcription until all arrays are exhausted
     var lowestWordArray;
     var wordToAdd;
     var foundSmaller;
     while(hasPopulatedArrays(arrays)){
-        //first select the lowest array;
+        // first select the lowest array;
         lowestWordArray = arrays[0];
         arrays.forEach(function(wordArray){
             if(wordArray[0].begin < lowestWordArray[0].begin){
                 lowestWordArray = wordArray;
             }
         });
-        //put the word in the transcription
+        // put the word in the transcription
         wordToAdd = lowestWordArray.shift();
         this.updateTranscription(wordToAdd,lowestWordArray.name);
 
-        //keep going until a word in another array has a smaller time
-        //or the array is empty
+        // keep going until a word in another array has a smaller time
+        // or the array is empty
         while(!foundSmaller && lowestWordArray.length > 0){
             arrays.forEach(function(wordArray){
                 if(wordArray[0].begin < lowestWordArray[0].begin){
                     foundSmaller = true;
                 }
             });
-            //add next word if no smaller time has been found
+            // add next word if no smaller time has been found
             if(!foundSmaller){
                 wordToAdd = lowestWordArray.shift();
                 this.updateTranscription(wordToAdd, null);
@@ -200,7 +200,7 @@ transcriber.prototype.merge = function() {
 
     }
 
-    //set the state to finished and do the necessary left-over tasks
+    // set the state to finished and do the necessary left-over tasks
     this.state = FINISHED_STATE;
     if(this.callback){
         this.callback(this.transcription);
@@ -216,14 +216,14 @@ transcriber.prototype.merge = function() {
 transcriber.prototype.updateTranscription = function(word, name){
     if(name !== undefined && name !== null){
         this.transcription += "\n" + name + ":";
-        this.lineLength = name.length + 1; //+1 for the semi-colon
+        this.lineLength = name.length + 1; // +1 for the semi-colon
     }
     if(this.lineLength + word.word.length > MAXIMUM_SENTENCE_LENGTH){
         this.transcription += "\n    ";
-        this.lineLength = 4; //because of the 4 spaces after the new line
+        this.lineLength = 4; // because of the 4 spaces after the new line
     }
     this.transcription += " " + word.word;
-    this.lineLength += word.word.length + 1; //+1 for the space
+    this.lineLength += word.word.length + 1; // +1 for the space
 };
 
 /**
@@ -267,7 +267,7 @@ var pushWordToSortedArray = function(array, word){
                 return;
             }
         }
-        array.push(word); //fail safe
+        array.push(word); // fail safe
     }
 };
 

modules/transcription/transcriptionServices/SphinxTranscriptionService.js

@@ -8,7 +8,7 @@ var audioRecorder = require("./../audioRecorder");
  * Implements a TranscriptionService for a Sphinx4 http server
  */
 var SphinxService = function() {
-    //set the correct url
+    // set the correct url
     this.url = getURL();
 };
 
@@ -41,7 +41,7 @@ SphinxService.prototype.sendRequest = function(audioFileBlob, callback) {
             throw new Error("unable to accept response from sphinx server." +
                 "status: " + request.status);
         }
-        //if not ready no point to throw an error
+        // if not ready no point to throw an error
     };
     request.open("POST", this.url);
     request.setRequestHeader("Content-Type",
@@ -58,8 +58,8 @@ SphinxService.prototype.sendRequest = function(audioFileBlob, callback) {
  */
 SphinxService.prototype.formatResponse = function(response) {
     var result = JSON.parse(response).objects;
-    //make sure to delete the session id object, which is always
-    //the first value in the JSON array
+    // make sure to delete the session id object, which is always
+    // the first value in the JSON array
     result.shift();
     var array = [];
     result.forEach(function(word){
@@ -77,11 +77,11 @@ SphinxService.prototype.formatResponse = function(response) {
  */
 SphinxService.prototype.verify = function(response){
     console.log("response from server:" + response.toString());
-    //test if server responded with a string object
+    // test if server responded with a string object
     if(typeof response !== "string"){
         return false;
     }
-    //test if the string can be parsed into valid JSON
+    // test if the string can be parsed into valid JSON
     var json;
     try{
         json = JSON.parse(response);
@@ -89,16 +89,16 @@ SphinxService.prototype.verify = function(response){
         console.log(error);
         return false;
     }
-    //check if the JSON has a "objects" value
+    // check if the JSON has a "objects" value
     if(json.objects === undefined){
         return false;
     }
-    //get the "objects" value and check for a session ID
+    // get the "objects" value and check for a session ID
     var array = json.objects;
     if(!(array[0] && array[0]["session-id"])){
         return false;
     }
-    //everything seems to be in order
+    // everything seems to be in order
     return true;
 };
 

modules/util/EventEmitterForwarder.js

@@ -26,9 +26,9 @@ EventEmitterForwarder.prototype.forward = function () {
     // This line is only for fixing jshint errors.
     var args = arguments;
     var srcEvent = args[0];
-    //This will be the "this" value for emit function.
+    // This will be the "this" value for emit function.
     args[0] = this.dest;
-    //Using bind.apply to pass the arguments as Array-like object ("arguments")
+    // Using bind.apply to pass the arguments as Array-like object ("arguments")
     this.src.addListener(srcEvent,
         Function.prototype.bind.apply(this.dest.emit, args));
 };

modules/xmpp/Caps.js

@@ -164,7 +164,7 @@ export default class Caps extends Listenable {
      * Handles this.version changes.
      */
     _notifyVersionChanged() {
-        //update the version for all rooms
+        // update the version for all rooms
         this.rooms.forEach(room => this._fixChatRoomPresenceMap(room));
         this.submit();
     }

modules/xmpp/ChatRoom.js

@@ -628,7 +628,7 @@ export default class ChatRoom extends Listenable {
     }
 
     lockRoom (key, onSuccess, onError, onNotSupported) {
-        //http://xmpp.org/extensions/xep-0045.html#roomconfig
+        // http://xmpp.org/extensions/xep-0045.html#roomconfig
         var ob = this;
         this.connection.sendIQ($iq({to: this.roomjid, type: 'get'}).c('query', {xmlns: 'http://jabber.org/protocol/muc#owner'}),
             function (res) {

modules/xmpp/JingleSessionPC.js

@@ -310,7 +310,7 @@ export default class JingleSessionPC extends JingleSession {
         }
         // might merge last-candidate notification into this, but it is called
         // a lot later. See webrtc issue #2340
-        //logger.log('was this the last candidate', this.lasticecandidate);
+        // logger.log('was this the last candidate', this.lasticecandidate);
         this.connection.sendIQ(
             cand, null, this.newJingleErrorHandler(cand, function (error) {
                 GlobalOnErrorHandler.callErrorHandler(
@@ -620,8 +620,8 @@ export default class JingleSessionPC extends JingleSession {
         this.state = 'ended';
 
         // Do something with reason and reasonCondition when we start to care
-        //this.reasonCondition = reasonCondition;
-        //this.reasonText = reasonText;
+        // this.reasonCondition = reasonCondition;
+        // this.reasonText = reasonText;
         logger.info("Session terminated", this, reasonCondition, reasonText);
 
         this.close();
@@ -867,7 +867,7 @@ export default class JingleSessionPC extends JingleSession {
             sdp: remoteSdp.raw
         });
 
-        //TODO(brian): in the code below there are 2 chunks of code that relate
+        // TODO(brian): in the code below there are 2 chunks of code that relate
         //  to observing changes in local and remove ufrags.  since they
         //  just need to read and observe the SDPs, we should create the
         //  notion of an SDP observer in TraceablePeerConnection that
@@ -1137,7 +1137,7 @@ export default class JingleSessionPC extends JingleSession {
      * @param stream: webrtc media stream
      */
     _handleFirefoxRemoveStream(stream) {
-        if (!stream) { //There is nothing to be changed
+        if (!stream) { // There is nothing to be changed
             return;
         }
         let sender = null;
@@ -1548,7 +1548,7 @@ export default class JingleSessionPC extends JingleSession {
                     const sourceNode
                         = desc.find(">source[ssrc=\"" + ssrc + "\"]");
                     if (!sourceNode || !sourceNode.length) {
-                        //Maybe we have to include cname, msid, etc here?
+                        // Maybe we have to include cname, msid, etc here?
                         desc.append(
                             "<source " +
                             "xmlns=\"urn:xmpp:jingle:apps:rtp:ssma:0\" " +

modules/xmpp/RtxModifier.spec.js

@@ -1,5 +1,5 @@
-/*eslint-disable max-len*/
-/*jshint maxlen:false*/
+/* eslint-disable max-len*/
+/* jshint maxlen:false*/
 import RtxModifier from "./RtxModifier.js";
 import * as SampleSdpStrings from "./SampleSdpStrings.js";
 import * as transform from 'sdp-transform';
@@ -307,4 +307,4 @@ describe ("RtxModifier", function() {
     });
 });
 
-/*eslint-enable max-len*/
+/* eslint-enable max-len*/

modules/xmpp/SDPUtil.js

@@ -326,7 +326,7 @@ var SDPUtil = {
             protocol = 'tcp';
         }
 
-        line += protocol; //.toUpperCase(); // chrome M23 doesn't like this
+        line += protocol; // .toUpperCase(); // chrome M23 doesn't like this
         line += ' ';
         line += cand.getAttribute('priority');
         line += ' ';

modules/xmpp/SampleSdpStrings.js

@@ -1,5 +1,5 @@
-/*eslint-disable max-len*/
-/*jshint maxlen:false*/
+/* eslint-disable max-len*/
+/* jshint maxlen:false*/
 import * as transform from 'sdp-transform';
 
 // A generic sdp session block
@@ -199,4 +199,4 @@ export const plainVideoSdp = transform.parse(plainVideoSdpStr);
 export const rtxVideoSdp = transform.parse(rtxVideoSdpStr);
 export const multiCodecVideoSdp = transform.parse(multiCodecVideoSdpStr);
 
-/*eslint-enable max-len*/
+/* eslint-enable max-len*/

modules/xmpp/moderator.js

@@ -41,7 +41,7 @@ function Moderator(roomName, xmpp, emitter, options) {
 
     this.connection = this.xmppService.connection;
     this.focusUserJid;
-    //FIXME:
+    // FIXME:
     // Message listener that talks to POPUP window
     function listener(event) {
         if (event.data && event.data.sessionId) {
@@ -329,7 +329,8 @@ Moderator.prototype._allocateConferenceFocusError = function (error, callback) {
         logger.warn("Unauthorized to start the conference", error);
         var toDomain = Strophe.getDomainFromJid(error.getAttribute('to'));
         if (toDomain !== this.options.connection.hosts.anonymousdomain) {
-            //FIXME "is external" should come either from the focus or config.js
+            // FIXME "is external" should come either from the focus or
+            // config.js
             this.externalAuthEnabled = true;
         }
         this.eventEmitter.emit(XMPPEvents.AUTHENTICATION_REQUIRED);
@@ -342,7 +343,7 @@ Moderator.prototype._allocateConferenceFocusError = function (error, callback) {
     // Show message
     var focusComponent = this.getFocusComponent();
     var retrySec = waitMs / 1000;
-    //FIXME: message is duplicated ? Do not show in case of session invalid
+    // FIXME: message is duplicated ? Do not show in case of session invalid
     // which means just a retry
     if (!invalidSession) {
         this.eventEmitter.emit(

modules/xmpp/strophe.ping.js

@@ -106,7 +106,7 @@ class PingConnectionPlugin extends ConnectionPlugin {
                     // Not really sure what's the right thing to do in that
                     // situation, but just closing the connection makes no
                     // sense.
-                    //self.connection.disconnect();
+                    // self.connection.disconnect();
                 } else {
                     logger.warn(errmsg, error);
                 }

modules/xmpp/xmpp.js

@@ -72,14 +72,14 @@ export default class XMPP extends Listenable {
 
         // this is dealt with by SDP O/A so we don't need to announce this
         // XEP-0293
-        //this.caps.addFeature('urn:xmpp:jingle:apps:rtp:rtcp-fb:0');
+        // this.caps.addFeature('urn:xmpp:jingle:apps:rtp:rtcp-fb:0');
         // XEP-0294
-        //this.caps.addFeature('urn:xmpp:jingle:apps:rtp:rtp-hdrext:0');
+        // this.caps.addFeature('urn:xmpp:jingle:apps:rtp:rtp-hdrext:0');
 
         this.caps.addFeature('urn:ietf:rfc:5761'); // rtcp-mux
         this.caps.addFeature('urn:ietf:rfc:5888'); // a=group, e.g. bundle
 
-        //this.caps.addFeature('urn:ietf:rfc:5576'); // a=ssrc
+        // this.caps.addFeature('urn:ietf:rfc:5576'); // a=ssrc
 
         // Enable Lipsync ?
         if (RTCBrowserType.isChrome() && false !== this.options.enableLipSync) {