29153 - timing-dependent crash closing mail window

Status: VERIFIED FIXED

(SeaMonkey :: General, defect, P3)

Description

[Dan M]

Though this happens in a window, I'm dumping it on the guy who recently wrote 
the code where the crash is happening, instead of me.
  I'm seeing an occasional, difficult to reproduce crash in 
GlobalWindowImpl::RunTimeout. Generally I've noticed it, if at all, when closing 
the mail window. The offending line is #2974 (rev 1.236)

nsIScriptContext* tempContext = mContext;

(or actually the next line). Sometimes mContext is null when I stumble into this 
line while tearing the window down. Mozilla recovers and seems healthy if I just 
(NS_RELEASE(temp) and) skip the rest of the function and continue. Maybe that's 
the solution, or maybe the timers want to stop firing earlier.

Comment 1

[vidur (gone)]

Couldn't recreate the bug, but the null check for mContext seemed like a good 
thing. Fix checked in on 2/25/2000.

Comment 2

[Asa Dotzler [:asa]]

not sure what is required to verify this as fixed.  vidur or danm can one of you 
verify this one.  Thanks. 

Comment 3

[Asa Dotzler [:asa]]

I  have not crashed in GlobalWindowImpl::RunTimeout closing mail window after
watching for this for the last few months.  Verifying this fixed with mozilla
win32 talkback bits 072408.