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.