Closed
Bug 74165
Opened 25 years ago
Closed 25 years ago
imgLoader::LoadImage() chokes when AsyncOpen() fails
Categories
(Core :: Graphics: ImageLib, defect)
Core
Graphics: ImageLib
Tracking
()
VERIFIED
FIXED
People
(Reporter: waterson, Assigned: pavlov)
References
Details
(Keywords: regression, Whiteboard: [imglib])
Attachments
(1 file)
imgLoader::LoadImage() assumes that AsyncOpen() will always succeed. It doesn't
(e.g., if you try to load a non-existant file: URL).
Save this test case on your hard drive:
<html>
<body>
<img src="blah">
</body>
</html>
Expected: throbber stops spinning
Actual: load never completes
I'll attach a kludge to deal with this.
|
Reporter | |
Comment 1•25 years ago
|
||
|
|
Reporter | |
Comment 2•25 years ago
|
||
This is actually a pretty serious bug, because it screws the layout regression
tests.
Severity: normal → critical
|
|
||
Comment 3•25 years ago
|
||
r=saari, thanks waterson
|
|
Reporter | |
Comment 4•25 years ago
|
||
netwerk gurus, what say you? Are there other protocols besides ``file:'' where
AsyncOpen() can fail?
|
|
||
Comment 5•25 years ago
|
||
sure... any AsyncOpen implementation could fail for a variety of reasons.
|
Assignee | |
Comment 7•25 years ago
|
||
i've posted a patch in bug 6074 that includes a fix for this.
Status: NEW → ASSIGNED
Depends on: 6074
|
|
||
Updated•25 years ago
|
Whiteboard: [imglib]
|
|
Assignee | |
Comment 8•25 years ago
|
||
fixed.
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
|
|
||
Comment 9•24 years ago
|
||
Verified Fixed Win XP build 2002012903, linux build 2002012909 and Mac OS X
build 2002012808
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•