Closed Bug 1792437 Opened 2 years ago Closed 2 years ago

Remove nsIModule

Categories

(Core :: XPCOM, task, P3)

task

Tracking

()

RESOLVED FIXED
107 Branch
Tracking Status
firefox107 --- fixed

People

(Reporter: mccr8, Assigned: mccr8)

References

Details

Attachments

(2 files)

Some of the XPCOM component improvements must have removed the use of this, because it doesn't look like it does anything. Let's remove it.

(The interface still exists in Thunderbird, but it is just a useless extra interface in a couple of QIs, so it should be trivially fixable.)

nsIComponentRegistrar and nsIServiceManager look like they can be easily marked builtinclass, so I'll do that at the same time.

nsIClassInfo has some non-trivial uses (including JS implementations) in Calendar, so I'll leave that alone. There are also a few trivial uses in mozilla-central in JS, but nothing important.

See Also: → 1792438

I think some prior XPCOM component work removed the uses of this.

Pushed by amccreight@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/375d7f62699f
part 1 - Mark some xpcom/components interfaces builtinclass. r=xpcom-reviewers,nika
https://hg.mozilla.org/integration/autoland/rev/7a43d6de56d1
part 2 - Remove nsIModule. r=xpcom-reviewers,nika
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 107 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: