523064 - Is there a good way to close background firefox properly?

Status: RESOLVED FIXED

(Firefox :: General, defect, P1)

Description

[nicksu]

User-Agent:       Mozilla/4.0 (compatible; MSIE 8.0; Windows NT 6.0; Trident/4.0; WPS; GTB6; SLCC1; .NET CLR 2.0.50727; Media Center PC 5.0; eSobiSubscriber 2.0.4.16; .NET CLR 3.5.30729; .NET CLR 3.0.30618)
Build Identifier: Mozilla/5.0 WinCE6.0 rv1.9.2a2 Gecko/20090920 Firefox/3.6a2


We've experimented with closing firefox today, but couldn't find a reliable way to do it. 
The firefoxfaststart creates a window with class "MozillaHiddenWindowClas". 
If we send it a WM_DESTROY event it causes it to crash and displays an error message to the user. 
We can call TerminateProcess on it, 
but that doesn't seem like a very good idea because it doesn't give firefox a chance to properly shutdown. 

Reproducible: Always

Comment 1

[Justin Dolske [:Dolske]]

Closing the app from File -> Exit will terminate the background FastStart process. Or are you looking for a way to do this from code outside the app?

Comment 2

[nicksu]

Yes, We are looking for a way to do this from code outside the app.
We have an application A.
When A launched, it needs a way to close FastStart process at the same time.
Before A closed, it will launch faststart again.

Comment 4

[Vladimir Vukicevic [:vlad] [:vladv] (needinfo me, slow to respond)]

Un-dup'ing this -- this is a separate issue from bug 522625, where we don't have a message for this.   On the desktop, we use WM_QUERYSESSION/WM_ENDSESSION, but those aren't available on CE.  I'm testing a patch currently that will make WM_QUIT behave like WM_ENDSESSION on CE, in which case sending a WM_QUIT to Firefox should do a safe controlled shutdown.

Comment 5

[Vladimir Vukicevic [:vlad] [:vladv] (needinfo me, slow to respond)]

Attachment: MOZ_WM_APP_QUIT

This defines MOZ_WM_APP_QUIT, which is (WM_APP+0x0300).  It basically signals the app to exit, similar to the desktop QUERYENDSESSION stuff, but that's not available on CE.

Comment 6

[Vladimir Vukicevic [:vlad] [:vladv] (needinfo me, slow to respond)]

http://hg.mozilla.org/releases/mozilla-1.9.2/rev/fb5c9594a8f1
http://hg.mozilla.org/mozilla-central/rev/3fa88ff37b2c

Comment 7

[[e-mail]]

[Side note: Years ago we used to have a command-line option to do this, so that you could invoke $APP -KillAll to close an existing instance.]