<?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>15071</bug_id>
          
          <creation_ts>2007-08-24 10:44:20 -0700</creation_ts>
          <short_desc>[cairo] SVG skews are incorrect</short_desc>
          <delta_ts>2007-08-24 13:17:30 -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>New Bugs</component>
          <version>523.x (Safari 3)</version>
          <rep_platform>All</rep_platform>
          <op_sys>All</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc>http://www.croczilla.com/svg/samples/skew1/skew1.xml</bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>Cairo, Gtk</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Alp Toker">alp</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1537</commentid>
    <comment_count>0</comment_count>
    <who name="Alp Toker">alp</who>
    <bug_when>2007-08-24 10:44:20 -0700</bug_when>
    <thetext>The SVG sample at http://www.croczilla.com/svg/samples/skew1/skew1.xml exhibits incorrect skewing, suggesting a problem with shear transforms in the Cairo graphics backend.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1538</commentid>
    <comment_count>1</comment_count>
      <attachid>16107</attachid>
    <who name="George Wright">gwright</who>
    <bug_when>2007-08-24 10:52:58 -0700</bug_when>
    <thetext>Created attachment 16107
Proposed fix</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1540</commentid>
    <comment_count>2</comment_count>
      <attachid>16107</attachid>
    <who name="Adam Roben (:aroben)">aroben</who>
    <bug_when>2007-08-24 11:18:18 -0700</bug_when>
    <thetext>Comment on attachment 16107
Proposed fix

The fix looks fine, but this needs a regression test. I&apos;ll r+ the patch once we have a test with results.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1542</commentid>
    <comment_count>3</comment_count>
      <attachid>16107</attachid>
    <who name="Oliver Hunt">oliver</who>
    <bug_when>2007-08-24 12:32:21 -0700</bug_when>
    <thetext>Comment on attachment 16107
Proposed fix

This is gtk only and the gtk port does not yet support drt.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1543</commentid>
    <comment_count>4</comment_count>
    <who name="Oliver Hunt">oliver</who>
    <bug_when>2007-08-24 12:43:38 -0700</bug_when>
    <thetext>Sending        ChangeLog
Sending        platform/graphics/cairo/AffineTransformCairo.cpp
Transmitting file data ..
Committed revision 25214.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1535</commentid>
    <comment_count>5</comment_count>
    <who name="Adam Roben (:aroben)">aroben</who>
    <bug_when>2007-08-24 13:17:30 -0700</bug_when>
    <thetext>Oliver says that we have 3 tests that already cover SVG skew: svg/W3C-SVG-1.1/coords-trans-01-b, svg/W3C-SVG-1.1/pservers-grad-07-b, svg/W3C-SVG-1.1/linking-a-05-t. Presumably those tests would have failed on Gtk if Gtk were able to run the regression tests. So that&apos;s why we don&apos;t need a new test for this fix.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>16107</attachid>
            <date>2007-08-24 10:52:58 -0700</date>
            <delta_ts>2007-08-24 12:32:21 -0700</delta_ts>
            <desc>Proposed fix</desc>
            <filename>affinetransform.patch</filename>
            <type>text/plain</type>
            <size>1279</size>
            <attacher name="George Wright">gwright</attacher>
            
              <data encoding="base64">SW5kZXg6IENoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBDaGFuZ2VMb2cJKHJldmlzaW9uIDI1MjEz
KQorKysgQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMTYgQEAKKzIwMDctMDgt
MjQgIEdlb3JnZSBXcmlnaHQgIDxnZW9yZ2Uud3JpZ2h0QGNvbGxhYm9yYS5jby51az4KKworICAg
ICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBodHRwOi8vYnVncy53
ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xNTA3MQorICAgICAgICBbY2Fpcm9dIFNWRyBza2V3
cyBhcmUgaW5jb3JyZWN0CisKKyAgICAgICAgRml4IENhaXJvIGltcGxlbWVudGF0aW9uIG9mIEFm
ZmluZVRyYW5zZm9ybTo6c2hlYXIgc28gdGhhdCBzaGVhcmluZyBpcworICAgICAgICBkb25lIGlu
IHRoZSBjb3JyZWN0IGRpcmVjdGlvbi4KKworICAgICAgICAqIHBsYXRmb3JtL2dyYXBoaWNzL2Nh
aXJvL0FmZmluZVRyYW5zZm9ybUNhaXJvLmNwcDoKKyAgICAgICAgKFdlYkNvcmU6OkFmZmluZVRy
YW5zZm9ybTo6c2hlYXIpOgorCiAyMDA3LTA4LTIzICBBbmRlcnMgQ2FybHNzb24gIDxhbmRlcnNj
YUBhcHBsZS5jb20+CiAKICAgICAgICAgUmV2aWV3ZWQgYnkgU3RldmUuCkluZGV4OiBwbGF0Zm9y
bS9ncmFwaGljcy9jYWlyby9BZmZpbmVUcmFuc2Zvcm1DYWlyby5jcHAKPT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0g
cGxhdGZvcm0vZ3JhcGhpY3MvY2Fpcm8vQWZmaW5lVHJhbnNmb3JtQ2Fpcm8uY3BwCShyZXZpc2lv
biAyNTIxMykKKysrIHBsYXRmb3JtL2dyYXBoaWNzL2NhaXJvL0FmZmluZVRyYW5zZm9ybUNhaXJv
LmNwcAkod29ya2luZyBjb3B5KQpAQCAtMjIxLDcgKzIyMSw3IEBAIEFmZmluZVRyYW5zZm9ybSAm
QWZmaW5lVHJhbnNmb3JtOjp0cmFuc2wKIEFmZmluZVRyYW5zZm9ybSAmQWZmaW5lVHJhbnNmb3Jt
OjpzaGVhcihkb3VibGUgc3gsIGRvdWJsZSBzeSkKIHsKICAgICBjYWlyb19tYXRyaXhfdCBzaGVh
cjsKLSAgICBjYWlyb19tYXRyaXhfaW5pdCgmc2hlYXIsIDEsIHN4LCBzeSwgMSwgMCwgMCk7Cisg
ICAgY2Fpcm9fbWF0cml4X2luaXQoJnNoZWFyLCAxLCBzeSwgc3gsIDEsIDAsIDApOwogCiAgICAg
Y2Fpcm9fbWF0cml4X3QgcmVzdWx0OwogICAgIGNhaXJvX21hdHJpeF9tdWx0aXBseSgmcmVzdWx0
LCAmc2hlYXIsICZtX3RyYW5zZm9ybSk7Cg==
</data>
<flag name="review"
          id="6763"
          type_id="1"
          status="+"
          setter="oliver"
    />
          </attachment>
      

    </bug>

</bugzilla>