Bug 1526981 - [2.4] Simplify JSM module logger initialization and ensure compatibility with ES6. r=geckoview-reviewers,agi

Differential Revision: https://phabricator.services.mozilla.com/D19580

--HG--
extra : moz-landing-system : lando

mobile/android/chrome/geckoview/ErrorPageEventHandler.js

@@ -7,8 +7,7 @@ const {GeckoViewUtils} = ChromeUtils.import("resource://gre/modules/GeckoViewUti
 var {XPCOMUtils} = ChromeUtils.import("resource://gre/modules/XPCOMUtils.jsm");
 var {Services} = ChromeUtils.import("resource://gre/modules/Services.jsm");
 
-/* global debug:false, warn:false */
+const {debug, warn} = GeckoViewUtils.initLogging("ErrorPageEventHandler"); // eslint-disable-line no-unused-vars
-GeckoViewUtils.initLogging("ErrorPageEventHandler", this);
 
 ChromeUtils.defineModuleGetter(this, "SSLExceptions",
                                "resource://gre/modules/SSLExceptions.jsm");

mobile/android/chrome/geckoview/GeckoViewContentChild.js

@@ -451,5 +451,5 @@ class GeckoViewContentChild extends GeckoViewChildModule {
   }
 }
 
-let {debug, warn} = GeckoViewContentChild.initLogging("GeckoViewContent");
+const {debug, warn} = GeckoViewContentChild.initLogging("GeckoViewContent"); // eslint-disable-line no-unused-vars
-let module = GeckoViewContentChild.create(this);
+const module = GeckoViewContentChild.create(this);

mobile/android/chrome/geckoview/GeckoViewMediaChild.js

@@ -428,5 +428,5 @@ class GeckoViewMediaChild extends GeckoViewChildModule {
   }
 }
 
-const {debug, warn} = GeckoViewMediaChild.initLogging("GeckoViewMedia");
+const {debug, warn} = GeckoViewMediaChild.initLogging("GeckoViewMedia"); // eslint-disable-line no-unused-vars
 const module = GeckoViewMediaChild.create(this);

mobile/android/chrome/geckoview/GeckoViewNavigationChild.js

@@ -137,5 +137,5 @@ class GeckoViewNavigationChild extends GeckoViewChildModule {
   }
 }
 
-let {debug, warn} = GeckoViewNavigationChild.initLogging("GeckoViewNavigation");
+const {debug, warn} = GeckoViewNavigationChild.initLogging("GeckoViewNavigation"); // eslint-disable-line no-unused-vars
-let module = GeckoViewNavigationChild.create(this);
+const module = GeckoViewNavigationChild.create(this);

mobile/android/chrome/geckoview/GeckoViewProgressChild.js

@@ -333,5 +333,5 @@ const ProgressTracker = {
 };
 
 
-let {debug, warn} = GeckoViewProgressChild.initLogging("GeckoViewProgress");
+const {debug, warn} = GeckoViewProgressChild.initLogging("GeckoViewProgress"); // eslint-disable-line no-unused-vars
-let module = GeckoViewProgressChild.create(this);
+const module = GeckoViewProgressChild.create(this);

mobile/android/chrome/geckoview/GeckoViewScrollChild.js

@@ -37,5 +37,5 @@ class GeckoViewScrollChild extends GeckoViewChildModule {
   }
 }
 
-let {debug, warn} = GeckoViewScrollChild.initLogging("GeckoViewScroll");
+const {debug, warn} = GeckoViewScrollChild.initLogging("GeckoViewScroll"); // eslint-disable-line no-unused-vars
-let module = GeckoViewScrollChild.create(this);
+const module = GeckoViewScrollChild.create(this);

mobile/android/chrome/geckoview/GeckoViewSelectionActionChild.js

@@ -238,6 +238,5 @@ class GeckoViewSelectionActionChild extends GeckoViewChildModule {
   }
 }
 
-let {debug, warn} =
+const {debug, warn} = GeckoViewSelectionActionChild.initLogging("GeckoViewSelectionAction"); // eslint-disable-line no-unused-vars
-    GeckoViewSelectionActionChild.initLogging("GeckoViewSelectionAction");
+const module = GeckoViewSelectionActionChild.create(this);
-let module = GeckoViewSelectionActionChild.create(this);

mobile/android/chrome/geckoview/GeckoViewSettingsChild.js

@@ -152,5 +152,5 @@ class GeckoViewSettingsChild extends GeckoViewChildModule {
   }
 }
 
-let {debug, warn} = GeckoViewSettingsChild.initLogging("GeckoViewSettings");
+const {debug, warn} = GeckoViewSettingsChild.initLogging("GeckoViewSettings"); // eslint-disable-line no-unused-vars
-let module = GeckoViewSettingsChild.create(this);
+const module = GeckoViewSettingsChild.create(this);

