Closed Bug 171053 Opened 22 years ago Closed 22 years ago

network image requests should NOT generate progress notifications...

Categories

(Core :: Layout: Images, Video, and HTML Frames, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: rpotts, Assigned: rpotts)

Details

Attachments

(1 file)

When imglib initiates an underlying network request for an image, it should be loaded using LOAD_BACKGROUND. Since the request does not belong to any loadgroup -- but is instead 'owned' by its proxy image requests -- any progress/status notifications will be ignored (anyways) -- rick
Comment on attachment 100780 [details] [diff] [review] force all network image requests to be loaded in the background... r/sr=darin
Attachment #100780 - Flags: superreview+
Comment on attachment 100780 [details] [diff] [review] force all network image requests to be loaded in the background... r=jst
Attachment #100780 - Flags: review+
patch checked in. -- rick
Status: NEW → RESOLVED
Closed: 22 years ago
Component: ImageLib → Image: Layout
Resolution: --- → FIXED
Comment on attachment 100780 [details] [diff] [review] force all network image requests to be loaded in the background... a=rjesup@wgate.com for 1.0 branch; change mozilla1.0.2+ to fixed1.0.2 when checked in.
Attachment #100780 - Flags: approval+
I'll check this into 1.0 branch along with the checkin for bug 93015
Fix checked into 1.0 branch along with bug 93015
Verified fix checked into lxr.mozilla.org, changing keyword to verified1.0.2
This patch causes us to not send StatusChange notifications for any image. See bug 131256 for why I'd like to back this out.
Product: Core → Core Graveyard
Product: Core Graveyard → Core
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: