99553 - Crash while closing the last browser window

Status: VERIFIED DUPLICATE of bug 88844

(SeaMonkey :: General, defect)

Description

[Sean Su]

When installing the browser, make sure Turbo is selected to be set.  After the 
browser window starts up at the end of the installation, close it.  The crash 
happens before the Quick Launch information dialog is shown.

This seems to only happen in debug builds on the branch.  Optimized builds work 
fine.

I have a debug branch build that can reproduce this in my cube.

If, while the browser is running, Turbo is disabled, then closing the last 
browser window does not trigger the crash.

Stack trace coming up.

Comment 1

[Sean Su]

Attachment: stack trace of crash.

Comment 2

[Suresh Duddi (gone)]

Marking TFV 0.9.4 since this looks like would stop the branch ship and to get it
on the radar. (Sorry Bill, I know I shouldn't touch TFV but I didn't know any
better way).

Comment 3

[chris hofmann]

I think this is a dup of other bug(s)...

flagging as nsbranch+ until we make sure.

Comment 4

[Blake Ross]

Suresh, why is a debug-only crash a stop ship?

Comment 5

[Suresh Duddi (gone)]

If the cause if data corruption, it can manifest itself differently when say the
third window is closed. I thought we should analyze the reason for the crash and
then satisfy ourselves that this wont affect optimized builds too in any way. We
shouldnt be making that assumption without the analysis.

Comment 6

[Bill Law]

I am very hopeful that dp's latest patch for the remaining difficulty with bug
88844 will fix this as well.  But, we talked about this and dp said that his
earlier patch was applied to Sean's machine and they still were seeing the bug.
 So I don't know.

Only able to reproduce (so far as I know) on Sean's machine, debug mode.  We'll
apply dp's latest patch on that machine and see if the problem is still present,
on Monday.

Comment 7

[Sean Su]

I've already tried it.  It no longer crashes, but it does assert in debug mode.

Comment 8

[Asa Dotzler [:asa]]

0.9.4 is out the door.

Comment 9

[Suresh Duddi (gone)]

yeah! as sean said, it no longer crashes with the patch from 88844. Thats
progress...

Comment 10

[Bill Law]

Ok, then.  What to do with this bug?

1 Remove "nsbranch+" (change to "nsbranch"?

2 Make it a dup of 88844?

3 Dependent on 88844?

I vote for 2.

*** This bug has been marked as a duplicate of 88844 ***

Comment 11

[Grace Bush]

verified