Closed
Bug 128939
Opened 24 years ago
Closed 24 years ago
[xlib] Transparent animated gifs use large amounts of CPU
Categories
(Core :: Graphics: ImageLib, defect)
Tracking
()
RESOLVED
FIXED
mozilla1.0
People
(Reporter: jesup, Assigned: jesup)
References
Details
(Keywords: perf)
Attachments
(1 file, 1 obsolete file)
|
5.08 KB,
patch
|
jesup
:
review+
jesup
:
superreview+
asa
:
approval+
|
Details | Diff | Splinter Review |
Follow-on bug to 125025 - same fix for Xlib
Note: this doesn't include tor's rework to move storage of images to the server
already applied to the GTK code. This is just the speedup for transparent GIFs,
the exact equivalent to bug 125025
Patch to be attached
|
Assignee | |
Comment 1•24 years ago
|
||
|
|
Assignee | |
Comment 2•24 years ago
|
||
Targeted at 1.0 for now. Perf-only issue. Adjusting subject to be more descriptive
Summary: Animated gifs use large amounts of CPU when not visible → Transparent animated gifs use large amounts of CPU
Target Milestone: --- → mozilla1.0
|
|
Assignee | |
Comment 3•24 years ago
|
||
Looking for r/sr so I can get a=. I'd like to get xlib synced up with gtk for 1.0
|
||
Comment 4•24 years ago
|
||
Comment on attachment 72504 [details] [diff] [review]
patch
r=Roland.Mainz@informatik.med.uni-giessen.de
Attachment #72504 -
Flags: review+
This appears to be the version without the 64-bit fixes. Add those and sr=tor.
|
|
Assignee | |
Comment 6•24 years ago
|
||
Update to match patch for bug 127455 - thanks.
Attachment #72504 -
Attachment is obsolete: true
|
|
Assignee | |
Updated•24 years ago
|
Attachment #72739 -
Flags: superreview+
Attachment #72739 -
Flags: review+
|
||
Comment 7•24 years ago
|
||
Comment on attachment 72739 [details] [diff] [review]
patch updated for 64bit issues
a=asa (on behalf of drivers) for checkin to the 1.0 trunk
Attachment #72739 -
Flags: approval+
|
|
||
Updated•24 years ago
|
Blocks: 79119
Summary: Transparent animated gifs use large amounts of CPU → [xlib] Transparent animated gifs use large amounts of CPU
|
|
Assignee | |
Comment 8•24 years ago
|
||
Fix checked in
Status: NEW → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•