29733 - App doesn't quit (JS Error) if quitting from the compose window

Status: VERIFIED WORKSFORME

(SeaMonkey :: UI Design, defect, P1)

Description

[Mike Pinkerton (not reading bugmail)]

- launch mozilla and get a browser window
- click the composer icon in the main window
- hit cmd-q to quit.

expected results:
- app quits

actual results:
- browser and editor windows close but a JS error is encountered which stops the 
shutdown process:

JavaScript Error: null
URL: chrome://global/content/globalOverlay.js
LineNo: 33

Comment 1

[Kathleen :Brade]

Simon found out that this is also broken on Linux and I think it was Charley who 
told me Windows also has this problem.

Comment 2

[Kathleen :Brade]

*** Bug 30104 has been marked as a duplicate of this bug. ***

Comment 3

[Kathleen :Brade]

*** Bug 32071 has been marked as a duplicate of this bug. ***

Comment 4

[don]

Ben, can you figure who should get this bug?  Travis maybe?

Comment 5

[Ben Goodger (use ben at mozilla dot org for email)]

don says to give to travis...

apparently failing in the QI to nsIAppShellService

Comment 6

[Simon Fraser [no longer active]]

Is this really travis? It seems more like a bill law xpapps problem to me.

Comment 7

[Simon Fraser [no longer active]]

This is quite tricky. It seems that after all windows have been closed in the 
call to goQuitApplication(), the JS context in which the script is running has 
gone away, so we get a failure on the next line of the script (even if it's just 
a dump()).

Perhaps we need to call into C++ to quit?

Comment 8

[don]

Move to M16 for now ...

Comment 9

[Mike]

M16 has been out for a while now, these bugs target milestones need to be 
updated.

Comment 10

[Judson Valeski]

I'm not seeing this on windows. is is still a problem?

Comment 11

[Simon Fraser [no longer active]]

I believe this works now. sujay, please test.

Comment 12

[Dawn Endico]

Reassigning all travis bugs to Valeski for triage. Don thought these were
all docshell related. Travis is no longer at netscape so these bugs are
unowned.

I'm reassigning 23 bugs to Valeski right now. To search for them,
search for "BUGSFORMERLYKNOWNASTRAVIS" in the description. The url below 
should work.

http://bugzilla.mozilla.org/buglist.cgi?bug_status=UNCONFIRMED&bug_status=NEW&bug_status=ASSIGNED&bug_status=REOPENED&email1=&emailtype1=substring&emailassigned_to1=1&email2=&emailtype2=substring&emailreporter2=1&bugidtype=include&bug_id=&changedin=&votes=&chfieldfrom=&chfieldto=Now&chfieldvalue=&short_desc=&short_desc_type=substring&long_desc=BUGSFORMERLYKNOWNASTRAVIS&long_desc_type=substring&bug_file_loc=&bug_file_loc_type=substring&status_whiteboard=&status_whiteboard_type=substring&keywords=&keywords_type=anywords&field0-0-0=noop&type0-0-0=noop&value0-0-0=&cmdtype=doit&newqueryname=&order=Reuse+same+sort+as+last+time

Comment 13

[Paul McGarry]

worksforme
Linux 2000091021

Comment 14

[Gervase Markham [:gerv]]

Me too - 20001023MN6 Win95.

Gerv

Comment 15

[sujay]

verified in 10/23 build.