bug 1523104: remote: fix eslint by removing unecessary commas; r=ato
authorAlexandre Poirot <poirot.alex@gmail.com>
Tue, 12 Feb 2019 06:44:54 -0800
changeset 521078 d7e26a1b6ef7
parent 521077 ac7c2e018cb6
child 521079 819bb544204d
push id10862
push userffxbld-merge
push dateMon, 11 Mar 2019 13:01:11 +0000
treeherdermozilla-beta@a2e7f5c935da [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersato
bugs1523104
milestone67.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
bug 1523104: remote: fix eslint by removing unecessary commas; r=ato
remote/Actor.jsm
remote/Collections.jsm
remote/Connection.jsm
remote/Domain.jsm
remote/Error.jsm
remote/EventEmitter.jsm
remote/Handler.jsm
remote/Log.jsm
remote/MessageChannel.jsm
remote/Observer.jsm
remote/Session.jsm
remote/Target.jsm
remote/TargetListener.jsm
remote/WindowManager.jsm
remote/domain/Log.jsm
remote/domain/Page.jsm
--- a/remote/Actor.jsm
+++ b/remote/Actor.jsm
@@ -27,17 +27,17 @@ class RemoteAgentActorChild extends Acto
     log.trace(`<--(message ${name}) ${JSON.stringify(data)}`);
     super.sendAsyncMessage(name, data);
   }
 
   receiveMessage(name, data) {
     log.trace(`(message ${name})--> ${JSON.stringify(data)}`);
     super.receiveMessage(name, data);
   }
-};
+}
 
 // TODO(ato): Move to MessageChannel.jsm?
 // TODO(ato): This can eventually be replaced by ActorChild and IPDL generation
 // TODO(ato): Can we find a shorter name?
 class MessageChannelActorChild extends RemoteAgentActorChild {
   constructor(dispatcher) {
     super(dispatcher);
     this.name = `RemoteAgent:${this.constructor.name}`;
@@ -64,9 +64,9 @@ class MessageChannelActorChild extends R
 
       const result = await func.call(this, params);
       this.send({id, result});
     } catch ({message, stack}) {
       const error = `${message}\n${stack}`;
       this.send({id, error});
     }
   }
-};
+}
--- a/remote/Collections.jsm
+++ b/remote/Collections.jsm
@@ -17,9 +17,9 @@ class AtomicMap extends Map {
   pop(key) {
     if (!super.has(key)) {
       throw new RangeError("No such key in map: " + key);
     }
     const rv = super.get(key);
     super.delete(key);
     return rv;
   }
-};
+}
--- a/remote/Connection.jsm
+++ b/remote/Connection.jsm
@@ -61,9 +61,9 @@ class Connection {
     }
   }
 
   onClosed(status) {}
 
   toString() {
     return `[object Connection ${this.id}]`;
   }
-};
+}
--- a/remote/Domain.jsm
+++ b/remote/Domain.jsm
@@ -26,16 +26,16 @@ class Domain {
 
   get docShell() {
     return this.session.docShell;
   }
 
   get chromeEventHandler() {
     return this.docShell.chromeEventHandler;
   }
-};
+}
 
 XPCOMUtils.defineLazyModuleGetters(Domain, {
   Log: "chrome://remote/content/domain/Log.jsm",
   Network: "chrome://remote/content/domain/Network.jsm",
   Page: "chrome://remote/content/domain/Page.jsm",
   Runtime: "chrome://remote/content/domain/Runtime.jsm",
 });
--- a/remote/Error.jsm
+++ b/remote/Error.jsm
@@ -42,19 +42,19 @@ class FatalError extends RemoteAgentErro
 
   notify() {
     log.fatal(formatError(this, {stack: true}));
   }
 
   quit(mode = Ci.nsIAppStartup.eForceQuit) {
     Services.startup.quit(mode);
   }
-};
+}
 
-class UnsupportedError extends RemoteAgentError {};
+class UnsupportedError extends RemoteAgentError {}
 
 function formatError(error, {stack = false} = {}) {
   const s = [];
   s.push(`${error.name}: ${error.message}`);
   s.push("");
   if (stack) {
     s.push("Stacktrace:");
   }
--- a/remote/EventEmitter.jsm
+++ b/remote/EventEmitter.jsm
@@ -126,17 +126,17 @@ class EventEmitter {
 
   once(...args) {
     EventEmitter.once(this, ...args);
   }
 
   emit(...args) {
     EventEmitter.emit(this, ...args);
   }
-};
+}
 
 function invoke(listener, target, type, ...args) {
   if (!listener) {
     return;
   }
 
   try {
     if (isEventHandler(listener)) {
--- a/remote/Handler.jsm
+++ b/remote/Handler.jsm
@@ -47,27 +47,27 @@ class TargetListHandler extends JSONHand
 
   get path() {
     return "/json/list";
   }
 
   handle(request, response) {
     response.write([...this.targets]);
   }
-};
+}
 
 class ProtocolHandler extends JSONHandler {
   get path() {
     return "/json/protocol";
   }
 
   handle(request, response) {
     response.write(Protocol.Description);
   }
-};
+}
 
 /**
  * Wraps an httpd.js response and serialises anything passed to
  * write() to JSON.
  */
 class JSONWriter {
   constructor(response) {
     this._response = response;
--- a/remote/Log.jsm
+++ b/remote/Log.jsm
@@ -12,9 +12,9 @@ class Log {
     const StdLog = ChromeUtils.import("resource://gre/modules/Log.jsm").Log;
     const logger = StdLog.repository.getLogger("RemoteAgent");
     if (logger.ownAppenders.length == 0) {
       logger.addAppender(new StdLog.DumpAppender());
       logger.manageLevelFromPref("remote.log.level");
     }
     return logger;
   }
-};
+}
--- a/remote/MessageChannel.jsm
+++ b/remote/MessageChannel.jsm
@@ -82,9 +82,9 @@ class MessageChannel {
   // XPCOM
 
   get QueryInterface() {
     return ChromeUtils.generateQI([
       Ci.nsIMessageListener,
       Ci.nsISupportsWeakReference,
     ]);
   }
-};
+}
--- a/remote/Observer.jsm
+++ b/remote/Observer.jsm
@@ -22,9 +22,9 @@ class Observer {
       const wrappedObserver = (first, ...rest) => {
         Observer.unobserve(type, wrappedObserver);
         observer.call(first, ...rest);
         resolve();
       };
       Observer.observe(type, wrappedObserver);
     });
   }
