Closed Bug 1775011 Opened 2 years ago Closed 2 years ago

Replace multiple ChromeUtils.defineModuleGetter call with one XPCOMUtils.defineLazyModuleGetters

Categories

(Toolkit :: Password Manager, task, P3)

task

Tracking

()

RESOLVED FIXED
103 Branch
Tracking Status
firefox103 --- fixed

People

(Reporter: dlee, Assigned: janikaneuberger, Mentored)

References

(Blocks 1 open bug)

Details

(Keywords: good-first-bug)

Attachments

(1 file, 1 obsolete file)

We use ChromeUtils.defineModuleGetter in a lot of places in the password maanger component.
We can replace this with XPCOMUtils.defineLazyModuleGetters to merge multiple ChromeUtils.defineModuleGetter calls into one XPCOMUtils.defineLazyModuleGetters call.

Just to note that some of the work I'm doing in bug 1775028 will remove some unused imports, which will probably make some of this simpler.

Attachment #9282750 - Attachment is obsolete: true
Pushed by dlee@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/aa54436f9dac
Replaced multiple ChromeUtils.defineModuleGetter calls with one XPCOMUtils.defineLazyModuleGetters, r=dimi
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 103 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: