Bug 1814104 Comment 31 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

Original comment by
on 
(In reply to Jens Stutte [:jstutte] from comment #29)
> I wonder if we should just explicitly wait for all the startup promises to have finished [at the beginning of `shutdown`](https://searchfox.org/mozilla-central/rev/df2a7bff8fa01ebc6292e68ce7e3c7b7cf8fc33c/toolkit/components/extensions/Extension.jsm#3307) rather than hoping to be able to abort the startup at any time?

Oh, that [seems already to be the case?](https://searchfox.org/mozilla-central/rev/0e3779abe6ad44feb787a295fbb67bc9fe4e927a/toolkit/mozapps/extensions/internal/XPIProvider.jsm#1991)
Revision 1 by
on 
(In reply to Jens Stutte [:jstutte] from comment #29)
> I wonder if we should just explicitly wait for all the startup promises to have finished [at the beginning of `shutdown`](https://searchfox.org/mozilla-central/rev/df2a7bff8fa01ebc6292e68ce7e3c7b7cf8fc33c/toolkit/components/extensions/Extension.jsm#3307) rather than hoping to be able to abort the startup at any time?

Oh, that [seems already to be the case?](https://searchfox.org/mozilla-central/rev/0e3779abe6ad44feb787a295fbb67bc9fe4e927a/toolkit/mozapps/extensions/internal/XPIProvider.jsm#1991) But IIUC we wait there blocking the `quitApplicationGranted` barrier rather than returning a promise we wait for in `profileChangeTeardown`. So maybe we should move that `await` into the async execution part?

Back to Bug 1814104 Comment 31