-};
+}
--- a/remote/Session.jsm
+++ b/remote/Session.jsm
@@ -70,17 +70,17 @@ class Session {
         this.connection.send(data.event);
         break;
       case "remote-protocol:error":
         const error = formatError(data.error, {stack: true});
         this.connection.send({id: data.id, error});
         break;
     }
   }
-};
+}
 
 function assertSchema(domainName, methodName, params) {
   const domain = Domain[domainName];
   if (!domain) {
     throw new TypeError("No such domain: " + domainName);
   }
   if (!domain.schema) {
     throw new Error(`Domain ${domainName} missing schema description`);
--- a/remote/Target.jsm
+++ b/remote/Target.jsm
@@ -129,9 +129,9 @@ class Target {
     if (subject === this.mm && subject == "message-manager-disconnect") {
       // disconnect debugging target if <browser> is disconnected,
       // otherwise this is just a host process change
       if (this.closed) {
         this.disconnect();
       }
     }
   }
-};
+}
--- a/remote/TargetListener.jsm
+++ b/remote/TargetListener.jsm
@@ -65,9 +65,9 @@ class TargetListener {
       return `${network}://${host}:${port}/`;
     }
     return null;
   }
 
   toString() {
     return `[object Debugger ${this.url || "disconnected"}]`;
   }
-};
+}
--- a/remote/WindowManager.jsm
+++ b/remote/WindowManager.jsm
@@ -83,17 +83,17 @@ class WindowObserver {
     this.emit("close", window);
   }
 
   // XPCOM
 
   get QueryInterface() {
     return ChromeUtils.generateQI([Ci.nsIWindowMediatorListener]);
   }
-};
+}
 
 /**
  * Observe Firefox tabs as they open and close.
  *
  * "open" fires when a tab opens.
  * "close" fires when a tab closes.
  */
 class TabObserver {
@@ -140,17 +140,17 @@ class TabObserver {
 
     window.addEventListener("TabOpen", ({target}) => this.onTabOpen(target));
     window.addEventListener("TabClose", ({target}) => this.onTabClose(target));
   }
 
   onWindowClose(window) {
     // TODO(ato): Is TabClose fired when the window closes?
   }
-};
+}
 
 /**
  * BrowserObserver is more powerful than TabObserver,
  * as it watches for any content browser appearing anywhere in Gecko.
  * TabObserver on the other hand is limited to browsers associated with a tab.
  *
  * This class is currently not used by the remote agent,
  * but leave it in here because we may have use for it later
@@ -213,17 +213,17 @@ class BrowserObserver {
   // XPCOM
 
   get QueryInterface() {
     return ChromeUtils.generateQI([
       Ci.nsIMessageListener,
       Ci.nsIObserver,
     ]);
   }
-};
+}
 
 /**
  * Determine if WindowProxy is part of the boundary window.
  *
  * @param {DOMWindow} boundary
  * @param {DOMWindow} target
  *
  * @return {boolean}
--- a/remote/domain/Log.jsm
+++ b/remote/domain/Log.jsm
@@ -94,17 +94,17 @@ class Log extends Domain {
         enable: {},
         disable: {},
       },
       events: {
         entryAdded: Log.LogEntry.schema,
       },
     };
   }
-};
+}
 
 Log.LogEntry = {
   schema: {
     source: t.Enum(ALLOWED_SOURCES),
     level: t.Enum(ALLOWED_LEVELS),
     text: t.String,
     timestamp: Runtime.Timestamp,
     url: t.Optional(t.String),
--- a/remote/domain/Page.jsm
+++ b/remote/domain/Page.jsm
@@ -111,17 +111,17 @@ class Page extends Domain {
           timestamp: Domain.Network.MonotonicTime.schema,
         },
         loadEventFired: {
           timestamp: Domain.Network.MonotonicTime.schema,
         },
       },
     };
   }
-};
+}
 
 Page.FrameId = {schema: t.String};
 Page.TransitionType = {
   schema: t.Enum([
     "auto_bookmark",
     "auto_subframe",
     "link",
     "manual_subframe",