mobile/android/components/geckoview/GeckoViewExternalAppService.js

@@ -7,8 +7,7 @@
 const {XPCOMUtils} = ChromeUtils.import("resource://gre/modules/XPCOMUtils.jsm");
 const {GeckoViewUtils} = ChromeUtils.import("resource://gre/modules/GeckoViewUtils.jsm");
 
-/* global debug:false, warn:false */
+const {debug, warn} = GeckoViewUtils.initLogging("ExternalAppService"); // eslint-disable-line no-unused-vars
-GeckoViewUtils.initLogging("ExternalAppService", this);
 
 ChromeUtils.defineModuleGetter(this, "EventDispatcher",
   "resource://gre/modules/Messaging.jsm");

mobile/android/components/geckoview/GeckoViewStartup.js

@@ -15,8 +15,7 @@ XPCOMUtils.defineLazyModuleGetters(this, {
   Services: "resource://gre/modules/Services.jsm",
 });
 
-/* global debug:false, warn:false */
+const {debug, warn} = GeckoViewUtils.initLogging("Startup"); // eslint-disable-line no-unused-vars
-GeckoViewUtils.initLogging("Startup", this);
 
 function GeckoViewStartup() {
 }

mobile/android/modules/geckoview/ContentCrashHandler.jsm

@@ -18,7 +18,7 @@ XPCOMUtils.defineLazyModuleGetters(this, {
 ChromeUtils.defineModuleGetter(this, "OS",
                                "resource://gre/modules/osfile.jsm");
 
-GeckoViewUtils.initLogging("ContentCrashHandler", this);
+const {debug, warn} = GeckoViewUtils.initLogging("ContentCrashHandler"); // eslint-disable-line no-unused-vars
 
 function getDir(name) {
   let uAppDataPath = Services.dirsvc.get("UAppData", Ci.nsIFile).path;

mobile/android/modules/geckoview/GeckoViewAccessibility.jsm

@@ -25,3 +25,5 @@ class GeckoViewAccessibility extends GeckoViewModule {
     }, "GeckoView:AccessibilitySettings");
   }
 }
+
+const {debug, warn} = GeckoViewAccessibility.initLogging("GeckoViewAccessibility"); // eslint-disable-line no-unused-vars

mobile/android/modules/geckoview/GeckoViewAutoFill.jsm

@@ -15,7 +15,7 @@ XPCOMUtils.defineLazyModuleGetters(this, {
   LoginManagerContent: "resource://gre/modules/LoginManagerContent.jsm",
 });
 
-GeckoViewUtils.initLogging("AutoFill", this);
+const {debug, warn} = GeckoViewUtils.initLogging("AutoFill"); // eslint-disable-line no-unused-vars
 
 class GeckoViewAutoFill {
   constructor(aEventDispatcher) {

mobile/android/modules/geckoview/GeckoViewChildModule.jsm

@@ -8,13 +8,13 @@ var EXPORTED_SYMBOLS = ["GeckoViewChildModule"];
 
 const {GeckoViewUtils} = ChromeUtils.import("resource://gre/modules/GeckoViewUtils.jsm");
 
-GeckoViewUtils.initLogging("Module[C]", this);
+const {debug, warn} = GeckoViewUtils.initLogging("Module[C]"); // eslint-disable-line no-unused-vars
 
 class GeckoViewChildModule {
   static initLogging(aModuleName) {
     this._moduleName = aModuleName;
     const tag = aModuleName.replace("GeckoView", "") + "[C]";
-    return GeckoViewUtils.initLogging(tag, {});
+    return GeckoViewUtils.initLogging(tag);
   }
 
   static create(aGlobal, aModuleName) {

mobile/android/modules/geckoview/GeckoViewConsole.jsm

@@ -12,7 +12,7 @@ XPCOMUtils.defineLazyModuleGetters(this, {
   Services: "resource://gre/modules/Services.jsm",
 });
 
-GeckoViewUtils.initLogging("Console", this);
+const {debug, warn} = GeckoViewUtils.initLogging("Console"); // eslint-disable-line no-unused-vars
 
 const LOG_EVENT_TOPIC = "console-api-log-event";
 

mobile/android/modules/geckoview/GeckoViewContent.jsm

@@ -350,3 +350,5 @@ class GeckoViewContent extends GeckoViewModule {
     finder.highlight(true, finder.searchString, linksOnly, !!aData.drawOutline);
   }
 }
+
+const {debug, warn} = GeckoViewContent.initLogging("GeckoViewContent"); // eslint-disable-line no-unused-vars

mobile/android/modules/geckoview/GeckoViewContentBlocking.jsm

@@ -54,3 +54,5 @@ class GeckoViewContentBlocking extends GeckoViewModule {
     this.eventDispatcher.sendRequest(message);
   }
 }
+
+const {debug, warn} = GeckoViewContentBlocking.initLogging("GeckoViewContentBlocking"); // eslint-disable-line no-unused-vars

mobile/android/modules/geckoview/GeckoViewMedia.jsm

@@ -31,3 +31,5 @@ class GeckoViewMedia extends GeckoViewModule {
     this.messageManager.sendAsyncMessage(aEvent, aData);
   }
 }
+
+const {debug, warn} = GeckoViewMedia.initLogging("GeckoViewMedia"); // eslint-disable-line no-unused-vars

mobile/android/modules/geckoview/GeckoViewModule.jsm

@@ -8,9 +8,14 @@ var EXPORTED_SYMBOLS = ["GeckoViewModule"];
 
 const {GeckoViewUtils} = ChromeUtils.import("resource://gre/modules/GeckoViewUtils.jsm");
 
-GeckoViewUtils.initLogging("Module", this);
+const {debug, warn} = GeckoViewUtils.initLogging("Module"); // eslint-disable-line no-unused-vars
 
 class GeckoViewModule {
+  static initLogging(aModuleName) {
+    const tag = aModuleName.replace("GeckoView", "");
+    return GeckoViewUtils.initLogging(tag);
+  }
+
   constructor(aModuleInfo) {
     this._info = aModuleInfo;
 

mobile/android/modules/geckoview/GeckoViewNavigation.jsm

@@ -340,3 +340,5 @@ class GeckoViewNavigation extends GeckoViewModule {
     this.eventDispatcher.sendRequest(message);
   }
 }
+
+const {debug, warn} = GeckoViewNavigation.initLogging("GeckoViewNavigation"); // eslint-disable-line no-unused-vars

mobile/android/modules/geckoview/GeckoViewProgress.jsm

@@ -259,3 +259,5 @@ class GeckoViewProgress extends GeckoViewModule {
     }
   }
 }
