Closed Bug 1817460 Opened 2 years ago Closed 2 years ago

Push service should support ES modules

Categories

(Core :: DOM: Push Subscriptions, task)

task

Tracking

()

RESOLVED FIXED
112 Branch
Tracking Status
firefox112 --- fixed

People

(Reporter: standard8, Assigned: standard8)

References

Details

(Whiteboard: [esmification-timeline])

Attachments

(1 file)

In bug 1816934 I am converting services/settings to ES modules. I noticed that the push service only supports legacy JSM modules.

Whilst we can still use the old format name to reference the modules, we should migrate the code to use importESModules.

In bug 1816934, I am going to hard-code the existing URI so that it will keep working, it should be replaced by import.meta.url as part of this transition (the approximate location of the code is here).

Assignee: nobody → standard8
Status: NEW → ASSIGNED
Pushed by mbanner@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/6d8732b3ddd0 Change PushBroadcastService to handle its listeners calling directly into ES modules. r=mt
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 112 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: