1774385 - Replace multiple ChromeUtils.defineModuleGetter call with one XPCOMUtils.defineLazyModuleGetters

Status: RESOLVED FIXED

(Toolkit :: Form Autofill, task, P3)

Description

[Dimi Lee [:dimi]]

We use ChromeUtils.defineModuleGetter in a lot of places in the formautofill component (Ex. FormAutofillHeuristics.jsm, FormAutofillChild.jsm, etc)
We can replace this with XPCOMUtils.defineLazyModuleGetters to merge multiple ChromeUtils.defineModuleGetter calls into one XPCOMUtils.defineLazyModuleGetters call.

Comment 1

[Anthony Hung]

Can I work on this?

Comment 2

[Dimi Lee [:dimi]]

(In reply to Anthony Hung from comment #1)

Can I work on this?

Sure, feel free to take this bug

Comment 3

[Anthony Hung]

Attachment: Bug 1774385 - Replace multiple ChromeUtils.defineModuleGetter call with one XPCOMUtils.defineLazyModuleGetters in Form Autofill. r=dimi

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/a33bc3dc504b
Replace multiple ChromeUtils.defineModuleGetter call with one XPCOMUtils.defineLazyModuleGetters in Form Autofill. r=dimi

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/58b4244e5188
fix lint failures in FormAutofillHandler.jsm. CLOSED TREE r=fix

Comment 6

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/a33bc3dc504b
https://hg.mozilla.org/mozilla-central/rev/58b4244e5188