98732 - Message compose window steals other windows

Status: VERIFIED FIXED

(SeaMonkey :: MailNews: Message Display, defect)

Description

[Tracy Walker [:tracy]]

seen on commercial trunk builds: (note: branch is okay)

windows 2001-09-07-05-trunk
linux 2001-09-07-08-trunk
mac 2001-09-07-08-trunk

-open mail app
-reply to a message or start a new message

the resulting compose window takes over the last Browser window....if a browser 
wasn't open it steals the mail app window. 

-compose and send the message.

what ever window that was stolen is now gone altogether with the mail compose 
window closing.

a similar behaviior can be seen by running through the XPInstall smoketest:
-click on the XPinstall link at this page 
http://slip/projects/seamonkey/smoketests/pre_checkin_trigger.html
-then click install

the resulting dialogue steals the browser window (normally it opens a small 
dialogue window of its own.  in this case the dialogue can't be dismissed and 
the window is hung.

I know the summary I wrote here is not accurate.   please update it to reflect 
what is really happening, thanks.

Comment 1

[Asa Dotzler [:asa]]

browser general is the worst place to put a blocker bug. ->mailnews.

Comment 2

[(not reading, please use [e-mail] instead)]

investigating...

do any other windows steal focus like this?

Comment 3

[William Stuart]

I am seeing this bug in win2k, 2001090703, makes it impossible to reply to mail
as well...

Comment 4

[(not reading, please use [e-mail] instead)]

whoa, here is what I'm seeing:

start mozilla to about:blank, do "File | New | Message", and the browser window
seems to be *replaced* with a compose window!

twalker, is that what you are seeing?

Comment 5

[(not reading, please use [e-mail] instead)]

cc'ing danm, might be related to his changes.

Comment 6

[(not reading, please use [e-mail] instead)]

backing out danm fixes the problem.

I backed out these check ins:

cvs update -j1.32 -j1.31 mozilla/xpfe/appshell/src/nsXULWindow.h
cvs update -j1.75 -j1.74 mozilla/xpfe/appshell/src/nsXULWindow.cpp
cvs update -j1.11 -j1.10 mozilla/xpfe/appshell/src/nsAppShellService.h
cvs update -j1.159 -j1.158 mozilla/xpfe/appshell/src/nsAppShellService.cpp

I'll continue to narrow it down.

Comment 7

[(not reading, please use [e-mail] instead)]

these are the changes that caused this:

cvs update -j1.11 -j1.10 mozilla/xpfe/appshell/src/nsAppShellService.h
cvs update -j1.159 -j1.158 mozilla/xpfe/appshell/src/nsAppShellService.cpp

Comment 8

[(not reading, please use [e-mail] instead)]

I backed out danm's changes to nsAppShellService.h and nsAppShellService.cpp 
for #97514.

Comment 9

[(not reading, please use [e-mail] instead)]

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

Comment 10

[Tracy Walker [:tracy]]

verified fixed in Loan's respin of linux trunk build 2001-09-07-12-trunk

Comment 11

[John Morrison]

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

Comment 12

[R.K.Aa.]

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

Comment 13

[John Morrison]

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

Comment 14

[jay garcia]

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

Comment 15

[Olivier Cahagne]

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

Comment 16

[R.K.Aa.]

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

Comment 17

[Tuukka Tolvanen (sp3000)]

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

Comment 18

[R.K.Aa.]

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

Comment 19

[Jesse Houwing]

I'm seeing this in the september 10 build. This should include this fix. reopen?

Comment 20

[Boris Zbarsky [:bzbarsky]]

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