Closed Bug 1696806 Opened 5 years ago Closed 5 years ago

Stop using the JS scriptloader to load scripts which are essentially JS modules

Categories

(MailNews Core :: Feed Reader, task)

task

Tracking

(thunderbird_esr78 wontfix)

RESOLVED FIXED
88 Branch
Tracking Status
thunderbird_esr78 --- wontfix

People

(Reporter: standard8, Assigned: standard8)

References

Details

Attachments

(1 file)

Feed.js, FeedItem.js and feed-parser.js are loaded in FeedUtils.jsm via the subscript loader.

This is the only place they are used, and so loading them via the subscript loader is unnecessary - they should be treated as normal modules.

I'm attaching a patch for this, and to tidy up some of the ESLint global definitions for better detection of missing globals etc.

Blocks: 1696808
Status: NEW → ASSIGNED
Target Milestone: --- → 88 Branch

Pushed by mbanner@mozilla.com:
https://hg.mozilla.org/comm-central/rev/9787fe7a5df6
Stop using the scriptloader to load newsblog feed scripts which essentially modules. r=mkmelin

Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: