Skip to content

Commit d016a2b

Browse files
committed
Bug 1839388 - Convert distribution.js to an ES module. r=mkaply
Differential Revision: https://phabricator.services.mozilla.com/D181578
1 parent 0ab0b67 commit d016a2b

File tree

5 files changed

+11
-14
lines changed

5 files changed

+11
-14
lines changed

browser/components/BrowserGlue.sys.mjs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -898,8 +898,8 @@ export function BrowserGlue() {
898898
);
899899

900900
XPCOMUtils.defineLazyGetter(this, "_distributionCustomizer", function () {
901-
const { DistributionCustomizer } = ChromeUtils.import(
902-
"resource:///modules/distribution.js"
901+
const { DistributionCustomizer } = ChromeUtils.importESModule(
902+
"resource:///modules/distribution.sys.mjs"
903903
);
904904
return new DistributionCustomizer();
905905
});

browser/components/distribution.js renamed to browser/components/distribution.sys.mjs

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -2,24 +2,21 @@
22
* License, v. 2.0. If a copy of the MPL was not distributed with this
33
* file, You can obtain one at http://mozilla.org/MPL/2.0/. */
44

5-
var EXPORTED_SYMBOLS = ["DistributionCustomizer"];
6-
75
const DISTRIBUTION_CUSTOMIZATION_COMPLETE_TOPIC =
86
"distribution-customization-complete";
97

108
const PREF_CACHED_FILE_EXISTENCE = "distribution.iniFile.exists.value";
119
const PREF_CACHED_FILE_APPVERSION = "distribution.iniFile.exists.appversion";
1210

13-
const { AppConstants } = ChromeUtils.importESModule(
14-
"resource://gre/modules/AppConstants.sys.mjs"
15-
);
11+
import { AppConstants } from "resource://gre/modules/AppConstants.sys.mjs";
12+
1613
const lazy = {};
1714
ChromeUtils.defineESModuleGetters(lazy, {
1815
AddonManager: "resource://gre/modules/AddonManager.sys.mjs",
1916
PlacesUtils: "resource://gre/modules/PlacesUtils.sys.mjs",
2017
});
2118

22-
function DistributionCustomizer() {}
19+
export function DistributionCustomizer() {}
2320

2421
DistributionCustomizer.prototype = {
2522
// These prefixes must only contain characters

browser/components/moz.build

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
with Files("**"):
88
BUG_COMPONENT = ("Firefox", "General")
99

10-
with Files("distribution.js"):
10+
with Files("distribution.sys.mjs"):
1111
BUG_COMPONENT = ("Firefox", "Distributions")
1212

1313
with Files("tests/**"):
@@ -92,7 +92,7 @@ EXTRA_COMPONENTS += [
9292
EXTRA_JS_MODULES += [
9393
"BrowserContentHandler.sys.mjs",
9494
"BrowserGlue.sys.mjs",
95-
"distribution.js",
95+
"distribution.sys.mjs",
9696
]
9797

9898
if CONFIG["MOZ_DEBUG"] or CONFIG["MOZ_DEV_EDITION"] or CONFIG["NIGHTLY_BUILD"]:

browser/components/places/tests/unit/test_browserGlue_distribution.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -47,8 +47,8 @@ registerCleanupFunction(function () {
4747
});
4848

4949
add_task(async function () {
50-
let { DistributionCustomizer } = ChromeUtils.import(
51-
"resource:///modules/distribution.js"
50+
let { DistributionCustomizer } = ChromeUtils.importESModule(
51+
"resource:///modules/distribution.sys.mjs"
5252
);
5353
let distribution = new DistributionCustomizer();
5454

browser/components/tests/unit/test_distribution_cachedexistence.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,8 +23,8 @@ add_task(async function () {
2323
Services.prefs.clearUserPref(PREF_CACHED_FILE_EXISTENCE);
2424
setupTest();
2525

26-
let { DistributionCustomizer } = ChromeUtils.import(
27-
"resource:///modules/distribution.js"
26+
let { DistributionCustomizer } = ChromeUtils.importESModule(
27+
"resource:///modules/distribution.sys.mjs"
2828
);
2929
let distribution = new DistributionCustomizer();
3030

0 commit comments

Comments
 (0)