1771444 - Streamline the shutdown sequence in nsAppStartup::Quit

Status: NEW

(Core :: XPCOM, task)

Description

[Jens Stutte [:jstutte]]

The shutdown sequence is in nsAppStartup::Quit. AFAIU it first checks if all windows can be closed.

Then we have the following sequence:

1. notify "quit-application-granted" (which can block the main thread for a while)
2. CloseAllWindows();
3. AppShutdown::OnShutdownConfirmed();
4. AppShutdown::AdvanceShutdownPhase(AppShutdownConfirmed);

It looks to me as if 3. and 4. should happen instead of 1. (as they notify, too), followed by 2. in order to make "quit-application-granted" covered by the terminator's timer. There is a state machine build on ferocity that might complicate this change a bit, though.