Bug 1533679 - Remove unused target argument passed to Domain constructor. r=ato
authorAlexandre Poirot <poirot.alex@gmail.com>
Sun, 10 Mar 2019 12:50:55 +0000
changeset 521278 6e5c72c4c884
parent 521277 76ba842e38f5
child 521279 886ef9f05fa3
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
bugs1533679
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 1533679 - Remove unused target argument passed to Domain constructor. r=ato Depends on D22684 Differential Revision: https://phabricator.services.mozilla.com/D22685
remote/domains/Domain.jsm
remote/domains/Domains.jsm
remote/domains/content/Log.jsm
remote/domains/content/Page.jsm
--- a/remote/domains/Domain.jsm
+++ b/remote/domains/Domain.jsm
@@ -2,19 +2,18 @@
  * License, v. 2.0. If a copy of the MPL was not distributed with this
  * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
 
 "use strict";
 
 var EXPORTED_SYMBOLS = ["Domain"];
 
 class Domain {
-  constructor(session, target) {
+  constructor(session) {
     this.session = session;
-    this.target = target;
     this.name = this.constructor.name;
 
     this.eventListeners_ = new Set();
   }
 
   destructor() {}
 
   emit(eventName, params = {}) {
--- a/remote/domains/Domains.jsm
+++ b/remote/domains/Domains.jsm
@@ -53,17 +53,17 @@ class Domains {
       const Cls = this.modules[name];
       if (!Cls) {
         throw new UnknownMethodError(name);
       }
       if (!isConstructor(Cls)) {
         throw new TypeError("Domain cannot be constructed");
       }
 
-      inst = new Cls(this.session, this.session.target);
+      inst = new Cls(this.session);
       if (!(inst instanceof Domain)) {
         throw new TypeError("Instance not a domain");
       }
 
       inst.addEventListener(this.session);
 
       this.instances.set(name, inst);
     }
--- a/remote/domains/content/Log.jsm
+++ b/remote/domains/content/Log.jsm
@@ -5,18 +5,18 @@
 "use strict";
 
 var EXPORTED_SYMBOLS = ["Log"];
 
 const {Domain} = ChromeUtils.import("chrome://remote/content/domains/Domain.jsm");
 const {Services} = ChromeUtils.import("resource://gre/modules/Services.jsm");
 
 class Log extends Domain {
-  constructor(session, target) {
-    super(session, target);
+  constructor(session) {
+    super(session);
     this.enabled = false;
   }
 
   destructor() {
     this.disable();
   }
 
   enable() {
--- a/remote/domains/content/Page.jsm
+++ b/remote/domains/content/Page.jsm
@@ -6,18 +6,18 @@
 
 var EXPORTED_SYMBOLS = ["Page"];
 
 const {Domain} = ChromeUtils.import("chrome://remote/content/domains/Domain.jsm");
 const {Services} = ChromeUtils.import("resource://gre/modules/Services.jsm");
 const {UnsupportedError} = ChromeUtils.import("chrome://remote/content/Error.jsm");
 
 class Page extends Domain {
-  constructor(session, target) {
-    super(session, target);
+  constructor(session) {
+    super(session);
     this.enabled = false;
   }
 
   destructor() {
     this.disable();
   }
 
   QueryInterface(iid) {