967839 - onProfileShutdown work in BrowserGlue doesn't really have to happen at profile shutdown

Status: RESOLVED FIXED

(Firefox :: General, defect)

Description

[Marco Bonardo [:mak]]

in bug 824433 I have a crash (explained in bug 967833 and https://bugzilla.mozilla.org/show_bug.cgi?id=824433#c36) that is actually caused by WebAppsUI.uninit() being called in profile-before-change.

So, while looking at that I figured that none of the stuff we have in _onProfileShutdown should actually happen at profile shutdown, it's all UI stuff that can and should shutdown earlier. Ideally should happen when the UI goes away, but since we can't rely on last-window-closed on Mac, we can likely use quit-application-granted.
This happened mostly cause it looks usual to put shutdown stuff in a nicely named shutdown method, so people just piled up stuff in _onProfileShutdown.

I'm going to move everything to q-a-g and remove the onProfileShutdown pitfall from browserGlue. This actually matters cause xpcshell is supposed to have a profile, but not a UI, stuff that controls the ui should generally not happen at profile shutdown.

Comment 1

[Marco Bonardo [:mak]]

Attachment: patch v1

Comment 2

[Marco Bonardo [:mak]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/9fb9effa74a6

Comment 3

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/9fb9effa74a6