Closed Bug 967839 Opened 9 years ago Closed 9 years ago
Profile Shutdown work in Browser Glue doesn't really have to happen at profile shutdown
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.
Target Milestone: --- → Firefox 30
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.