Bug 1610653 - Part 3: Do not use global this property assignment to define global variable in browser/modules. r=Gijs

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

browser/modules/BrowserWindowTracker.jsm

@@ -147,7 +147,7 @@ var WindowHelper = {
   },
 };
 
-this.BrowserWindowTracker = {
+const BrowserWindowTracker = {
   /**
    * Get the most recent browser window.
    *

browser/modules/EveryWindow.jsm

@@ -42,7 +42,7 @@ function callForEveryWindow(callback) {
   }
 }
 
-this.EveryWindow = {
+const EveryWindow = {
   /**
    * Registers init and uninit functions to be called on every window.
    *

browser/modules/OpenInTabsUtils.jsm

@@ -4,7 +4,7 @@
 
 "use strict";
 
-this.EXPORTED_SYMBOLS = ["OpenInTabsUtils"];
+const EXPORTED_SYMBOLS = ["OpenInTabsUtils"];
 
 const { XPCOMUtils } = ChromeUtils.import(
   "resource://gre/modules/XPCOMUtils.jsm"
@@ -21,7 +21,7 @@ XPCOMUtils.defineLazyGetter(this, "bundle", function() {
  * Utility functions that can be used when opening multiple tabs, that can be
  * called without any tabbrowser instance.
  */
-this.OpenInTabsUtils = {
+const OpenInTabsUtils = {
   getString(key) {
     return bundle.GetStringFromName(key);
   },

browser/modules/PingCentre.jsm

@@ -159,7 +159,7 @@ class PingCentre {
 }
 
 this.PingCentre = PingCentre;
-this.PingCentreConstants = {
+const PingCentreConstants = {
   FHR_UPLOAD_ENABLED_PREF,
   TELEMETRY_PREF,
   LOGGING_PREF,