145737 - while loading index.html the browser crash [@RuleProcessorData::RuleProcessorData]

Status: RESOLVED FIXED

(Core :: CSS Parsing and Computation, defect)

Description

[Nicolas Christener]

From Bugzilla Helper:
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.0.0+) Gecko/20020519
BuildID:    2002051907

When i load the index.html file from a local harddrive, it will load the file
home.html in an iframe. Sometimes it works, but if i close the browser, and open
the file again, the browser mostly crash. I'm not sure, what the problem is.
Maybe there is a problem with the non existing css (which i try to load in the
home.html file).
I have the same Problem under win2k and linux. I tryed 2 different builds.
(Sorry... i hope you understand my bad english.)

Reproducible: Sometimes
Steps to Reproduce:
0.downlaod the tarball and untar it.
http://www.datacomm.ch/niki.christener/alles.tar
1.Open the file "index.html" 
2.Close the Browser.
3.Open the file again.

Actual Results:  The browser'll crash (not every time)

Expected Results:  :) The browser should not crash... 

sorry... the html files are bad!! they aren't w3c validated. But in my opinion,
the browser shouldn't crash.

Comment 1

[Matthias Versen [:Matti]]

Reporter:
Can you give us a talkback ID from that crash ?

Comment 2

[Doron Rosenberg (IBM)]

Incident ID 6537177
Stack Signature RuleProcessorData::RuleProcessorData 1ef521d4
Email Address
Product ID MozillaTrunk
Build ID 2002052108
Trigger Time 2002-05-22 00:37:53
Platform Win32
Operating System Windows 98 4.10 build 67766222
Module GKCONTENT.DLL
URL visited
User Comments
Trigger Reason Access violation
Source File Name nsCSSStyleSheet.cpp
Trigger Line No. 3257
Stack Trace
RuleProcessorData::RuleProcessorData [nsCSSStyleSheet.cpp, line 3257]
StyleSetImpl::ResolveStyleFor [nsStyleSet.cpp, line 1104]
nsPresContext::ResolveStyleContextFor [nsPresContext.cpp, line 952]
FrameManager::ReResolveStyleContext [nsFrameManager.cpp, line 1780]
FrameManager::ReResolveStyleContext [nsFrameManager.cpp, line 1863]
FrameManager::ReResolveStyleContext [nsFrameManager.cpp, line 1863]
FrameManager::ComputeStyleChangeFor [nsFrameManager.cpp, line 1907]
PresShell::ReconstructStyleData [nsPresShell.cpp, line 5495]
PresShell::StyleSheetAdded [nsPresShell.cpp, line 5519]
nsDocument::InsertStyleSheetAt [nsDocument.cpp, line 1633]
CSSLoaderImpl::InsertSheetInDoc [nsCSSLoader.cpp, line 1191]
CSSLoaderImpl::SheetComplete [nsCSSLoader.cpp, line 894]
CSSLoaderImpl::ParseSheet [nsCSSLoader.cpp, line 949]
CSSLoaderImpl::DidLoadStyle [nsCSSLoader.cpp, line 985]
SheetLoadData::OnStreamComplete [nsCSSLoader.cpp, line 745]
nsStreamLoader::OnStopRequest [nsStreamLoader.cpp, line 163]
nsFileChannel::OnStopRequest [nsFileChannel.cpp, line 522]
nsOnStopRequestEvent::HandleEvent [nsRequestObserverProxy.cpp, line 213]
PL_HandleEvent [plevent.c, line 597]
PL_ProcessPendingEvents [plevent.c, line 530]
_md_EventReceiverProc [plevent.c, line 1078]
KERNEL32.DLL + 0x242e7 (0xbff942e7)
0x00648bf6 

Comment 3

[Matthias Versen [:Matti]]

-> Style System (?) (or Layout)

Comment 4

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Attachment: fix

This fixes the problem.  It's a little bit of a hack, but it won't break
anything, unlike previous workarounds for this problem, since it's only
clearing the undisplayed map a little earlier in the shutdown process.	Scripts
won't run after this point, so we don't really need to worry about dynamic
style changes (well, there's :hover, but I'm not too worried).

(If the bug 118014 / bug 136704 topcrash doesn't go away soon, I'll make the
undisplayed map hold an owning reference to the content nodes just like frames
do.)

Comment 5

[Chris Waterson]

Comment on attachment 84699 [details] [diff] [review]
fix

sr=waterson

Comment 6

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 84699 [details] [diff] [review]
fix

r=bzbarsky

Comment 7

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Fix checked in 2002-05-26 10:03 PDT.