<?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>5526</bug_id>
          
          <creation_ts>2005-10-28 00:44:11 -0700</creation_ts>
          <short_desc>SVGs with filters look grainy when scaled</short_desc>
          <delta_ts>2014-05-12 05:54:35 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>SVG</component>
          <version>523.x (Safari 3)</version>
          <rep_platform>Mac</rep_platform>
          <op_sys>OS X 10.4</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>DUPLICATE</resolution>
          <dup_id>31370</dup_id>
          
          <bug_file_loc>http://svg-whiz.com/svg/firefox-logo.svg</bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>HasReduction</keywords>
          <priority>P4</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          <dependson>11980</dependson>
    
    <dependson>19835</dependson>
    
    <dependson>19991</dependson>
          <blocked>68469</blocked>
    
    <blocked>5579</blocked>
    
    <blocked>26389</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Eric Seidel (no email)">eric</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>abarth</cc>
    
    <cc>emacemac7</cc>
    
    <cc>eric</cc>
    
    <cc>ian</cc>
    
    <cc>jeffschiller</cc>
    
    <cc>krit</cc>
    
    <cc>ossy</cc>
    
    <cc>webkit.review.bot</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>23099</commentid>
    <comment_count>0</comment_count>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2005-10-28 00:44:11 -0700</bug_when>
    <thetext>SVGs with filters look grainy when scaled

Currently we apply filters according to the abstract coordinate system of the graphic, NOT the device 
coordinate system (i.e. the final pixel coords when drawn to the screen).  This can be *bad* for a couple 
reasons.

1.  When you scale the resulting pixels, the result looks grainy.
2.  You can end up rendering 6 billion pixels and only using 1% of that if that part of the image is 
scaled down.

A couple ways to solve this:
1.  Just figure out what the final CTM for the item/group in question and render only the necessary 
number of pixels.
2.  (Better!) *delay* compositing of the result image.  Instead pass the CIImage back to the parent 
container (the one calling draw()) and let it do the compositing (or apply it&apos;s own filters using that 
CIImage as an input).  When you finally have to do the compositing, figure out the absolute CTM, etc.

I guess 2. is really just an optimized version of 1.  In either case, this will need to be solved.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>25630</commentid>
    <comment_count>1</comment_count>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2005-12-06 04:57:55 -0800</bug_when>
    <thetext>Erick said he would take a crack at this.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>141467</commentid>
    <comment_count>2</comment_count>
    <who name="Dirk Schulze">krit</who>
    <bug_when>2009-08-19 06:47:30 -0700</bug_when>
    <thetext>If I understand your comment right, we do this right now. I scaled the feGaussianBlur and feColorMatrix of the W3C test suite and i don&apos;t see pixels. But maybe this is a result of antialiasing?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>290164</commentid>
    <comment_count>3</comment_count>
    <who name="Dirk Schulze">krit</who>
    <bug_when>2010-10-06 02:46:19 -0700</bug_when>
    <thetext>Committed r69181: &lt;http://trac.webkit.org/changeset/69181&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>290170</commentid>
    <comment_count>4</comment_count>
    <who name="Csaba Osztrogonác">ossy</who>
    <bug_when>2010-10-06 03:17:30 -0700</bug_when>
    <thetext>(In reply to comment #3)
&gt; Committed r69181: &lt;http://trac.webkit.org/changeset/69181&gt;

It broke 2 tests on Qt bot:
http://build.webkit.org/results/Qt%20Linux%20Release/r69181%20(21569)/svg/filters/shadow-on-filter-pretty-diff.html
http://build.webkit.org/results/Qt%20Linux%20Release/r69181%20(21569)/svg/filters/shadow-on-rect-with-filter-pretty-diff.html</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>290181</commentid>
    <comment_count>5</comment_count>
    <who name="Dirk Schulze">krit</who>
    <bug_when>2010-10-06 03:56:49 -0700</bug_when>
    <thetext>Committed r69182: &lt;http://trac.webkit.org/changeset/69182&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>290240</commentid>
    <comment_count>6</comment_count>
    <who name="Dirk Schulze">krit</who>
    <bug_when>2010-10-06 05:48:20 -0700</bug_when>
    <thetext>Committed r69187: &lt;http://trac.webkit.org/changeset/69187&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>290244</commentid>
    <comment_count>7</comment_count>
    <who name="Dirk Schulze">krit</who>
    <bug_when>2010-10-06 06:01:18 -0700</bug_when>
    <thetext>This bug was fixed with the patch on bug 31370. Marking this bug as duplication of 31370 because fixes for the patch mention this bug. Sorry for that.

*** This bug has been marked as a duplicate of bug 31370 ***</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>290323</commentid>
    <comment_count>8</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2010-10-06 08:42:54 -0700</bug_when>
    <thetext>http://trac.webkit.org/changeset/69187 might have broken GTK Linux 32-bit Debug</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>290324</commentid>
    <comment_count>9</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2010-10-06 08:43:03 -0700</bug_when>
    <thetext>http://trac.webkit.org/changeset/69188 might have broken GTK Linux 32-bit Debug</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>