121239 - Crash after changing theme in preferences and restarting

Status: RESOLVED WORKSFORME

(SeaMonkey :: Themes, defect)

Description

[Geva Patz]

From Bugzilla Helper:
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:0.9.7) Gecko/20011221
BuildID:    2001122106

If I change themes in the preferences dialog, hit the 'Apply' button, then quit
Mozilla and restart, I get a crash dialog along the lines of "The instruction at
location 'FOO' could not access the memory at location 'BAR'. The memory could
not be 'written'." If I then restart Mozilla again, everything runs fine. I'm
running the Mozilla quickstart option, which may or may not be significant. The
first time I restart Mozilla, quickstart dies in the crash, so the second time I
restart (successfully), I'm starting 'from scratch' -- splash screen and all.

This problem does _not_ occur if I change themes from the View/Apply Theme menu

Reproducible: Always
Steps to Reproduce:
1. Enable quick start (this may not be a necessary step - I haven't tried 
without it)
2. Open the preferences dialog box and select a different theme to the current one
3. Click 'Apply', close the dialog box that ensues, close preferences, and close
Mozilla.
4. Open Mozilla again and watch it crash
5. Open Mozilla once more and watch it run happily

Actual Results:  Crash dialog box

Expected Results:  Successful restart with the new theme applied

Comment 1

[Boris Zbarsky [:bzbarsky]]

There is no more apply button in the preference dialog and the theme change
engine has changed completely since 0.9.7.  Please retest with a recent nightly
or the upcoming 0.9.8.

Comment 2

[Henrik Gemal]

wfm 20020302

Comment 3

[Patty Mac]

Boris, yes, it's true, there's no "Apply button" to switch themes
in preferences dialog.  Anwyay, the new buttons are "Uninstall Classic" and
"Uninstall Modern". However, it still crashes if you switch themes
in preferences dialog. 

Stacktrace on windows 98 (commercial trunk: 2002-02-27-06-trunk):

