Closed Bug 1609885 Opened 5 years ago Closed 3 years ago

Port bug 1609271 to Thunderbird: Stop using ChromeUtils.import(..., null) which returns the global object for a module instead of its exports

Categories

(Thunderbird :: Upstream Synchronization, task, P3)

Tracking

(thunderbird_esr91 wontfix)

RESOLVED FIXED
Thunderbird 74.0
Tracking Status
thunderbird_esr91 --- wontfix

People

(Reporter: mkmelin, Assigned: mkmelin)

References

Details

Attachments

(1 file)

Port bug 1609271: Stop using ChromeUtils.import(..., null)

https://phabricator.services.mozilla.com/D60158 has a patch to enable an eslint rule to find offenders.

The linting rule just landed. There were 9 failures. I've fixed 6 and disabled the rule on the others in:

  • mail/components/enterprisepolicies/tests/browser/browser_policies_setAndLockPref_API.js
  • mail/components/enterprisepolicies/tests/xpcshell/test_proxy.js
  • mail/components/enterprisepolicies/tests/xpcshell/test_runOnce_helper.js

We should wait and see what happens to the Firefox version of these files, then copy it.

Keywords: leave-open
Pushed by geoff@darktrojan.net: https://hg.mozilla.org/comm-central/rev/06b59eceaa11 Port bug 1609271 to Thunderbird: Stop using ChromeUtils.import(..., null). rs=linting DONTBUILD
Priority: P1 → P3
Component: General → Upstream Synchronization
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 74.0

Bug bug 1609271 isn't finished yet, so this is also not complete.

Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Summary: Port bug 1609271 to Thunderbird: Stop using ChromeUtils.import(..., null) which returns the global object for a module instead of its exports → Port bug 1609271 to Thunderbird: Stop using ChromeUtils.import(..., null) which returns the global object for a module instead of its exports
Assignee: nobody → mkmelin+mozilla

https://hg.mozilla.org/comm-central/rev/2133b49b165b73d0dbc30385f951989bcc5a1610 fixed the remaining bits.
I'll update the patch to only have the pieces it did not cover

Attachment #9249573 - Attachment description: Bug 1609885 - Fix remaining Thunderbird violations for ChromeUtils.import(..., null). r=john.bieling → Bug 1609885 - fix remaining m-c syncronization from bug 1733549 and bug 1574475. r=john.bieling
Pushed by mkmelin@iki.fi: https://hg.mozilla.org/comm-central/rev/2b6b7de71b93 fix remaining m-c syncronization from bug 1733549 and bug 1574475. r=john.bieling
Status: REOPENED → RESOLVED
Closed: 4 years ago3 years ago
Resolution: --- → FIXED
Keywords: leave-open
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: