(In reply to Michael Wu [:mwu] from comment #22)
> 1. You should probably use nsAppStartup::Quit because it will do the Right
> Things. This includes sending the proper shutdown notifications to code that
> needs to clean up.

Mmm, good point.

> 2. Should probably make this generic enough to work on all unixish systems
> instead of just GTK2. Sounds like ted's second suggestion does that.

It does, but it runs rather a lot of code from inside the signal
handler, which seems like a Bad Idea.  To make sure things run from
outside the signal handler, we need integration with the event loop.  I
guess we could have nsAppShell::PostSignalEvent(int signum) with a
default do-nothing implementation?

-- 
You received this bug notification because you are a member of Desktop
Packages, which is subscribed to firefox in Ubuntu.
https://bugs.launchpad.net/bugs/73536

Title:
  MASTER Firefox crashes on instant X server shutdown

Status in The Mozilla Firefox Browser:
  Confirmed
Status in “firefox” package in Ubuntu:
  Won't Fix
Status in “firefox-3.0” package in Ubuntu:
  Triaged

Bug description:
  Firefox crashes when X server is forcefully torn down (e.g. by
  pressing ctrl-alt-backspace) and a crash report gets generated on next
  login.

  (Original Report:
  I've reproduced it once on my machine with the following steps. With, oh, 
about 5 tabs open, I just pressed ctrl-alt-backspace, logged back in when the X 
server restarted, and FF crashed with a bug report when Gnome tried to restore 
the session.

  It's not terribly important, I don't think anyone does this very often, but 
maybe it'll be helpful.
  )

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/73536/+subscriptions

-- 
Mailing list: https://launchpad.net/~desktop-packages
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~desktop-packages
More help   : https://help.launchpad.net/ListHelp

Reply via email to