Skip to content

Commit

Permalink
Bug 1667455 - Part 5: Stop importing Services.jsm from chrome-priv JS…
Browse files Browse the repository at this point in the history
… code, top-level single-line cases. r=kmag,webdriver-reviewers,perftest-reviewers,webcompat-reviewers,geckoview-reviewers,extension-reviewers,application-update-reviewers,pip-reviewers,twisniewski,m_kato,jdescottes,mconley,AlexandruIonescu,mossop

Differential Revision: https://phabricator.services.mozilla.com/D150894
  • Loading branch information
arai-a committed Jul 11, 2022
1 parent 73b3299 commit b2c6482
Show file tree
Hide file tree
Showing 500 changed files with 0 additions and 675 deletions.
2 changes: 0 additions & 2 deletions accessible/tests/mochitest/browser.js
Original file line number Diff line number Diff line change
Expand Up @@ -95,8 +95,6 @@ function reloadButton() {
// //////////////////////////////////////////////////////////////////////////////
// private section

var { Services } = ChromeUtils.import("resource://gre/modules/Services.jsm");

var gBrowserContext = {
browserWnd: null,
testFunc: null,
Expand Down
1 change: 0 additions & 1 deletion accessible/tests/mochitest/common.js
Original file line number Diff line number Diff line change
Expand Up @@ -103,7 +103,6 @@ const MAX_TRIM_LENGTH = 100;
/**
* Services to determine if e10s is enabled.
*/
var { Services } = ChromeUtils.import("resource://gre/modules/Services.jsm");

/**
* nsIAccessibilityService service.
Expand Down
2 changes: 0 additions & 2 deletions accessible/tests/mochitest/events.js
Original file line number Diff line number Diff line change
Expand Up @@ -51,8 +51,6 @@ const kFromUserInput = 1;
// //////////////////////////////////////////////////////////////////////////////
// General

var { Services } = ChromeUtils.import("resource://gre/modules/Services.jsm");

/**
* Set up this variable to dump events into DOM.
*/
Expand Down
1 change: 0 additions & 1 deletion browser/base/content/aboutDialog-appUpdater.js
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,6 @@

/* import-globals-from aboutDialog.js */

var { Services } = ChromeUtils.import("resource://gre/modules/Services.jsm");
var { XPCOMUtils } = ChromeUtils.import(
"resource://gre/modules/XPCOMUtils.jsm"
);
Expand Down
1 change: 0 additions & 1 deletion browser/base/content/aboutDialog.js
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,6 @@
/* import-globals-from aboutDialog-appUpdater.js */

// Services = object with smart getters for common XPCOM services
var { Services } = ChromeUtils.import("resource://gre/modules/Services.jsm");
var { AppConstants } = ChromeUtils.import(
"resource://gre/modules/AppConstants.jsm"
);
Expand Down
1 change: 0 additions & 1 deletion browser/base/content/aboutRestartRequired.js
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@
* file, You can obtain one at http://mozilla.org/MPL/2.0/. */

/* eslint-env mozilla/frame-script */
const { Services } = ChromeUtils.import("resource://gre/modules/Services.jsm");

var AboutRestartRequired = {
/* Only do autofocus if we're the toplevel frame; otherwise we
Expand Down
1 change: 0 additions & 1 deletion browser/base/content/browser.js
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@
var { XPCOMUtils } = ChromeUtils.import(
"resource://gre/modules/XPCOMUtils.jsm"
);
var { Services } = ChromeUtils.import("resource://gre/modules/Services.jsm");
var { AppConstants } = ChromeUtils.import(
"resource://gre/modules/AppConstants.jsm"
);
Expand Down
2 changes: 0 additions & 2 deletions browser/base/content/safeMode.js
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,6 @@
* 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/. */

const { Services } = ChromeUtils.import("resource://gre/modules/Services.jsm");

const appStartup = Services.startup;

const { ResetProfile } = ChromeUtils.import(
Expand Down
1 change: 0 additions & 1 deletion browser/base/content/sanitizeDialog.js
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@
/* import-globals-from /toolkit/content/preferencesBindings.js */

var { Sanitizer } = ChromeUtils.import("resource:///modules/Sanitizer.jsm");
var { Services } = ChromeUtils.import("resource://gre/modules/Services.jsm");

Preferences.addAll([
{ id: "privacy.cpd.history", type: "bool" },
Expand Down
1 change: 0 additions & 1 deletion browser/base/content/test/static/head.js
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,6 @@ const IS_ALPHA = /^[a-z]+$/i;
var { PerfTestHelpers } = ChromeUtils.import(
"resource://testing-common/PerfTestHelpers.jsm"
);
var { Services } = ChromeUtils.import("resource://gre/modules/Services.jsm");
var { OS, require } = ChromeUtils.import("resource://gre/modules/osfile.jsm");

/**
Expand Down
1 change: 0 additions & 1 deletion browser/base/content/utilityOverlay.js
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,6 @@
var { AppConstants } = ChromeUtils.import(
"resource://gre/modules/AppConstants.jsm"
);
var { Services } = ChromeUtils.import("resource://gre/modules/Services.jsm");
var { XPCOMUtils } = ChromeUtils.import(
"resource://gre/modules/XPCOMUtils.jsm"
);
Expand Down
1 change: 0 additions & 1 deletion browser/base/content/webrtcIndicator.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@
* 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/. */

const { Services } = ChromeUtils.import("resource://gre/modules/Services.jsm");
const { XPCOMUtils } = ChromeUtils.import(
"resource://gre/modules/XPCOMUtils.jsm"
);
Expand Down
1 change: 0 additions & 1 deletion browser/base/content/webrtcLegacyIndicator.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@
* 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/. */

const { Services } = ChromeUtils.import("resource://gre/modules/Services.jsm");
const { webrtcUI } = ChromeUtils.import("resource:///modules/webrtcUI.jsm");

const BUNDLE_URL = "chrome://browser/locale/webrtcIndicator.properties";
Expand Down
2 changes: 0 additions & 2 deletions browser/components/about/test/unit/test_getURIFlags.js
Original file line number Diff line number Diff line change
@@ -1,5 +1,3 @@
const { Services } = ChromeUtils.import("resource://gre/modules/Services.jsm");

const contract = "@mozilla.org/network/protocol/about;1?what=newtab";
const am = Cc[contract].getService(Ci.nsIAboutModule);
const uri = Services.io.newURI("about:newtab");
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,6 @@
const { RemoteSettings } = ChromeUtils.import(
"resource://services-settings/remote-settings.js"
);
const { Services } = ChromeUtils.import("resource://gre/modules/Services.jsm");

// Initializing BrowserGlue requires a profile on Windows.
do_get_profile();
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@
const { AppConstants } = ChromeUtils.import(
"resource://gre/modules/AppConstants.jsm"
);
const { Services } = ChromeUtils.import("resource://gre/modules/Services.jsm");

add_task(async () => {
await setupStubs();
Expand Down
1 change: 0 additions & 1 deletion browser/components/distribution.js
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,6 @@ const PREF_CACHED_FILE_APPVERSION = "distribution.iniFile.exists.appversion";
const { AppConstants } = ChromeUtils.import(
"resource://gre/modules/AppConstants.jsm"
);
const { Services } = ChromeUtils.import("resource://gre/modules/Services.jsm");
const lazy = {};
ChromeUtils.defineModuleGetter(
lazy,
Expand Down
1 change: 0 additions & 1 deletion browser/components/doh/test/unit/head.js
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@

"use strict";

const { Services } = ChromeUtils.import("resource://gre/modules/Services.jsm");
const { NetUtil } = ChromeUtils.import("resource://gre/modules/NetUtil.jsm");
const { PromiseUtils } = ChromeUtils.import(
"resource://gre/modules/PromiseUtils.jsm"
Expand Down
1 change: 0 additions & 1 deletion browser/components/downloads/content/allDownloadsView.js
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@
var { XPCOMUtils } = ChromeUtils.import(
"resource://gre/modules/XPCOMUtils.jsm"
);
var { Services } = ChromeUtils.import("resource://gre/modules/Services.jsm");

XPCOMUtils.defineLazyModuleGetters(this, {
BrowserWindowTracker: "resource:///modules/BrowserWindowTracker.jsm",
Expand Down
1 change: 0 additions & 1 deletion browser/components/downloads/content/downloads.js
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,6 @@
var { XPCOMUtils } = ChromeUtils.import(
"resource://gre/modules/XPCOMUtils.jsm"
);
var { Services } = ChromeUtils.import("resource://gre/modules/Services.jsm");

ChromeUtils.defineModuleGetter(
this,
Expand Down
2 changes: 0 additions & 2 deletions browser/components/downloads/test/unit/head.js
Original file line number Diff line number Diff line change
@@ -1,5 +1,3 @@
const { Services } = ChromeUtils.import("resource://gre/modules/Services.jsm");

ChromeUtils.defineModuleGetter(
this,
"Downloads",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@

"use strict";

const { Services } = ChromeUtils.import("resource://gre/modules/Services.jsm");
const { XPCOMUtils } = ChromeUtils.import(
"resource://gre/modules/XPCOMUtils.jsm"
);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@

"use strict";

const { Services } = ChromeUtils.import("resource://gre/modules/Services.jsm");
const { Preferences } = ChromeUtils.import(
"resource://gre/modules/Preferences.jsm"
);
Expand Down
2 changes: 0 additions & 2 deletions browser/components/extensions/child/ext-menus.js
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,6 @@

"use strict";

const { Services } = ChromeUtils.import("resource://gre/modules/Services.jsm");

var { withHandlingUserInput } = ExtensionCommon;

var { ExtensionError } = ExtensionUtils;
Expand Down
1 change: 0 additions & 1 deletion browser/components/extensions/test/xpcshell/head.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@

/* exported createHttpServer, promiseConsoleOutput, assertPersistentListeners */

var { Services } = ChromeUtils.import("resource://gre/modules/Services.jsm");
var { XPCOMUtils } = ChromeUtils.import(
"resource://gre/modules/XPCOMUtils.jsm"
);
Expand Down
1 change: 0 additions & 1 deletion browser/components/firefoxview/recently-closed-tabs.mjs
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@

"use strict";

const { Services } = ChromeUtils.import("resource://gre/modules/Services.jsm");
const lazy = {};
ChromeUtils.defineModuleGetter(
lazy,
Expand Down
2 changes: 0 additions & 2 deletions browser/components/installerprefs/test/unit/head.js
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,6 @@
* 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/. */

const { Services } = ChromeUtils.import("resource://gre/modules/Services.jsm");

const { AppConstants } = ChromeUtils.import(
"resource://gre/modules/AppConstants.jsm"
);
Expand Down
2 changes: 0 additions & 2 deletions browser/components/ion/content/ion.js
Original file line number Diff line number Diff line change
Expand Up @@ -23,8 +23,6 @@ const { TelemetryController } = ChromeUtils.import(
"resource://gre/modules/TelemetryController.jsm"
);

const { Services } = ChromeUtils.import("resource://gre/modules/Services.jsm");

let parserUtils = Cc["@mozilla.org/parserutils;1"].getService(
Ci.nsIParserUtils
);
Expand Down
1 change: 0 additions & 1 deletion browser/components/migration/content/migration.js
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,6 @@
const kIMig = Ci.nsIBrowserProfileMigrator;
const kIPStartup = Ci.nsIProfileStartup;

const { Services } = ChromeUtils.import("resource://gre/modules/Services.jsm");
const { AppConstants } = ChromeUtils.import(
"resource://gre/modules/AppConstants.jsm"
);
Expand Down
1 change: 0 additions & 1 deletion browser/components/migration/tests/unit/head_migration.js
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,6 @@ var { Preferences } = ChromeUtils.import(
var { PromiseUtils } = ChromeUtils.import(
"resource://gre/modules/PromiseUtils.jsm"
);
var { Services } = ChromeUtils.import("resource://gre/modules/Services.jsm");
var { XPCOMUtils } = ChromeUtils.import(
"resource://gre/modules/XPCOMUtils.jsm"
);
Expand Down
1 change: 0 additions & 1 deletion browser/components/newtab/test/xpcshell/head.js
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@ http://creativecommons.org/publicdomain/zero/1.0/ */

/* eslint-disable no-unused-vars */

const { Services } = ChromeUtils.import("resource://gre/modules/Services.jsm");
const { XPCOMUtils } = ChromeUtils.import(
"resource://gre/modules/XPCOMUtils.jsm"
);
1 change: 0 additions & 1 deletion browser/components/pagedata/tests/unit/head.js
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@
const { XPCOMUtils } = ChromeUtils.import(
"resource://gre/modules/XPCOMUtils.jsm"
);
const { Services } = ChromeUtils.import("resource://gre/modules/Services.jsm");

XPCOMUtils.defineLazyModuleGetters(this, {
PageDataSchema: "resource:///modules/pagedata/PageDataSchema.jsm",
Expand Down
1 change: 0 additions & 1 deletion browser/components/places/content/bookmarkProperties.js
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,6 @@
/* import-globals-from controller.js */

/* Shared Places Import - change other consumers if you change this: */
var { Services } = ChromeUtils.import("resource://gre/modules/Services.jsm");
var { XPCOMUtils } = ChromeUtils.import(
"resource://gre/modules/XPCOMUtils.jsm"
);
Expand Down
1 change: 0 additions & 1 deletion browser/components/places/content/bookmarksSidebar.js
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@
* file, You can obtain one at http://mozilla.org/MPL/2.0/. */

/* Shared Places Import - change other consumers if you change this: */
var { Services } = ChromeUtils.import("resource://gre/modules/Services.jsm");
var { XPCOMUtils } = ChromeUtils.import(
"resource://gre/modules/XPCOMUtils.jsm"
);
Expand Down
2 changes: 0 additions & 2 deletions browser/components/places/content/editBookmark.js
Original file line number Diff line number Diff line change
Expand Up @@ -13,8 +13,6 @@ var { XPCOMUtils } = ChromeUtils.import(
"resource://gre/modules/XPCOMUtils.jsm"
);

var { Services } = ChromeUtils.import("resource://gre/modules/Services.jsm");

XPCOMUtils.defineLazyModuleGetters(this, {
CustomizableUI: "resource:///modules/CustomizableUI.jsm",
PlacesUtils: "resource://gre/modules/PlacesUtils.jsm",
Expand Down
1 change: 0 additions & 1 deletion browser/components/places/content/historySidebar.js
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@
* file, You can obtain one at http://mozilla.org/MPL/2.0/. */

/* Shared Places Import - change other consumers if you change this: */
var { Services } = ChromeUtils.import("resource://gre/modules/Services.jsm");
var { XPCOMUtils } = ChromeUtils.import(
"resource://gre/modules/XPCOMUtils.jsm"
);
Expand Down
2 changes: 0 additions & 2 deletions browser/components/places/content/instantEditBookmark.js
Original file line number Diff line number Diff line change
Expand Up @@ -12,8 +12,6 @@ var { XPCOMUtils } = ChromeUtils.import(
"resource://gre/modules/XPCOMUtils.jsm"
);

var { Services } = ChromeUtils.import("resource://gre/modules/Services.jsm");

XPCOMUtils.defineLazyModuleGetters(this, {
CustomizableUI: "resource:///modules/CustomizableUI.jsm",
PlacesUtils: "resource://gre/modules/PlacesUtils.jsm",
Expand Down
1 change: 0 additions & 1 deletion browser/components/places/content/places.js
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,6 @@
/* import-globals-from /browser/components/downloads/content/allDownloadsView.js */

/* Shared Places Import - change other consumers if you change this: */
var { Services } = ChromeUtils.import("resource://gre/modules/Services.jsm");
var { XPCOMUtils } = ChromeUtils.import(
"resource://gre/modules/XPCOMUtils.jsm"
);
Expand Down
2 changes: 0 additions & 2 deletions browser/components/places/content/treeView.js
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,6 @@

/* import-globals-from controller.js */

var { Services } = ChromeUtils.import("resource://gre/modules/Services.jsm");

/**
* This returns the key for any node/details object.
*
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,6 @@
const { Interactions } = ChromeUtils.import(
"resource:///modules/Interactions.jsm"
);
const { Services } = ChromeUtils.import("resource://gre/modules/Services.jsm");
const { Snapshots } = ChromeUtils.import("resource:///modules/Snapshots.jsm");
const { PlacesUtils } = ChromeUtils.import(
"resource://gre/modules/PlacesUtils.jsm"
Expand Down
1 change: 0 additions & 1 deletion browser/components/places/tests/chrome/head.js
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
/* Any copyright is dedicated to the Public Domain.
http://creativecommons.org/publicdomain/zero/1.0/ */

var { Services } = ChromeUtils.import("resource://gre/modules/Services.jsm");
Services.scriptloader.loadSubScript(
"chrome://global/content/globalOverlay.js",
this
Expand Down
2 changes: 0 additions & 2 deletions browser/components/places/tests/unit/head_bookmarks.js
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,6 @@
* 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/. */

var { Services } = ChromeUtils.import("resource://gre/modules/Services.jsm");

// Import common head.
/* import-globals-from ../../../../../toolkit/components/places/tests/head_common.js */
var commonFile = do_get_file(
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@
const { XPCOMUtils } = ChromeUtils.import(
"resource://gre/modules/XPCOMUtils.jsm"
);
const { Services } = ChromeUtils.import("resource://gre/modules/Services.jsm");

XPCOMUtils.defineLazyModuleGetters(this, {
Interactions: "resource:///modules/Interactions.jsm",
Expand Down
1 change: 0 additions & 1 deletion browser/components/preferences/dialogs/blocklists.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@
* 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/. */

var { Services } = ChromeUtils.import("resource://gre/modules/Services.jsm");
const BASE_LIST_ID = "base";
const CONTENT_LIST_ID = "content";
const TRACK_SUFFIX = "-track-digest256";
Expand Down
2 changes: 0 additions & 2 deletions browser/components/preferences/dialogs/browserLanguages.js
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,6 @@

/* import-globals-from /toolkit/content/preferencesBindings.js */

var { Services } = ChromeUtils.import("resource://gre/modules/Services.jsm");

// This is exported by preferences.js but we can't import that in a subdialog.
let { LangPackMatcher } = window.top;

Expand Down
2 changes: 0 additions & 2 deletions browser/components/preferences/dialogs/languages.js
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,6 @@

/* import-globals-from /toolkit/content/preferencesBindings.js */

var { Services } = ChromeUtils.import("resource://gre/modules/Services.jsm");

document
.getElementById("LanguagesDialog")
.addEventListener("dialoghelp", window.top.openPrefsHelp);
Expand Down
1 change: 0 additions & 1 deletion browser/components/preferences/dialogs/permissions.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@
* 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/. */

var { Services } = ChromeUtils.import("resource://gre/modules/Services.jsm");
var { AppConstants } = ChromeUtils.import(
"resource://gre/modules/AppConstants.jsm"
);
Expand Down
1 change: 0 additions & 1 deletion browser/components/preferences/dialogs/selectBookmark.js
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@
* file, You can obtain one at http://mozilla.org/MPL/2.0/. */

/* Shared Places Import - change other consumers if you change this: */
var { Services } = ChromeUtils.import("resource://gre/modules/Services.jsm");
var { XPCOMUtils } = ChromeUtils.import(
"resource://gre/modules/XPCOMUtils.jsm"
);
Expand Down
1 change: 0 additions & 1 deletion browser/components/preferences/dialogs/siteDataSettings.js
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,6 @@
var { AppConstants } = ChromeUtils.import(
"resource://gre/modules/AppConstants.jsm"
);
var { Services } = ChromeUtils.import("resource://gre/modules/Services.jsm");

ChromeUtils.defineModuleGetter(
this,
Expand Down
Loading

0 comments on commit b2c6482

Please sign in to comment.