Closed Bug 745 Opened 26 years ago Closed

ss:url interrupts cause crash

Categories

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

x86
Windows NT
defect

Tracking

()

VERIFIED WORKSFORME

People

(Reporter: jud, Assigned: rpotts)

References

()

Details

(Keywords: testcase)

http://w3 does two redirects, one right after the other. the redirect code calls the WebShell's urlLoad routine with the new url to load. the w3 test case simulates entering a url, hitting enter and entering another url and hitting enter before the first one had a chance to finish. the assert and subsequent crash occurs on the third url entered (i.e. the second interrupt/redirect) in the image library (see modules\libimg\src\if.cpp il_frame_complete_notify() around line: 186). We never seem to be interrupting/aborting the streams we create during the url loads when they're interrupted with another top level load. Are we handling interruption yet? Does this bug look like an interruption issue?
Status: NEW → ASSIGNED
I believe that the problem is that currently, images are not loaded via the Document Loader, so they do *not* get interrupted. Once Images are loaded through the DocLoader, this problem should go away...
Summary: url interrupts cause crash → ss:url interrupts cause crash
Putting on ss: radar.
I can't reproduce this - may be an old bug. Rick, speak up if there could still be problems in this area.
I can't reproduce this - may be an old bug. Rick, speak up if there could still be problems in this area.
Can someone mark this VERIFIED-WORKSFORME, either paulmac or valeski? thanks
Status: RESOLVED → VERIFIED
Inserting Milestone info.
Inserting Milestone info.
Target Milestone: M1
Inserting Milestone info.
Changing all Networking Library/Browser bugs to Networking-Core component for Browser. Occasionally, Bugzilla will burp and cause Verified bugs to reopen when I do this in a bulk change. If this happens, I will fix. ;-)
Bulk move of all Networking-Core (to be deleted component) bugs to new Networking component.
Component: Networking → Embedding: Docshell
Keywords: testcase
You need to log in before you can comment on or make changes to this bug.