<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>16028</bug_id>
          
          <creation_ts>2007-11-17 06:42:44 -0800</creation_ts>
          <short_desc>[GTK] Track Mozilla&apos;s GIF decoder memory reductions</short_desc>
          <delta_ts>2012-02-18 11:41:24 -0800</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>Images</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>All</rep_platform>
          <op_sys>All</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>INVALID</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>Cairo, Gtk, Performance</keywords>
          <priority>P2</priority>
          <bug_severity>Minor</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Alp Toker">alp</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>gustavo</cc>
    
    <cc>mrobinson</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>61464</commentid>
    <comment_count>0</comment_count>
    <who name="Alp Toker">alp</who>
    <bug_when>2007-11-17 06:42:44 -0800</bug_when>
    <thetext>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</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>69123</commentid>
    <comment_count>1</comment_count>
    <who name="Alp Toker">alp</who>
    <bug_when>2008-02-01 15:40:06 -0800</bug_when>
    <thetext>They also have some libjpeg enhancements:

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

This one&apos;s trickier since we don&apos;t bundle libjpeg. Maybe we need to get these fixes pushed into the distributions so we can take advantage of them. Didn&apos;t study the changes closely yet.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>126387</commentid>
    <comment_count>2</comment_count>
    <who name="Peter Kasting">pkasting</who>
    <bug_when>2009-06-16 22:22:49 -0700</bug_when>
    <thetext>I could be mistaken, but I don&apos;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&apos;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.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>172249</commentid>
    <comment_count>3</comment_count>
    <who name="Gustavo Noronha (kov)">gustavo</who>
    <bug_when>2009-12-16 03:17:21 -0800</bug_when>
    <thetext>I would say this is not applicable, given the above? Should we close this bug?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>559788</commentid>
    <comment_count>4</comment_count>
    <who name="Martin Robinson">mrobinson</who>
    <bug_when>2012-02-18 11:41:24 -0800</bug_when>
    <thetext>It seems this should be closed. Feel free reopen it, if it seems necessary.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>