Winhooks (under xpfe/components/winhooks) appears to implement three command line arguments based on the old xpfe commandline handler: -setDefaultMail -unsetDefaultMail -setDefaultBrowser I'm guessing we probably don't have equivalents elsewhere but they need testing at least.

Since comment #0, more than a year has gone by. Should this bug fall into oblivion?

(In reply to comment #1) > Since comment #0, more than a year has gone by. Should this bug fall into > oblivion? > No, please stop making useless comments, see the dependent bug, that needs to be completed first (at least on Windows).

Now Frank has the UI in place, its time for this to die. Here's the comm-central changes, including one hook change for the setting of the desktop background (note there is other code in that file which means it is only enabled on windows). I think the pref-winhooks* code is no longer used, but I'll let someone else deal with that.

Here's the mozilla-central changes. Dropping winhooks means we can get rid of that nasty jar.mn in xpfe/components and put it in the autocomplete directory instead making it all much simpler.

(In reply to comment #3) > (note there is other code in that file which means it is only enabled on windows) ... which is now the wrong test, of course ...

Comment on attachment 336828 [details] [diff] [review] [checked in] Drop winhooks (comm-central changes) Is there a bug on fixing the wallpaper enabling code to check for the shell service instead?

(In reply to comment #3) > I think the pref-winhooks* code is no longer used Agreed. > but I'll let someone else deal with that. Does that include filing the bug? ;-)

Comment on attachment 336829 [details] [diff] [review] [checked in] Drop winhooks (mozilla-central changes) >diff --git a/xpfe/components/build/appcomps.pkg b/xpfe/components/build/appcomps.pkg .pkg files were born dead and have never come to life, so it's usually not worth updating them.

(In reply to comment #3) > I think the pref-winhooks* code is no longer used, but I'll let someone else > deal with that. Should be easy to clean up though...

This additional patch kills pref-winhooks. I have no idea how it was included in the prefwindow, I find no reference to it.

Comment on attachment 336865 [details] [diff] [review] [checked in] kill pref-winhooks (In reply to comment #10) >I have no idea how it was included in the prefwindow, I find no reference to it. Mnyromyr removed platformPrefOverlay.xul in bug 394522. >- "chrome://communicator/content/pref/pref-winhooks.xul": "advanced_pref_system", You don't really need to change this, it won't break the rest of the legacy prefwindow and it will get removed along with it anyway.

Comment on attachment 336828 [details] [diff] [review] [checked in] Drop winhooks (comm-central changes) Checked in, changeset id 258:4e64b9298751

Comment on attachment 336829 [details] [diff] [review] [checked in] Drop winhooks (mozilla-central changes) Checked in, changeset id: 18800:2e8e583fc354

Comment on attachment 336865 [details] [diff] [review] [checked in] kill pref-winhooks pushed removal of pref-winhooks as http://hg.mozilla.org/comm-central/rev/94c0142a32ac Is that bug done now?

(In reply to comment #6) > (From update of attachment 336828 [details] [diff] [review]) > Is there a bug on fixing the wallpaper enabling code to check for the shell > service instead? Bug 454017. (In reply to comment #14) > (From update of attachment 336865 [details] [diff] [review]) > pushed removal of pref-winhooks as > http://hg.mozilla.org/comm-central/rev/94c0142a32ac > > Is that bug done now? Yes.

Pushed by frgrahl@gmx.net: https://hg.mozilla.org/comm-central/rev/d417e8132c48 Winhooks commandline options won't work in suiterunner. r/sr=Neil