Bug 1789254 - Migrate enterprisepolicies consumers to use ESM imports directly. r=mkaply,necko-reviewers,application-update-reviewers,nalexander

Differential Revision: https://phabricator.services.mozilla.com/D156464
This commit is contained in:
Mark Banner 2022-09-08 14:40:07 +00:00
parent 27c9a7780a
commit c9b0aa74c1
27 changed files with 64 additions and 58 deletions

@ -27,6 +27,7 @@ ChromeUtils.defineESModuleGetters(this, {
UrlbarTokenizer: "resource:///modules/UrlbarTokenizer.sys.mjs", UrlbarTokenizer: "resource:///modules/UrlbarTokenizer.sys.mjs",
UrlbarUtils: "resource:///modules/UrlbarUtils.sys.mjs", UrlbarUtils: "resource:///modules/UrlbarUtils.sys.mjs",
UrlbarValueFormatter: "resource:///modules/UrlbarValueFormatter.sys.mjs", UrlbarValueFormatter: "resource:///modules/UrlbarValueFormatter.sys.mjs",
WebsiteFilter: "resource:///modules/policies/WebsiteFilter.sys.mjs",
}); });
XPCOMUtils.defineLazyModuleGetters(this, { XPCOMUtils.defineLazyModuleGetters(this, {
@ -96,7 +97,6 @@ XPCOMUtils.defineLazyModuleGetters(this, {
Weave: "resource://services-sync/main.js", Weave: "resource://services-sync/main.js",
WebNavigationFrames: "resource://gre/modules/WebNavigationFrames.jsm", WebNavigationFrames: "resource://gre/modules/WebNavigationFrames.jsm",
webrtcUI: "resource:///modules/webrtcUI.jsm", webrtcUI: "resource:///modules/webrtcUI.jsm",
WebsiteFilter: "resource:///modules/policies/WebsiteFilter.jsm",
ZoomUI: "resource:///modules/ZoomUI.jsm", ZoomUI: "resource:///modules/ZoomUI.jsm",
}); });

@ -2,12 +2,12 @@
* http://creativecommons.org/publicdomain/zero/1.0/ */ * http://creativecommons.org/publicdomain/zero/1.0/ */
ChromeUtils.defineESModuleGetters(this, { ChromeUtils.defineESModuleGetters(this, {
EnterprisePolicyTesting:
"resource://testing-common/EnterprisePolicyTesting.sys.mjs",
PlacesTestUtils: "resource://testing-common/PlacesTestUtils.sys.mjs", PlacesTestUtils: "resource://testing-common/PlacesTestUtils.sys.mjs",
}); });
XPCOMUtils.defineLazyModuleGetters(this, { XPCOMUtils.defineLazyModuleGetters(this, {
EnterprisePolicyTesting:
"resource://testing-common/EnterprisePolicyTesting.jsm",
ExtensionCommon: "resource://gre/modules/ExtensionCommon.jsm", ExtensionCommon: "resource://gre/modules/ExtensionCommon.jsm",
sinon: "resource://testing-common/Sinon.jsm", sinon: "resource://testing-common/Sinon.jsm",
TelemetryTestUtils: "resource://testing-common/TelemetryTestUtils.jsm", TelemetryTestUtils: "resource://testing-common/TelemetryTestUtils.jsm",

@ -9,8 +9,8 @@ const { ExperimentFakes } = ChromeUtils.import(
"resource://testing-common/NimbusTestUtils.jsm" "resource://testing-common/NimbusTestUtils.jsm"
); );
const { EnterprisePolicyTesting } = ChromeUtils.import( const { EnterprisePolicyTesting } = ChromeUtils.importESModule(
"resource://testing-common/EnterprisePolicyTesting.jsm" "resource://testing-common/EnterprisePolicyTesting.sys.mjs"
); );
ChromeUtils.defineModuleGetter( ChromeUtils.defineModuleGetter(

@ -6,8 +6,8 @@
add_task(setup); add_task(setup);
const { EnterprisePolicyTesting } = ChromeUtils.import( const { EnterprisePolicyTesting } = ChromeUtils.importESModule(
"resource://testing-common/EnterprisePolicyTesting.jsm" "resource://testing-common/EnterprisePolicyTesting.sys.mjs"
); );
add_task(async function testPolicyOverride() { add_task(async function testPolicyOverride() {

@ -3,8 +3,8 @@
"use strict"; "use strict";
const { EnterprisePolicyTesting } = ChromeUtils.import( const { EnterprisePolicyTesting } = ChromeUtils.importESModule(
"resource://testing-common/EnterprisePolicyTesting.jsm" "resource://testing-common/EnterprisePolicyTesting.sys.mjs"
); );
ChromeUtils.defineModuleGetter( ChromeUtils.defineModuleGetter(

@ -3,8 +3,11 @@
"use strict"; "use strict";
async function runWithDisabledPrivateBrowsing(callback) { async function runWithDisabledPrivateBrowsing(callback) {
const { EnterprisePolicyTesting, PoliciesPrefTracker } = ChromeUtils.import( const {
"resource://testing-common/EnterprisePolicyTesting.jsm" EnterprisePolicyTesting,
PoliciesPrefTracker,
} = ChromeUtils.importESModule(
"resource://testing-common/EnterprisePolicyTesting.sys.mjs"
); );
PoliciesPrefTracker.start(); PoliciesPrefTracker.start();

@ -11,8 +11,8 @@ ChromeUtils.defineModuleGetter(
"resource://gre/modules/AbuseReporter.jsm" "resource://gre/modules/AbuseReporter.jsm"
); );
const { EnterprisePolicyTesting } = ChromeUtils.import( const { EnterprisePolicyTesting } = ChromeUtils.importESModule(
"resource://testing-common/EnterprisePolicyTesting.jsm" "resource://testing-common/EnterprisePolicyTesting.sys.mjs"
); );
const { TelemetryTestUtils } = ChromeUtils.import( const { TelemetryTestUtils } = ChromeUtils.import(
"resource://testing-common/TelemetryTestUtils.jsm" "resource://testing-common/TelemetryTestUtils.jsm"

@ -13,8 +13,8 @@ const { ASRouterTargeting } = ChromeUtils.import(
const { MacAttribution } = ChromeUtils.import( const { MacAttribution } = ChromeUtils.import(
"resource:///modules/MacAttribution.jsm" "resource:///modules/MacAttribution.jsm"
); );
const { EnterprisePolicyTesting } = ChromeUtils.import( const { EnterprisePolicyTesting } = ChromeUtils.importESModule(
"resource://testing-common/EnterprisePolicyTesting.jsm" "resource://testing-common/EnterprisePolicyTesting.sys.mjs"
); );
add_task(async function check_attribution_data() { add_task(async function check_attribution_data() {

@ -5,8 +5,11 @@
requestLongerTimeout(4); requestLongerTimeout(4);
const { EnterprisePolicyTesting, PoliciesPrefTracker } = ChromeUtils.import( const {
"resource://testing-common/EnterprisePolicyTesting.jsm" EnterprisePolicyTesting,
PoliciesPrefTracker,
} = ChromeUtils.importESModule(
"resource://testing-common/EnterprisePolicyTesting.sys.mjs"
); );
ChromeUtils.defineModuleGetter( ChromeUtils.defineModuleGetter(

@ -3,8 +3,8 @@
"use strict"; "use strict";
const { EnterprisePolicyTesting } = ChromeUtils.import( const { EnterprisePolicyTesting } = ChromeUtils.importESModule(
"resource://testing-common/EnterprisePolicyTesting.jsm" "resource://testing-common/EnterprisePolicyTesting.sys.mjs"
); );
let { TelemetryTestUtils } = ChromeUtils.import( let { TelemetryTestUtils } = ChromeUtils.import(

@ -3,8 +3,8 @@
"use strict"; "use strict";
const { EnterprisePolicyTesting } = ChromeUtils.import( const { EnterprisePolicyTesting } = ChromeUtils.importESModule(
"resource://testing-common/EnterprisePolicyTesting.jsm" "resource://testing-common/EnterprisePolicyTesting.sys.mjs"
); );
let { Region } = ChromeUtils.import("resource://gre/modules/Region.jsm"); let { Region } = ChromeUtils.import("resource://gre/modules/Region.jsm");

@ -3,8 +3,8 @@
"use strict"; "use strict";
const { EnterprisePolicyTesting } = ChromeUtils.import( const { EnterprisePolicyTesting } = ChromeUtils.importESModule(
"resource://testing-common/EnterprisePolicyTesting.jsm" "resource://testing-common/EnterprisePolicyTesting.sys.mjs"
); );
const PERMISSIONS_URL = const PERMISSIONS_URL =

@ -8,12 +8,12 @@
*/ */
ChromeUtils.defineESModuleGetters(this, { ChromeUtils.defineESModuleGetters(this, {
EnterprisePolicyTesting:
"resource://testing-common/EnterprisePolicyTesting.sys.mjs",
UrlbarQuickSuggest: "resource:///modules/UrlbarQuickSuggest.sys.mjs", UrlbarQuickSuggest: "resource:///modules/UrlbarQuickSuggest.sys.mjs",
}); });
XPCOMUtils.defineLazyModuleGetters(this, { XPCOMUtils.defineLazyModuleGetters(this, {
EnterprisePolicyTesting:
"resource://testing-common/EnterprisePolicyTesting.jsm",
sinon: "resource://testing-common/Sinon.jsm", sinon: "resource://testing-common/Sinon.jsm",
}); });

@ -24,8 +24,8 @@ updateAppInfo({
platformVersion: "48", platformVersion: "48",
}); });
const { EnterprisePolicyTesting } = ChromeUtils.import( const { EnterprisePolicyTesting } = ChromeUtils.importESModule(
"resource://testing-common/EnterprisePolicyTesting.jsm" "resource://testing-common/EnterprisePolicyTesting.sys.mjs"
); );
// This initializes the policy engine for xpcshell tests // This initializes the policy engine for xpcshell tests

@ -8,8 +8,8 @@ var EXPORTED_SYMBOLS = ["runBackgroundTask"];
const { Checker } = ChromeUtils.import( const { Checker } = ChromeUtils.import(
"resource://gre/modules/UpdateService.jsm" "resource://gre/modules/UpdateService.jsm"
); );
const { EnterprisePolicyTesting } = ChromeUtils.import( const { EnterprisePolicyTesting } = ChromeUtils.importESModule(
"resource://testing-common/EnterprisePolicyTesting.jsm" "resource://testing-common/EnterprisePolicyTesting.sys.mjs"
); );
async function runBackgroundTask(commandLine) { async function runBackgroundTask(commandLine) {

@ -18,8 +18,8 @@ updateAppInfo({
platformVersion: "48", platformVersion: "48",
}); });
const { EnterprisePolicyTesting } = ChromeUtils.import( const { EnterprisePolicyTesting } = ChromeUtils.importESModule(
"resource://testing-common/EnterprisePolicyTesting.jsm" "resource://testing-common/EnterprisePolicyTesting.sys.mjs"
); );
// This initializes the policy engine for xpcshell tests // This initializes the policy engine for xpcshell tests

@ -11,13 +11,13 @@ const { AppConstants } = ChromeUtils.import(
const lazy = {}; const lazy = {};
ChromeUtils.defineESModuleGetters(lazy, { ChromeUtils.defineESModuleGetters(lazy, {
Policies: "resource:///modules/policies/Policies.sys.mjs",
WindowsGPOParser: "resource://gre/modules/policies/WindowsGPOParser.sys.mjs", WindowsGPOParser: "resource://gre/modules/policies/WindowsGPOParser.sys.mjs",
macOSPoliciesParser: macOSPoliciesParser:
"resource://gre/modules/policies/macOSPoliciesParser.sys.mjs", "resource://gre/modules/policies/macOSPoliciesParser.sys.mjs",
}); });
XPCOMUtils.defineLazyModuleGetters(lazy, { XPCOMUtils.defineLazyModuleGetters(lazy, {
Policies: "resource:///modules/policies/Policies.jsm",
JsonSchemaValidator: JsonSchemaValidator:
"resource://gre/modules/components-utils/JsonSchemaValidator.jsm", "resource://gre/modules/components-utils/JsonSchemaValidator.jsm",
}); });

@ -102,8 +102,8 @@ export var PoliciesPrefTracker = {
_originalValues: new Map(), _originalValues: new Map(),
start() { start() {
let { PoliciesUtils } = ChromeUtils.import( let { PoliciesUtils } = ChromeUtils.importESModule(
"resource:///modules/policies/Policies.jsm" "resource:///modules/policies/Policies.sys.mjs"
); );
this._originalFunc = PoliciesUtils.setDefaultPref; this._originalFunc = PoliciesUtils.setDefaultPref;
PoliciesUtils.setDefaultPref = this.hoistedSetDefaultPref.bind(this); PoliciesUtils.setDefaultPref = this.hoistedSetDefaultPref.bind(this);
@ -112,8 +112,8 @@ export var PoliciesPrefTracker = {
stop() { stop() {
this.restoreDefaultValues(); this.restoreDefaultValues();
let { PoliciesUtils } = ChromeUtils.import( let { PoliciesUtils } = ChromeUtils.importESModule(
"resource:///modules/policies/Policies.jsm" "resource:///modules/policies/Policies.sys.mjs"
); );
PoliciesUtils.setDefaultPref = this._originalFunc; PoliciesUtils.setDefaultPref = this._originalFunc;
this._originalFunc = null; this._originalFunc = null;

@ -4,8 +4,8 @@
"use strict"; "use strict";
add_task(async function test_simple_policies() { add_task(async function test_simple_policies() {
let { Policies } = ChromeUtils.import( let { Policies } = ChromeUtils.importESModule(
"resource:///modules/policies/Policies.jsm" "resource:///modules/policies/Policies.sys.mjs"
); );
let policy0Ran = false, let policy0Ran = false,

@ -6,8 +6,8 @@
const PREF_DISALLOW_ENTERPRISE = "browser.policies.testing.disallowEnterprise"; const PREF_DISALLOW_ENTERPRISE = "browser.policies.testing.disallowEnterprise";
add_task(async function test_enterprise_only_policies() { add_task(async function test_enterprise_only_policies() {
let { Policies } = ChromeUtils.import( let { Policies } = ChromeUtils.importESModule(
"resource:///modules/policies/Policies.jsm" "resource:///modules/policies/Policies.sys.mjs"
); );
let normalPolicyRan = false, let normalPolicyRan = false,

@ -12,8 +12,8 @@ add_task(async function setup_preferences() {
}); });
add_task(async function test_gpo_policies() { add_task(async function test_gpo_policies() {
let { Policies } = ChromeUtils.import( let { Policies } = ChromeUtils.importESModule(
"resource:///modules/policies/Policies.jsm" "resource:///modules/policies/Policies.sys.mjs"
); );
let gpoPolicyRan = false; let gpoPolicyRan = false;
@ -68,8 +68,8 @@ add_task(async function test_gpo_policies() {
}); });
add_task(async function test_gpo_json_policies() { add_task(async function test_gpo_json_policies() {
let { Policies } = ChromeUtils.import( let { Policies } = ChromeUtils.importESModule(
"resource:///modules/policies/Policies.jsm" "resource:///modules/policies/Policies.sys.mjs"
); );
let gpoPolicyRan = false; let gpoPolicyRan = false;
@ -154,8 +154,8 @@ add_task(async function test_gpo_json_policies() {
}); });
add_task(async function test_gpo_broken_json_policies() { add_task(async function test_gpo_broken_json_policies() {
let { Policies } = ChromeUtils.import( let { Policies } = ChromeUtils.importESModule(
"resource:///modules/policies/Policies.jsm" "resource:///modules/policies/Policies.sys.mjs"
); );
let gpoPolicyRan = false; let gpoPolicyRan = false;

@ -4,8 +4,8 @@ const PREF_DISABLE_SECURITY =
"security.turn_off_all_security_so_that_" + "security.turn_off_all_security_so_that_" +
"viruses_can_take_over_this_computer"; "viruses_can_take_over_this_computer";
const { EnterprisePolicyTesting } = ChromeUtils.import( const { EnterprisePolicyTesting } = ChromeUtils.importESModule(
"resource://testing-common/EnterprisePolicyTesting.jsm" "resource://testing-common/EnterprisePolicyTesting.sys.mjs"
); );
// Setting PREF_DISABLE_SECURITY tells the policy engine that we are in testing // Setting PREF_DISABLE_SECURITY tells the policy engine that we are in testing

@ -9,8 +9,8 @@ const { BrowserUtils } = ChromeUtils.import(
"resource://gre/modules/BrowserUtils.jsm" "resource://gre/modules/BrowserUtils.jsm"
); );
const { EnterprisePolicyTesting } = ChromeUtils.import( const { EnterprisePolicyTesting } = ChromeUtils.importESModule(
"resource://testing-common/EnterprisePolicyTesting.jsm" "resource://testing-common/EnterprisePolicyTesting.sys.mjs"
); );
const { Region } = ChromeUtils.import("resource://gre/modules/Region.jsm"); const { Region } = ChromeUtils.import("resource://gre/modules/Region.jsm");

@ -3,8 +3,8 @@
"use strict"; "use strict";
const { EnterprisePolicyTesting } = ChromeUtils.import( const { EnterprisePolicyTesting } = ChromeUtils.importESModule(
"resource://testing-common/EnterprisePolicyTesting.jsm" "resource://testing-common/EnterprisePolicyTesting.sys.mjs"
); );
const TEST_PATH = getRootDirectory(gTestPath).replace( const TEST_PATH = getRootDirectory(gTestPath).replace(

@ -8,8 +8,8 @@ const { AddonTestUtils } = ChromeUtils.import(
"resource://testing-common/AddonTestUtils.jsm" "resource://testing-common/AddonTestUtils.jsm"
); );
const { EnterprisePolicyTesting } = ChromeUtils.import( const { EnterprisePolicyTesting } = ChromeUtils.importESModule(
"resource://testing-common/EnterprisePolicyTesting.jsm" "resource://testing-common/EnterprisePolicyTesting.sys.mjs"
); );
const TESTPATH = "webapi_checkavailable.html"; const TESTPATH = "webapi_checkavailable.html";

@ -5,8 +5,8 @@
// This test verifies that system addon updates are correctly blocked by the // This test verifies that system addon updates are correctly blocked by the
// DisableSystemAddonUpdate enterprise policy. // DisableSystemAddonUpdate enterprise policy.
const { EnterprisePolicyTesting } = ChromeUtils.import( const { EnterprisePolicyTesting } = ChromeUtils.importESModule(
"resource://testing-common/EnterprisePolicyTesting.jsm" "resource://testing-common/EnterprisePolicyTesting.sys.mjs"
); );
// Setting PREF_DISABLE_SECURITY tells the policy engine that we are in testing // Setting PREF_DISABLE_SECURITY tells the policy engine that we are in testing

@ -11,8 +11,8 @@ const { BackgroundUpdate } = ChromeUtils.import(
); );
let reasons = () => BackgroundUpdate._reasonsToNotUpdateInstallation(); let reasons = () => BackgroundUpdate._reasonsToNotUpdateInstallation();
let REASON = BackgroundUpdate.REASON; let REASON = BackgroundUpdate.REASON;
const { EnterprisePolicyTesting } = ChromeUtils.import( const { EnterprisePolicyTesting } = ChromeUtils.importESModule(
"resource://testing-common/EnterprisePolicyTesting.jsm" "resource://testing-common/EnterprisePolicyTesting.sys.mjs"
); );
const { UpdateService } = ChromeUtils.import( const { UpdateService } = ChromeUtils.import(
"resource://gre/modules/UpdateService.jsm" "resource://gre/modules/UpdateService.jsm"