Ruediger Lahl Reporter
	Description • 19 years ago

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a1) Gecko/20051107 SeaMonkey/1.5a
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a1) Gecko/20051107 SeaMonkey/1.5a

After you open the Composer to answer a posting(its not necessary to send the answer), SeaMonkey crash when you close SeaMonkey after that.

Reproducible: Always

Steps to Reproduce:
1.Open MailNews
2.Select a mail/posting
3.Klick on 'Reply'
4.Leave the composer after that(sending an answer is not necessary)
5.Close Seamonkey

Actual Results:  
Crash after the SeaMonkey-Window allready is closed

Expected Results:  
no crash

Linux-builds are allso affected.

	Tobias Fischer
	Comment 1 • 19 years ago

Hmm. I can confirm this crash using: 

Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9a1) Gecko/20051107 SeaMonkey/1.5a Mnenhy/0.7.2.10015 {Build ID: 2005110710}

Got an TB-Incident: TB11568018K but it was not processed at moment, so I cant add the Stack.

	Stephen Donner [:stephend] Not actively reading bugmail
	Comment 2 • 19 years ago

Confirming with build 2005-11-07-10 of SeaMonkey trunk on Windows XP.

	Stephen Donner [:stephend] Not actively reading bugmail
	Comment 3 • 19 years ago

Likely is the fix in bug 314776:

nsCSSFrameConstructor::RestyleEvent::HandleEvent  [e:/builds/tinderbox/thunderbird-trunk/WINNT_5.0_Depend/mozilla/layout/base/nsCSSFrameConstructor.cpp, line 13302]
0x778b0c24
nsBidi::doWriteReverse  [e:/builds/tinderbox/thunderbird-trunk/WINNT_5.0_Depend/mozilla/layout/base/nsBidi.cpp, line 2211]
0xe9c03330

	Boris Zbarsky [:bzbarsky] Assignee
	Comment 4 • 19 years ago

Yeah, this is a regression from bug 314776

	Boris Zbarsky [:bzbarsky] Assignee
	Comment 5 • 19 years ago

Composer is torn down after the frame tree in this case and tries to reconstruct style after messing with override sheets...  We want to just ignore that on a destroyed presshell.

	David Baron :dbaron:
	Comment 6 • 19 years ago

Comment on attachment 202194 [details] [diff] [review]
Proposed patch

Didn't I review moving mIsDestroying as part of another patch?  Anyway, r+sr=dbaron.

	Boris Zbarsky [:bzbarsky] Assignee
	Comment 7 • 19 years ago

I moved mDidInitialReflow in bug 314776...

	Boris Zbarsky [:bzbarsky] Assignee
	Comment 8 • 19 years ago

Fixed.

	Boris Zbarsky [:bzbarsky] Assignee
	Comment 9 • 19 years ago

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

	Stephen Donner [:stephend] Not actively reading bugmail
	Comment 10 • 19 years ago

Verified FIXED using SeaMonkey 1.5a

Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a1) Gecko/20051108 Mozilla/1.0