+
+const {debug, warn} = GeckoViewProgress.initLogging("GeckoViewProgress"); // eslint-disable-line no-unused-vars

mobile/android/modules/geckoview/GeckoViewRemoteDebugger.jsm

@@ -28,7 +28,7 @@ XPCOMUtils.defineLazyGetter(this, "SocketListener", () => {
   return SocketListener;
 });
 
-GeckoViewUtils.initLogging("RemoteDebugger", this);
+const {debug, warn} = GeckoViewUtils.initLogging("RemoteDebugger"); // eslint-disable-line no-unused-vars
 
 var GeckoViewRemoteDebugger = {
   observe(aSubject, aTopic, aData) {

mobile/android/modules/geckoview/GeckoViewSettings.jsm

@@ -119,3 +119,5 @@ class GeckoViewSettings extends GeckoViewModule {
     this.window.docShell.displayMode = aMode;
   }
 }
+
+const {debug, warn} = GeckoViewSettings.initLogging("GeckoViewSettings"); // eslint-disable-line no-unused-vars

mobile/android/modules/geckoview/GeckoViewTab.jsm

@@ -60,3 +60,5 @@ class GeckoViewTab extends GeckoViewModule {
     };
   }
 }
+
+const {debug, warn} = GeckoViewTab.initLogging("GeckoViewTab"); // eslint-disable-line no-unused-vars

mobile/android/modules/geckoview/GeckoViewUtils.jsm

@@ -375,6 +375,7 @@ var GeckoViewUtils = {
    * @param aScope Scope to add the logging functions to.
    */
   initLogging: function(aTag, aScope) {
+    aScope = aScope || {};
     const tag = "GeckoView." + aTag.replace(/^GeckoView\.?/, "");
 
     // Only provide two levels for simplicity.

mobile/android/modules/geckoview/LoadURIDelegate.jsm

@@ -13,7 +13,7 @@ XPCOMUtils.defineLazyModuleGetters(this, {
   Services: "resource://gre/modules/Services.jsm",
 });
 
-GeckoViewUtils.initLogging("LoadURIDelegate", this);
+const {debug, warn} = GeckoViewUtils.initLogging("LoadURIDelegate"); // eslint-disable-line no-unused-vars
 
 const LoadURIDelegate = {
   // Delegate URI loading to the app.

toolkit/components/telemetry/geckoview/GeckoViewTelemetryController.jsm

@@ -13,7 +13,7 @@ XPCOMUtils.defineLazyModuleGetters(this, {
   TelemetryUtils: "resource://gre/modules/TelemetryUtils.jsm",
 });
 
-GeckoViewUtils.initLogging("GeckoView.TelemetryController", this);
+const {debug, warn} = GeckoViewUtils.initLogging("GeckoView.TelemetryController"); // eslint-disable-line no-unused-vars
 
 var EXPORTED_SYMBOLS = ["GeckoViewTelemetryController"];