Open Bug 1370658 Opened 4 years ago Updated 4 years ago

Components.manager.addBootstrappedManifestLocation() can do main thread IO at startup

Categories

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

enhancement

Tracking

()

People

(Reporter: ehsan.akhgari, Unassigned)

References

(Blocks 1 open bug)

Details

(Whiteboard: [fxperf:p3])

Flags: needinfo?(rhelmer)
Assignee: nobody → rhelmer
Flags: needinfo?(rhelmer)
Depends on: 1363482
Assignee: rhelmer → nobody
Whiteboard: [fxperf]
Priority: -- → P3
Hey kmag, how much of this was covered by bug 1363482? It looks like this should be WFM now, but I don't want to jump the gun on that.

Giving it [fxperf:p3] for now to at least get it out of triage.
Flags: needinfo?(kmaglione+bmo)
Whiteboard: [fxperf] → [fxperf:p3]
(In reply to Doug Thayer [:dthayer] from comment #1)
> Hey kmag, how much of this was covered by bug 1363482? It looks like this
> should be WFM now, but I don't want to jump the gun on that.

Unfortunately none of it was covered by bug 1363482. It would help with system add-ons if they were bundled in OmniJar, but they're currently not. And in unpacked local builds, where they're loaded from the filesystem. In release builds, for the moment anyway, it only helps with the handful of manifests that we bundle in omni.ja.

At this point, though, we should really just stop loading chrome manifests from bootstrapped extensions. Our system extensions can be updated to register things without them, and no other extensions need them.
Flags: needinfo?(kmaglione+bmo)
You need to log in before you can comment on or make changes to this bug.