Closed
Bug 127430
Opened 24 years ago
Closed 24 years ago
throw away nsImageGTK client side copy
Categories
(Core :: XUL, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: tor, Assigned: tor)
Details
Attachments
(1 file, 1 obsolete file)
|
5.03 KB,
patch
|
pavlov
:
review+
blizzard
:
superreview+
chofmann
:
approval+
|
Details | Diff | Splinter Review |
Patch throws away client side copy on Optimize(). Postscript printing now
does a LockImagePixels(), which will read back the data from the server if
the image has been optimized.
|
||
Comment 2•24 years ago
|
||
Comment on attachment 71086 [details] [diff] [review]
throw away client side image copy
r/sr=blizzard
Attachment #71086 -
Flags: superreview+
Attachment #71086 -
Attachment is obsolete: true
|
||
Comment 5•24 years ago
|
||
Comment on attachment 71116 [details] [diff] [review]
as above, plus kill the black background of transparent images
r=pavlov
Attachment #71116 -
Flags: review+
|
|
||
Comment 6•24 years ago
|
||
Comment on attachment 71116 [details] [diff] [review]
as above, plus kill the black background of transparent images
sr=blizzard
Attachment #71116 -
Flags: superreview+
|
||
Comment 7•24 years ago
|
||
a=chofmann for 0.9.9
Checked in.
Status: NEW → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
|
|
||
Updated•24 years ago
|
Attachment #71116 -
Flags: approval+
Hrm. It looks like this actually slowed down page load times on btek by about
10 ms. Any ideas why?
|
|
Assignee | |
Comment 10•24 years ago
|
||
No idea - all it does it add a few delete[]s when images finish loading.
|
|
||
Comment 11•24 years ago
|
||
What's the space tradeoff here? It's signifigant, right?
You need to log in
before you can comment on or make changes to this bug.
Description
•