Closed Bug 2277 Opened 26 years ago Closed

hang assert when interrupting doc load

Categories

(Core :: DOM: HTML Parser, defect, P2)

x86
Windows NT
defect

Tracking

()

VERIFIED FIXED

People

(Reporter: buster, Assigned: rickg)

References

()

Details

this URL never fully loads.  It loads the top banner ad, then spins waiting for
who-knows-what.  I'd call this a hang.

Then lose patience.  Hit the back key.  Get the assert below.

I'm pretty sure these are 2 separate problems, but I'll leave that for Rick to
decide.

==========================================================

stack:

nsDebug::PostCondition() line 115 + 13 bytes
nsParser::~nsParser() line 157 + 36 bytes
nsParser::`scalar deleting destructor'() + 15 bytes
nsParser::Release() line 165 + 99 bytes
nsDocumentBindInfo::~nsDocumentBindInfo() line 1295 + 27 bytes
nsDocumentBindInfo::`scalar deleting destructor'() + 15 bytes
nsDocumentBindInfo::Release() line 1306 + 99 bytes
SupportsArrayImpl::Clear() line 316 + 36 bytes
nsDocLoaderImpl::Stop() line 923
nsWebShell::Stop() line 1303
nsWebShell::LoadURL() line 1268
nsBrowserWindow::GoTo() line 518
HandleSiteEvent() line 1126 + 23 bytes
nsWindow::DispatchEvent() line 355 + 10 bytes
nsWindow::DispatchWindowEvent() line 371
nsWindow::DispatchMouseEvent() line 1844 + 15 bytes
nsWindow::ProcessMessage() line 1383 + 24 bytes
nsWindow::WindowProc() line 414 + 27 bytes
USER32! 77e71250()
Status: NEW → ASSIGNED
Assignee: rickg → gagan
Status: ASSIGNED → NEW
I've removed the assertion, but the hang still occurs. This looks like a fine
example of the netlib problems I've been seeing recently.
Assignee: gagan → rickg
Seems like the nsHTMLContentSink::nsDoneLoadingScript is not conveying the
onStopBinding to the parser?
Status: NEW → ASSIGNED
So I've found the problem, and low and behold it's yet *another* special case of
interaction between JS/DOM/Parser. Since this is a tricky issue, I'm not going
to checkin the fix until after I've had more time to test.

Call me if you want to know the gory details.
Status: RESOLVED → VERIFIED
looks swell, marking verified
You need to log in before you can comment on or make changes to this bug.