47798 - Crash selecting 'Go' menu after switching Theme (SH crash).

Status: VERIFIED DUPLICATE of bug 47490

(Core :: DOM: Navigation, defect, P3)

Description

[John Morrison]

Overview Description: 

  Crash selecting 'Go' menu after switching Theme (SH crash).

  NOTE!!!: This is on the TRUNK; this is NOT a crasher in PR2 bits.

Steps to Reproduce: 
  1) Launch Mozilla
  2) Switch Themes
  3) Click on the 'Go' top level menu

Actual Results:                 Expected Results:             
  Crash				  Back/Forward/Home/...       

Reproducibility: 100% on mac/linux/win32

Build Date & Platform Bug Found: 
  2000080508 win95 & win98
  2000080508 mac
  2000080508 linux

Additional Information: 

Here's a stack trace via talkback ... I know that there are issues with 
getting the docshell properly hooked up after a skin switch (or some such
problem; hyatt can explain).

PR_AtomicIncrement [pratom.c, line 286] 
nsDocShell::AddRef [d:\builds\seamonkey\mozilla\docshell\base\nsDocShell.cpp, 
line 135] 
nsWebShell::AddRef [d:\builds\seamonkey\mozilla\docshell\base\nsWebShell.cpp, 
line 316] 
nsDocShell::QueryInterface 
[d:\builds\seamonkey\mozilla\docshell\base\nsDocShell.cpp] 
nsWebShell::QueryInterface 
[d:\builds\seamonkey\mozilla\docshell\base\nsWebShell.cpp, line 344] 
nsQueryInterface::operator() 
[d:\builds\seamonkey\mozilla\xpcom\base\nsCOMPtr.cpp, line 37] 
nsCOMPtr_base::assign_from_helper 
[d:\builds\seamonkey\mozilla\xpcom\base\nsCOMPtr.cpp, line 66] 
nsDocShell::GetSessionHistory 
[d:\builds\seamonkey\mozilla\docshell\base\nsDocShell.cpp, line 1507] 
nsDocShell::GetSessionHistory 
[d:\builds\seamonkey\mozilla\docshell\base\nsDocShell.cpp, line 1510] 
nsDocShell::GetSessionHistory 
[d:\builds\seamonkey\mozilla\docshell\base\nsDocShell.cpp, line 1510] 
nsDocShell::GetSessionHistory 
[d:\builds\seamonkey\mozilla\docshell\base\nsDocShell.cpp, line 1510] 
nsDocShell::GetSessionHistory 
[d:\builds\seamonkey\mozilla\docshell\base\nsDocShell.cpp, line 1510] 
nsDocShell::GetSessionHistory 
[d:\builds\seamonkey\mozilla\docshell\base\nsDocShell.cpp, line 1510] 
nsDocShell::GetSessionHistory 
[d:\builds\seamonkey\mozilla\docshell\base\nsDocShell.cpp, line 1510] 
nsDocShell::GetSessionHistory 
[d:\builds\seamonkey\mozilla\docshell\base\nsDocShell.cpp, line 1510] 

 ... snip .. this recursion carries on for the rest of the stack in the
             talkback report

Comment 1

[John Morrison]

A somewhat self-evident nsbeta3 (I could ship with some things not working
correctly after a skin/theme switch, but not a one-click crash). 

Comment 2

[Blake Ross]

Thanks, this is 47490.  Jag and I are working on it, believe me...

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

Comment 3

[John Morrison]

Yep, dupe.