[eslint] array-bracket-spacing

.eslintrc.js

@@ -148,6 +148,11 @@ module.exports = {
         'no-unused-vars': 2,
 
         // Stylistic issues group
+        'array-bracket-spacing': [
+            'error',
+            'always',
+            { 'objectsInArrays': true }
+        ],
         'brace-style': 2,
         'comma-style': 2,
         'eol-last': 2,

doc/example/example.js

@@ -193,7 +193,7 @@ 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,
@@ -226,7 +226,7 @@ const initOptions = {
     desktopSharingChromeDisabled: false,
     // The media sources to use when using screen sharing with the Chrome
     // extension.
-    desktopSharingChromeSources: ['screen', 'window'],
+    desktopSharingChromeSources: [ 'screen', 'window' ],
     // Required version of Chrome extension
     desktopSharingChromeMinExtVersion: '0.1',
 
@@ -264,7 +264,7 @@ JitsiMeetJS.init(initOptions)
             onDeviceListChanged);
 
         connection.connect();
-        JitsiMeetJS.createLocalTracks({devices: ['audio', 'video']})
+        JitsiMeetJS.createLocalTracks({devices: [ 'audio', 'video' ]})
             .then(onLocalTracks)
             .catch(error => {
                 throw error;

karma.conf.js

@@ -10,7 +10,7 @@ module.exports = function(config) {
 
     // frameworks to use
     // available frameworks: https://npmjs.org/browse/keyword/karma-adapter
-        frameworks: ['jasmine'],
+        frameworks: [ 'jasmine' ],
 
 
     // list of files / patterns to load in the browser
@@ -29,15 +29,15 @@ module.exports = function(config) {
     // available preprocessors:
     //  https://npmjs.org/browse/keyword/karma-preprocessor
         preprocessors: {
-            './JitsiMeetJS.js': ['webpack'],
-            './**/*.spec.js': ['webpack']
+            './JitsiMeetJS.js': [ 'webpack' ],
+            './**/*.spec.js': [ 'webpack' ]
         },
 
 
     // test results reporter to use
     // possible values: 'dots', 'progress'
     // available reporters: https://npmjs.org/browse/keyword/karma-reporter
-        reporters: ['progress'],
+        reporters: [ 'progress' ],
 
 
     // web server port
@@ -62,7 +62,7 @@ module.exports = function(config) {
     // start these browsers
     // available browser launchers:
     // https://npmjs.org/browse/keyword/karma-launcher
-        browsers: ['Chrome'],
+        browsers: [ 'Chrome' ],
 
 
     // Continuous Integration mode

modules/RTC/RTCUtils.js

@@ -206,9 +206,9 @@ function getConstraints(um, options) {
                 constraints.audio = {
                     mandatory: {},
                     deviceId: options.micDeviceId, // new style
-                    optional: [{
+                    optional: [ {
                         sourceId: options.micDeviceId // old style
-                    }]};
+                    } ]};
             } else {
                 constraints.audio = true;
             }
@@ -467,14 +467,14 @@ const getUserMediaStatus = {
 function wrapGetUserMedia(getUserMedia) {
     return function(constraints, successCallback, errorCallback) {
         getUserMedia(constraints, stream => {
-            maybeApply(successCallback, [stream]);
+            maybeApply(successCallback, [ stream ]);
             if (!getUserMediaStatus.initialized) {
                 getUserMediaStatus.initialized = true;
                 getUserMediaStatus.callbacks.forEach(callback => callback());
                 getUserMediaStatus.callbacks.length = 0;
             }
         }, error => {
-            maybeApply(errorCallback, [error]);
+            maybeApply(errorCallback, [ error ]);
         });
     };
 }
@@ -668,7 +668,7 @@ 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, reason: err});
@@ -1015,7 +1015,7 @@ class RTCUtils extends Listenable {
                 resolve(handleLocalStream(stream, options.resolution));
             };
 
-            options.devices = options.devices || ['audio', 'video'];
+            options.devices = options.devices || [ 'audio', 'video' ];
             if(!screenObtainer.isSupported()
                 && options.devices.indexOf('desktop') !== -1) {
                 reject(new Error('Desktop sharing is not supported!'));
@@ -1034,8 +1034,8 @@ class RTCUtils extends Listenable {
                 };
 
                 const deviceGUM = {
-                    'audio': GUM.bind(self, ['audio']),
-                    'video': GUM.bind(self, ['video'])
+                    'audio': GUM.bind(self, [ 'audio' ]),
+                    'video': GUM.bind(self, [ 'video' ])
                 };
 
                 if(screenObtainer.isSupported()) {

modules/RTC/ScreenObtainer.js

@@ -107,7 +107,7 @@ const ScreenObtainer = {
                             );
                         } else {
                             jitsiError = new JitsiTrackError(
-                                error, constraints, ['desktop']);
+                                error, constraints, [ 'desktop' ]);
                         }
                         (typeof onFailure === 'function')
                             && onFailure(jitsiError);
@@ -341,7 +341,7 @@ const ScreenObtainer = {
  * 'about:config'.
  */
 function obtainWebRTCScreen(options, streamCallback, failCallback) {
-    gumFunction(['screen'], streamCallback, failCallback);
+    gumFunction([ 'screen' ], streamCallback, failCallback);
 }
 
 /**
@@ -531,7 +531,7 @@ function waitForExtensionAfterInstall(options, waitInterval, retries) {
 function onGetStreamResponse(response, onSuccess, onFailure) {
     if (response.streamId) {
         gumFunction(
-            ['desktop'],
+            [ 'desktop' ],
             stream => onSuccess(stream),
             onFailure,
             { desktopStream: response.streamId });

modules/RTC/TraceablePeerConnection.js

@@ -988,7 +988,7 @@ TraceablePeerConnection.prototype.generateNewStreamSSRCInfo = function() {
         });
     } else {
         ssrcInfo = {
-            ssrcs: [SDPUtil.generateSsrc()],
+            ssrcs: [ SDPUtil.generateSsrc() ],
             groups: []
         };
     }
@@ -1005,7 +1005,7 @@ TraceablePeerConnection.prototype.generateNewStreamSSRCInfo = function() {
 
             ssrcInfo.ssrcs.push(rtxSsrc);
             ssrcInfo.groups.push({
-                ssrcs: [primarySsrc, rtxSsrc],
+                ssrcs: [ primarySsrc, rtxSsrc ],
                 semantics: 'FID'
             });
         }

modules/xmpp/Caps.js

@@ -5,8 +5,8 @@ import Listenable from '../util/Listenable';
 /**
  * The property
  */
-const IDENTITY_PROPERTIES = ['category', 'type', 'lang', 'name'];
-const IDENTITY_PROPERTIES_FOR_COMPARE = ['category', 'type', 'lang'];
+const IDENTITY_PROPERTIES = [ 'category', 'type', 'lang', 'name' ];
+const IDENTITY_PROPERTIES_FOR_COMPARE = [ 'category', 'type', 'lang' ];
 const HASH = 'sha-1';
 
 function compareIdentities(a, b) {

modules/xmpp/JingleSessionPC.js

@@ -256,7 +256,7 @@ export default class JingleSessionPC extends JingleSession {
                 }
                 this.drip_container.push(candidate);
             } else {
-                this.sendIceCandidates([candidate]);
+                this.sendIceCandidates([ candidate ]);
             }
         } else {
             logger.log('sendIceCandidate: last candidate.');

modules/xmpp/RtxModifier.spec.js

@@ -44,7 +44,7 @@ function getPrimaryVideoSsrcs(parsedSdp) {
     const videoMLine = parsedSdp.media.find(m => m.type === 'video');
 
     if (numVideoSsrcs(parsedSdp) === 1) {
-        return [videoMLine.ssrcs[0].id];
+        return [ videoMLine.ssrcs[0].id ];
     }
     const simGroups = getVideoGroups(parsedSdp, 'SIM');
 

modules/xmpp/SDPDiffer.js

@@ -82,7 +82,7 @@ SDPDiffer.prototype.getNewMedia = function() {
 
                 if (otherSsrcGroup.semantics == mySsrcGroup.semantics
                     && arrayEquals.apply(otherSsrcGroup.ssrcs,
-                                      [mySsrcGroup.ssrcs])) {
+                                      [ mySsrcGroup.ssrcs ])) {
 
                     matched = true;
                     break;

modules/xmpp/SDPUtil.js

@@ -82,7 +82,7 @@ const SDPUtil = {
         const streamCount = parts.length > 2 ? parts[2] : null;
 
 
-        return [sctpPort, protocol, streamCount];// SCTP port
+        return [ sctpPort, protocol, streamCount ];// SCTP port
     },
     build_rtpmap(el) {
         let line = `a=rtpmap:${el.getAttribute('id')} ${el.getAttribute('name')}/${el.getAttribute('clockrate')}`;
@@ -180,7 +180,7 @@ 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) {

modules/xmpp/strophe.logger.js

@@ -17,11 +17,11 @@ class StropheLogger extends ConnectionPlugin {
     }
 
     log_incoming(stanza) {
-        this.log.push([new Date().getTime(), 'incoming', stanza]);
+        this.log.push([ new Date().getTime(), 'incoming', stanza ]);
     }
 
     log_outgoing(stanza) {
-        this.log.push([new Date().getTime(), 'outgoing', stanza]);
+        this.log.push([ new Date().getTime(), 'outgoing', stanza ]);
     }
 }