Closed Bug 1773469 Opened 2 years ago Closed 2 years ago

Change about:protections module files to use mjs extensions.

Categories

(Firefox :: Protections UI, task)

task

Tracking

()

RESOLVED FIXED
103 Branch
Tracking Status
firefox103 --- fixed

People

(Reporter: standard8, Assigned: standard8)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

With the transition to using ES modules rather than JSMs getting started, we are now using the .mjs extension for those files.

I'd like to transition our content code that is using modules to use the .mjs extension as well - the primary gains of this are that we a) don't need to manually specify sourceType: module for ESLint, b) this gives a visual distinction to developers that the file is a module rather than a regular JavaScript file.

This bug is for the transition of the about:protections code.

Pushed by mbanner@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/65833ced5638 Change about:protections module files to use mjs extensions. r=pbz

Backed out 2 changesets (bug 1773469, bug 1773468) for causing browser-chrome failures in test/static/browser_all_files_referenced.js

Backout link: https://hg.mozilla.org/integration/autoland/rev/4cd8570b0fd1ff8d2c4569199893c0a2e76afecf

Push with failures

Failure log

INFO - TEST-UNEXPECTED-FAIL | browser/base/content/test/static/browser_all_files_referenced.js | there should be no unreferenced files - Got 2, expected +0
Flags: needinfo?(standard8)
Depends on: 1559554
Flags: needinfo?(standard8)
Pushed by mbanner@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/7f320fa4a227 Change about:protections module files to use mjs extensions. r=pbz
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 103 Branch
Blocks: 1776406
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: