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)
Core
Layout: Images, Video, and HTML Frames
Tracking
()
RESOLVED
FIXED
People
(Reporter: rpotts, Assigned: rpotts)
Details
Attachments
(1 file)
(deleted),
patch
|
jst
:
review+
darin.moz
:
superreview+
jesup
:
approval+
|
Details | Diff | Splinter Review |
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
Assignee | ||
Comment 1•22 years ago
|
||
Comment 2•22 years ago
|
||
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 3•22 years ago
|
||
Comment on attachment 100780 [details] [diff] [review]
force all network image requests to be loaded in the background...
r=jst
Attachment #100780 -
Flags: review+
Assignee | ||
Comment 4•22 years ago
|
||
patch checked in.
-- rick
Status: NEW → RESOLVED
Closed: 22 years ago
Component: ImageLib → Image: Layout
Resolution: --- → FIXED
Comment 5•22 years ago
|
||
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+
Updated•22 years ago
|
Keywords: mozilla1.0.2+
Comment 7•22 years ago
|
||
Fix checked into 1.0 branch along with bug 93015
Keywords: mozilla1.0.2+ → fixed1.0.2
Comment 8•22 years ago
|
||
Verified fix checked into lxr.mozilla.org, changing keyword to verified1.0.2
Keywords: fixed1.0.2 → verified1.0.2
Comment 9•22 years ago
|
||
This patch causes us to not send StatusChange notifications for any image. See
bug 131256 for why I'd like to back this out.
Updated•6 years ago
|
Product: Core → Core Graveyard
Updated•6 years ago
|
Product: Core Graveyard → Core
You need to log in
before you can comment on or make changes to this bug.
Description
•