Bug 16028 - [GTK] Track Mozilla's GIF decoder memory reductions
Summary: [GTK] Track Mozilla's GIF decoder memory reductions
Status: RESOLVED INVALID
Alias: None
Product: WebKit
Classification: Unclassified
Component: Images (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Minor
Assignee: Nobody
URL:
Keywords: Cairo, Gtk, Performance
Depends on:
Blocks:
 
Reported: 2007-11-17 06:42 PST by Alp Toker
Modified: 2012-02-18 11:41 PST (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Alp Toker 2007-11-17 06:42:44 PST
The image decoders in the GTK+/Cairo port and others are an old port of Mozilla code. Mozilla has been improving their code and it would be nice to track the changes:

https://bugzilla.mozilla.org/show_bug.cgi?id=143046
Bug 143046 (slowGIF) – Need to Keep GIFs at original 8 bit or optimize
Comment 1 Alp Toker 2008-02-01 15:40:06 PST
They also have some libjpeg enhancements:

  https://bugzilla.mozilla.org/show_bug.cgi?id=242145

This one's trickier since we don't bundle libjpeg. Maybe we need to get these fixes pushed into the distributions so we can take advantage of them. Didn't study the changes closely yet.
Comment 2 Peter Kasting 2009-06-16 22:22:49 PDT
I could be mistaken, but I don't think the changes on https://bugzilla.mozilla.org/show_bug.cgi?id=143046 are directly applicable to the cross-platform decoders in WebKit.  From a brief look, these are mostly changes to Mozilla's surrounding image architecture to allow the decoded frames to remain in 8bpp form instead of expanding to 24bpp.  Right now in WebKit the decoded pixels are stored as 32bpp in all circumstances.  We might be able to apply a similar idea, but none of the code will really translate.

This will be especially tricky given the memory management around the decoded frames.
Comment 3 Gustavo Noronha (kov) 2009-12-16 03:17:21 PST
I would say this is not applicable, given the above? Should we close this bug?
Comment 4 Martin Robinson 2012-02-18 11:41:24 PST
It seems this should be closed. Feel free reopen it, if it seems necessary.