Open Bug 1674799 Opened 5 years ago Updated 4 years ago

remove externalExtensionLoaders

Categories

(Toolkit :: Add-ons Manager, task, P3)

task

Tracking

()

People

(Reporter: mixedpuppy, Unassigned)

References

(Depends on 1 open bug)

Details

The addon manager still has a way to define external addon loaders, but the only thing it is used for is tests which appear to at some level test legacy extension support. We should rework those tests as necessary and remove the external loader code.

This was originally added for Thunderbird/Seamonkey, are they not using it?

(In reply to Andrew Swan [:aswan] from comment #1)

This was originally added for Thunderbird/Seamonkey, are they not using it?

I was going to ping them asking about it. I suspected there was a chance they might be using it.

If they're not using it, then all the strictCompatibility handling in the addon manager (and its related tests etc) can all go too.

Do you know if TB is making use of externalExtensionLoaders? I don't see any references in com-central on searchfox.

Flags: needinfo?(mkmelin+mozilla)

I don't think so. Thunderbird now only supports web extensions and experiments, which technically are WX. John, can you confirm?

Flags: needinfo?(mkmelin+mozilla) → needinfo?(john)

I have checked our code, our test documentation and got feedback from Rob: We do not use that. So it could go away.

Flags: needinfo?(john)
Depends on: 1678078
You need to log in before you can comment on or make changes to this bug.