Closed Bug 1758107 Opened 3 years ago Closed 3 years ago

Stop using ChromeUtils.import(..., null) for importing modules in dom/push

Categories

(Core :: DOM: Notifications, task)

task

Tracking

()

RESOLVED FIXED
100 Branch
Tracking Status
firefox100 --- fixed

People

(Reporter: standard8, Assigned: standard8)

References

Details

Attachments

(3 files)

There's a few instances remaining in dom/push/ where we're still passing null as the second argument.

I'm going to clean those up here.

Also tidy up how imports are managed.

Depends on D140321

Pushed by mbanner@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/6cdb5d20a02a Stop using ChromeUtils.import(..., null) for importing modules in dom/push/test/xpcshell/head.js. r=mt https://hg.mozilla.org/integration/autoland/rev/62a245d1fa5d Stop using ChromeUtils.import(..., null) for importing modules in dom/push/. r=mt https://hg.mozilla.org/integration/autoland/rev/e2acbe11b6a2 Fix some ESLint no-unused-vars warnings in dom/push/test. r=mt
Regressions: 1761022
Regressions: 1761023
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 100 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: