Closed Bug 74165 Opened 24 years ago Closed 24 years ago

imgLoader::LoadImage() chokes when AsyncOpen() fails

Categories

(Core :: Graphics: ImageLib, defect)

Product:
Core
Component:
Graphics: ImageLib
Type:
defect
Priority:
Not set
Severity:
critical

Tracking

()

Status:
VERIFIED FIXED

People

(Reporter: waterson, Assigned: pavlov)

References

Details

(Keywords: regression, Whiteboard: [imglib])

Attachments

(1 file)

kludge; force On[Start|Stop]Request() to be called if AsyncOpen() fails
(deleted), patch
Details | Diff | Splinter Review 
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.
This is actually a pretty serious bug, because it screws the layout regression
tests.
Severity: normal → critical
Keywords: mozilla0.9, patch, regression
Blocks: 62589
r=saari, thanks waterson
netwerk gurus, what say you? Are there other protocols besides ``file:'' where
AsyncOpen() can fail?
sure... any AsyncOpen implementation could fail for a variety of reasons.
*** Bug 75270 has been marked as a duplicate of this bug. ***
i've posted a patch in bug 6074 that includes a fix for this.
Status: NEW → ASSIGNED
Depends on: 6074
No longer depends on: 6074
Whiteboard: [imglib]
fixed.
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
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.

Attachment

General

Creator:
Created:
Updated:
Size: