Currently xpfe/components/killAll/ gets compiled for all apps that build in xpfe/components/ and have MOZ_HAVE_BROWSER defined. But nsKillAll.js mainly implements nsICmdLineHandler which is only built for apps where MOZ_XUL_APP is not defined. So I think we should either remove nsKillAll.js from the apps which build with MOZ_XUL_APP defined or fix it so it works with the toolkit/ commandline handler. I don't know about the code under http://lxr.mozilla.org/mozilla/source/toolkit/xre/nsNativeAppSupportWin.cpp#581, it's supposed to catch WM_QUERYENDSESSION messages and then send -killAll to the internal command line handler...?

(In reply to comment #0) > Currently xpfe/components/killAll/ gets compiled for all apps that build in > xpfe/components/ and have MOZ_HAVE_BROWSER defined. But nsKillAll.js mainly > implements nsICmdLineHandler which is only built for apps where MOZ_XUL_APP is > not defined. Which I think comes down to xulrunner, FireFox and SeaMonkey. Firefox doesn't package it. > So I think we should either remove nsKillAll.js from the apps which build with > MOZ_XUL_APP defined or fix it so it works with the toolkit/ commandline > handler. I don't know about the code under > http://lxr.mozilla.org/mozilla/source/toolkit/xre/nsNativeAppSupportWin.cpp#581, > it's supposed to catch WM_QUERYENDSESSION messages and then send -killAll to > the internal command line handler...? Firefox have just reimplemented this function (bug 333907) so it doesn't use the -killAll switch any more (which hasn't worked for them for a long time anyway...). Note at a quick glance I think bug 333907 will have fixed bug 212316 and possibly bug 212251 for trunk at least, but it'd be good if we could test that. So I vote that we just drop nsKillAll.js rather than re-implement it as I don't see a good use for it.

Removes nsKillAll.js - like I said in comment 1, Firefox doesn't actually package it (though it builds it), SeaMonkey packages it but it no longer works and isn't needed. From checking mxr, Camino doesn't appear to need it either, so this patch just drops the relevant items that we no longer need.

Comment on attachment 273021 [details] [diff] [review] Remove nsKillAll.js I didn't realise -killAll was for WM_QUERYENDSESSION, I had always thought it was either for xpinstall (to close an existing instance) or to exit turbo mode.

Updating title to reflect reality. Patch checked in (I also removed the .cvsignore file so the directory will be deleted).