Bug 1484153 - Drop duplicate outerWindowID logging in listener.js. r=whimboo
authorAndreas Tolfsen <ato@sny.no>
Fri, 17 Aug 2018 09:46:46 +0100
changeset 487241 f334098ab7bc805488005b3f09537e59a953cccb
parent 487240 131559bd826a1ea9c58f63de32a62eb67d1000bf
child 487242 00b7caa950247c3d1e1cb6cd8f728b1a5edbc3b7
push id9719
push userffxbld-merge
push dateFri, 24 Aug 2018 17:49:46 +0000
treeherdermozilla-beta@719ec98fba77 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerswhimboo
bugs1484153, 1464469
milestone63.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 1484153 - Drop duplicate outerWindowID logging in listener.js. r=whimboo When receiving observer notifications during navigation, we log the outerWindowID: logger.debug(`Received observer notification ${topic} for ${winID}`); This is no longer necessary as https://bugzilla.mozilla.org/show_bug.cgi?id=1464469 introduced the outerWindowID as a prefix to all log messages originating from this frame script.
testing/marionette/listener.js
--- a/testing/marionette/listener.js
+++ b/testing/marionette/listener.js
@@ -339,17 +339,17 @@ const loadListener = {
     }
   },
 
   observe(subject, topic) {
     const win = curContainer.frame;
     const winID = subject.QueryInterface(Ci.nsISupportsPRUint64).data;
     const curWinID = win.windowUtils.outerWindowID;
 
-    logger.debug(`Received observer notification ${topic} for ${winID}`);
+    logger.debug(`Received observer notification ${topic}`);
 
     switch (topic) {
       // In the case when the currently selected frame is closed,
       // there will be no further load events. Stop listening immediately.
       case "outer-window-destroyed":
         if (curWinID === winID) {
           this.stop();
           sendOk(this.commandID);