nsTemplateRule::GetContent [d:\builds\seamonkey\mozilla\content\xul\templates\
src\nsTemplateRule.cpp, line 63] 
nsStyleContext::Destroy [d:\builds\seamonkey\mozilla\content\base\src\
nsStyleContext.cpp, line 1279] 
nsStyleContext::Release [d:\builds\seamonkey\mozilla\content\base\src\
nsStyleContext.cpp, line 187] 
nsProperties::ReleaseValues [d:\builds\seamonkey\mozilla\xpcom\ds\
nsProperties.cpp, line 70] 
_hashEnumerate [d:\builds\seamonkey\mozilla\xpcom\ds\nsHashtable.cpp, line 199] 
PL_HashTableEnumerateEntries [../../../lib/ds/plhash.c, line 430] 
nsHashtable::Enumerate [d:\builds\seamonkey\mozilla\xpcom\ds\nsHashtable.cpp, 
line 362] 
nsSupportsHashtable::~nsSupportsHashtable [d:\builds\seamonkey\mozilla\xpcom\ds\
nsHashtable.cpp, line 866] 
nsSupportsHashtable::`scalar deleting destructor' 
nsOutlinerBodyFrame::ClearStyleAndImageCaches [d:\builds\seamonkey\mozilla\
layout\xul\base\src\outliner\src\nsOutlinerBodyFrame.cpp, line 3129] 
nsCOMPtr_base::assign_from_helper [d:\builds\seamonkey\mozilla\xpcom\glue\
nsCOMPtr.cpp, line 81] 
FlushMiscWidgetInfo [d:\builds\seamonkey\mozilla\layout\html\base\src\
nsPresShell.cpp, line 5288] 
FlushMiscWidgetInfo [d:\builds\seamonkey\mozilla\layout\html\base\src\
nsPresShell.cpp, line 5288] 
FlushMiscWidgetInfo [d:\builds\seamonkey\mozilla\layout\html\base\src\
nsPresShell.cpp, line 5288] 
FlushMiscWidgetInfo [d:\builds\seamonkey\mozilla\layout\html\base\src\
nsPresShell.cpp, line 5288] 
FlushMiscWidgetInfo [d:\builds\seamonkey\mozilla\layout\html\base\src\
nsPresShell.cpp, line 5288] 
FlushMiscWidgetInfo [d:\builds\seamonkey\mozilla\layout\html\base\src\
nsPresShell.cpp, line 5288] 
FlushMiscWidgetInfo [d:\builds\seamonkey\mozilla\layout\html\base\src\
nsPresShell.cpp, line 5288] 
FlushMiscWidgetInfo [d:\builds\seamonkey\mozilla\layout\html\base\src\
nsPresShell.cpp, line 5288] 
FlushMiscWidgetInfo [d:\builds\seamonkey\mozilla\layout\html\base\src\
nsPresShell.cpp, line 5288] 
FlushMiscWidgetInfo [d:\builds\seamonkey\mozilla\layout\html\base\src\
nsPresShell.cpp, line 5288] 
FlushMiscWidgetInfo [d:\builds\seamonkey\mozilla\layout\html\base\src\
nsPresShell.cpp, line 5288] 
FlushMiscWidgetInfo [d:\builds\seamonkey\mozilla\layout\html\base\src\
nsPresShell.cpp, line 5288] 
FlushMiscWidgetInfo [d:\builds\seamonkey\mozilla\layout\html\base\src\
nsPresShell.cpp, line 5288] 
FlushMiscWidgetInfo [d:\builds\seamonkey\mozilla\layout\html\base\src\
nsPresShell.cpp, line 5288] 
PresShell::ReconstructStyleData [d:\builds\seamonkey\mozilla\layout\html\base\
src\nsPresShell.cpp, line 5327] 
PresShell::StyleSheetRemoved [d:\builds\seamonkey\mozilla\layout\html\base\src\
nsPresShell.cpp, line 5371] 
nsXULDocument::UpdateStyleSheets [d:\builds\seamonkey\mozilla\content\xul\
document\src\nsXULDocument.cpp, line 1344] 
nsChromeRegistry::RefreshWindow [d:\builds\seamonkey\mozilla\rdf\chrome\src\
nsChromeRegistry.cpp, line 1334] 
nsChromeRegistry::RefreshSkins [d:\builds\seamonkey\mozilla\rdf\chrome\src\
nsChromeRegistry.cpp, line 1161] 
XPTC_InvokeByIndex [d:\builds\seamonkey\mozilla\xpcom\reflect\xptcall\src\md\
win32\xptcinvoke.cpp, line 106] 
XPCWrappedNative::CallMethod [d:\builds\seamonkey\mozilla\js\src\xpconnect\src\
xpcwrappednative.cpp, line 2022] 
XPC_WN_CallMethod [d:\builds\seamonkey\mozilla\js\src\xpconnect\src\
xpcwrappednativejsops.cpp, line 1267] 
js_Invoke [d:\builds\seamonkey\mozilla\js\src\jsinterp.c, line 790] 
js_Interpret [d:\builds\seamonkey\mozilla\js\src\jsinterp.c, line 2746] 
js_Invoke [d:\builds\seamonkey\mozilla\js\src\jsinterp.c, line 806] 
js_InternalInvoke [d:\builds\seamonkey\mozilla\js\src\jsinterp.c, line 881] 
JS_CallFunctionValue [d:\builds\seamonkey\mozilla\js\src\jsapi.c, line 3390] 
nsJSContext::CallEventHandler [d:\builds\seamonkey\mozilla\dom\src\base\
nsJSEnvironment.cpp, line 1019] 
GlobalWindowImpl::RunTimeout [d:\builds\seamonkey\mozilla\dom\src\base\
nsGlobalWindow.cpp, line 4074] 
GlobalWindowImpl::TimerCallback [d:\builds\seamonkey\mozilla\dom\src\base\
nsGlobalWindow.cpp, line 4386] 
nsTimerImpl::Process [d:\builds\seamonkey\mozilla\xpcom\threads\nsTimerImpl.cpp, 
line 284] 
PL_ProcessPendingEvents [d:\builds\seamonkey\mozilla\xpcom\threads\plevent.c, 
line 524] 
_md_EventReceiverProc [d:\builds\seamonkey\mozilla\xpcom\threads\plevent.c, line 
1072] 
KERNEL32.DLL + 0x24407 (0xbff94407) 
0x00688c16 


Please refer to bug: http://bugzilla.mozilla.org/show_bug.cgi?id=116038 for 
further details.