[xlib] Transparent animated gifs use large amounts of CPU

RESOLVED FIXED in mozilla1.0

Status

()

Core
ImageLib
RESOLVED FIXED
17 years ago
17 years ago

People

(Reporter: jesup, Assigned: jesup)

Tracking

({perf})

Trunk
mozilla1.0
x86
Linux
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

17 years ago
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

17 years ago
Created attachment 72504 [details] [diff] [review]
patch
(Assignee)

Comment 2

17 years ago
Targeted at 1.0 for now.  Perf-only issue.  Adjusting subject to be more descriptive
Keywords: mozilla1.0, patch, perf
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

17 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 5

17 years ago
This appears to be the version without the 64-bit fixes.  Add those and sr=tor.
(Assignee)

Comment 6

17 years ago
Created attachment 72739 [details] [diff] [review]
patch updated for 64bit issues

Update to match patch for bug 127455 - thanks.
Attachment #72504 - Attachment is obsolete: true
(Assignee)

Updated

17 years ago
Attachment #72739 - Flags: superreview+
Attachment #72739 - Flags: review+

Comment 7

17 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

17 years ago
Blocks: 79119
Summary: Transparent animated gifs use large amounts of CPU → [xlib] Transparent animated gifs use large amounts of CPU

Updated

17 years ago
Blocks: 119597
(Assignee)

Comment 8

17 years ago
Fix checked in
Status: NEW → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.