<?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>114578</bug_id>
          
          <creation_ts>2013-04-13 23:01:00 -0700</creation_ts>
          <short_desc>vector-effect: non-scaling-stroke not recognized when animating</short_desc>
          <delta_ts>2013-04-25 11:55:37 -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>528+ (Nightly build)</version>
          <rep_platform>All</rep_platform>
          <op_sys>All</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Stuart P. Bentley">stuart</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>kling</cc>
    
    <cc>krit</cc>
    
    <cc>pdr</cc>
    
    <cc>zimmermann</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>875285</commentid>
    <comment_count>0</comment_count>
      <attachid>197972</attachid>
    <who name="Stuart P. Bentley">stuart</who>
    <bug_when>2013-04-13 23:01:00 -0700</bug_when>
    <thetext>Created attachment 197972
Test case

Strokes with the style {vector-effect: non-scaling-stroke} are scaled when transforming an SVG by setting an otherwise-unspecified viewBox via animation.

Tested on Windows 7 @ Chrome stable, Windows 8 @ Chrome dev, Arch Linux @ Chromium 26.0.1410.63, and Android 4.2.1 @ Chrome latest.

See attached test case.

In writing this test case, I found that the &quot;Desired behavior&quot; (employing a workaround by setting the viewBox attribute to the end result) produces artifacts when animating on Windows and Linux. This is not part of the desired behavior, and I will file a bug summarily.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>875286</commentid>
    <comment_count>1</comment_count>
      <attachid>197973</attachid>
    <who name="Stuart P. Bentley">stuart</who>
    <bug_when>2013-04-13 23:06:47 -0700</bug_when>
    <thetext>Created attachment 197973
Unrelated artifacts

When noting the desired behavior, ignore the creation of artifacts like those outlined in red in this screenshot.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>875290</commentid>
    <comment_count>2</comment_count>
    <who name="Stuart P. Bentley">stuart</who>
    <bug_when>2013-04-13 23:33:16 -0700</bug_when>
    <thetext>Unrelated artifact bug filed as Bug 114579.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>875293</commentid>
    <comment_count>3</comment_count>
    <who name="Stuart P. Bentley">stuart</who>
    <bug_when>2013-04-13 23:34:53 -0700</bug_when>
    <thetext>(In reply to comment #0)

&gt; Tested on Windows 7 @ Chrome stable, Windows 8 @ Chrome dev, Arch Linux @ Chromium 26.0.1410.63, and Android 4.2.1 @ Chrome latest.

s/Windows 7 @ Chrome stable, Windows 8 @ Chrome dev/Windows 7 @ Chrome dev, Windows 8 @ Chrome stable/</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>875297</commentid>
    <comment_count>4</comment_count>
      <attachid>197978</attachid>
    <who name="Stuart P. Bentley">stuart</who>
    <bug_when>2013-04-13 23:59:04 -0700</bug_when>
    <thetext>Created attachment 197978
Test case

Further testing has revealed that *any* value on the viewBox attribute will inhibit this bug. Uploading simplified test case with viewBox=&quot;&quot;.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>875442</commentid>
    <comment_count>5</comment_count>
    <who name="Dirk Schulze">krit</who>
    <bug_when>2013-04-14 20:41:26 -0700</bug_when>
    <thetext>I agree that this is a bug. It looks like we do not take the viewBox transform into account for reverse transforming.

Thanks for reporting.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>881212</commentid>
    <comment_count>6</comment_count>
    <who name="Andreas Kling">kling</who>
    <bug_when>2013-04-25 05:56:35 -0700</bug_when>
    <thetext>Merged the fix for this from blink in &lt;http://trac.webkit.org/changeset/149102&gt; (thanks pdr!)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>881471</commentid>
    <comment_count>7</comment_count>
    <who name="Philip Rogers">pdr</who>
    <bug_when>2013-04-25 11:55:37 -0700</bug_when>
    <thetext>(In reply to comment #6)
&gt; Merged the fix for this from blink in &lt;http://trac.webkit.org/changeset/149102&gt; (thanks pdr!)

Aww man, I was preparing a patch for WebKit.

Thanks for merging this though.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="0"
              isprivate="0"
          >
            <attachid>197972</attachid>
            <date>2013-04-13 23:01:00 -0700</date>
            <delta_ts>2013-04-13 23:59:04 -0700</delta_ts>
            <desc>Test case</desc>
            <filename>testcase.html</filename>
            <type>text/html</type>
            <size>831</size>
            <attacher name="Stuart P. Bentley">stuart</attacher>
            
              <data encoding="base64">PCFET0NUWVBFIGh0bWw+DQo8aHRtbD4NCjxoZWFkPg0KPG1ldGEgY2hhcnNldD11dGYtOCAvPg0K
PHRpdGxlPnZlY3Rvci1lZmZlY3Q6IG5vbi1zY2FsaW5nLXN0cm9rZSBub3QgcmVjb2duaXplZCB3
aGVuIGFuaW1hdGluZzwvdGl0bGU+DQo8c3R5bGU+DQpzdmcgew0KICBkaXNwbGF5OiBibG9jazsN
CiAgaGVpZ2h0OiAyNTZweDsNCiAgd2lkdGg6IDI1NnB4Ow0KfQ0KDQpzdmcgcGF0aCB7DQogIHN0
cm9rZTogYmxhY2s7DQogIHN0cm9rZS13aWR0aDogMXB4Ow0KICBzdHJva2UtbGluZWNhcDogcm91
bmQ7DQogIHZlY3Rvci1lZmZlY3Q6IG5vbi1zY2FsaW5nLXN0cm9rZTsNCn0NCjwvc3R5bGU+DQo8
L2hlYWQ+DQo8Ym9keT4NCiAgPGgyPkFjdHVhbCBiZWhhdmlvcjwvaDI+DQogIDxzdmcgaWQ9ImFj
dHVhbCIgcHJlc2VydmVBc3BlY3RSYXRpbz0ibm9uZSI+DQogICAgPHBhdGggZD0iTSAxLDAgTDEs
NSIgLz4NCiAgICA8YW5pbWF0ZSBhdHRyaWJ1dGVOYW1lPSJ2aWV3Qm94IiBmcm9tPSIwIDAgNSA1
IiB0bz0iMCAwIDIgNSIgZHVyPSI1cyIgYmVnaW49IjBzIiBmaWxsPSJmcmVlemUiLz4NCiAgPC9z
dmc+DQogIDxoMj5EZXNpcmVkIGJlaGF2aW9yPC9oMj4NCiAgPHN2ZyBpZD0iZXhwZWN0ZWQiIHBy
ZXNlcnZlQXNwZWN0UmF0aW89Im5vbmUiIHZpZXdCb3g9IjAgMCAyIDUiPg0KICAgIDxwYXRoIGQ9
Ik0gMSwwIEwxLDUiIC8+DQogICAgPGFuaW1hdGUgYXR0cmlidXRlTmFtZT0idmlld0JveCIgZnJv
bT0iMCAwIDUgNSIgdG89IjAgMCAyIDUiIGR1cj0iNXMiIGJlZ2luPSIwcyIgZmlsbD0iZnJlZXpl
Ii8+DQogIDwvc3ZnPg0KPC9ib2R5Pg0KPC9odG1sPg0K
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="0"
              isprivate="0"
          >
            <attachid>197973</attachid>
            <date>2013-04-13 23:06:47 -0700</date>
            <delta_ts>2013-04-13 23:06:47 -0700</delta_ts>
            <desc>Unrelated artifacts</desc>
            <filename>artifacts.png</filename>
            <type>image/png</type>
            <size>2424</size>
            <attacher name="Stuart P. Bentley">stuart</attacher>
            
              <data encoding="base64">iVBORw0KGgoAAAANSUhEUgAAARgAAAFSCAMAAAAekzC0AAAAAXNSR0IArs4c6QAAAARnQU1BAACx
jwv8YQUAAAMAUExURQAAAAAAVVUAAHt7e3x8fAAAhABVowCEwlWEwlWj4YQAAKNVAP8AAIRVo6NV
hMKEAMKEVeGjVYSjo6urq7q6uoSEwqPCo4TC+aPh+cLChMLho+HChPnChPnCo/nho8LCwsbGxsrK
ys7OztLS0tXV1dnZ2d3d3cL54cL5+fn5wuXl5e3t7eH5+fn54fHx8fX19fn5+QAAAAAAAAAAAAAA
AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA
AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA
AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA
AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA
AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA
AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA
AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA
AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA
AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA
AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA
AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAKtGx14AAAAJcEhZcwAADsMA
AA7DAcdvqGQAAAAadEVYdFNvZnR3YXJlAFBhaW50Lk5FVCB2My41LjEwMPRyoQAABdtJREFUeF7t
1ut220QUQGG3kJaEi3G5uECAJiGlbkur9386zpkZWbKt3WStRIkpe/+wZXkkzXyWnSw6m0wYSBhI
GEgYSBhIGEgYSBhIGEgYSBiowiwXfU9elh3TLRcnb9rmZP98PT7DH0/j1bq9GEX7d7vpWnPX3zFJ
E5ONJ6bJFZ22beh8fHyOnwKg/eNuca1562FiRWWyq0/J3PwpxnruB+Zo7pgeZnO2uMuM7g/msduH
yVvmDvfwZwyTG8/ex0beO2WR+ZN6er4u6wmz8gv77H2MS7/tqPr79PMBTOwtpxsNLfvjoe3PjyLP
VU4cG/H6ebtWlPPZTmO47uwdwOSU8rt0Ho+xlNi3jIdVbuTKclK51t/KLEejYt4nb/LgXZjFOt4u
381haO5/ke+lzOYsHtOi7o+nbhV7+mut6lHFYnTd+ZuEibXFU0xmFZOoi4k7ZjTZXMAytodReYKy
uX/HlH111f3Q3F8++gSL1a7z4NyOkfn0fZyhXas/qsxtuO4DRDBlvjGVJy97jh2YtvphVN7osRlH
H8DUVQ9D2/42NE522sPkBeKNr+LYdq04Kh7j7XInjdXnDr5K9fuerXNyZdpjmPK6/Spk+YXJ4xFm
fMIdmCwPLmeMSz3vVjmPeq121ua2ve5DBD++uY56m9Q5ljXUyZYJ1q/5aFRbwidh+hPuwWzOTn5p
S86hf3+XWwDzID8vpQOYuHrMKNeRd28t75mY0iRMG3UbmP6EuzDLOFlbeTnhi/wmHR9MTi0nERZl
2r+XZca78eoAZjQqj4v3ACZIRifcgYn92x/fcup6fLtWvBuPsbf9xjweTCWor2Ou8UO4+TYmk381
J2CGUW3yUzDlB3c8dAxTz9mr5ct66natOKrC5NweBSam1iZSP7qyo7xo/2jEmz1MvNVPcDsq36x/
weqYLAHKn/uyYxi6D9P+q8m1x6h69HCt8n9M+RaOrjt/FaZOOmssUX7EOY/NT7mQmGQ+xW2RX5qo
LGMY1b6Evz4dbpg4oPyx6vf0Q+t5ikW8FXtj57IfdV4n0K6VO3IjL7Z73dnr7xjbSxhIGEgYSBhI
GEgYSBhIGEgYSBhIGEgYSBhIGEgYSBhIGEgYSBhIGEgYSBhIGEgYSBjoKGC+/KJtHFHCQMJAwkDC
QMJAwkDCQMJAwkDCQMJAwkDCQMJAwkDCQMJAwkDCQMJAwkDCQMJAwkDCQMJAwkDCQMJAwkDCQMJA
wkDCQMJAwkDCQMJAwkDCQMJAwkDCQMJAwkDCQMJAwkA3w3wzW+0CwuzXLvDfhWkb95wwkDCQMJAw
kDCQMJAwkDDQZwrzw8XVj1evXl2+/vPt1eXF1dvur+7d9UXXXV5/jM3r7sNlG5i967rX+fyxvOq6
D/kgjDDCRMIIQwkDCQMJAwkDCQMJAwkDCQMJAwkDCQMJAwkDCQMJAwkDCQMJAwkDCQMJAwkDCQMJ
AwkDCQMJAwkDCQMJAwkDCQMJAwkDCQMJAwkDCQMJAwkDCQMJAwkDCQMJAwkDCQMJAwkDCQMJAwkD
CQMJAwkDCQMJAwkDCQMJAwkDCQMJAwkDCQMJAwkDCQMJAwkDCQMJAwkDCQMJAwkDCQMJAwkDCQMJ
AwkDCQMJAwkDCQMJAwkDCQMJAwkDCQMJAwkDCQMJAwkDCQMJAwkDCQMJAwkDCQMJAwkDCQMJAwkD
CQMJAwkDCQMJAwkDCQMJAwkDCQMJAwkDCQMJAwkDCQMJAwkDCQMJAwkDCQMJAwkDCQMJAwkDCQMJ
AwkDCQMJAwkDCQMJAwkDCQMJAwkDCQMJAwkDCQMJAwkDCQMJAwkDCQMJAwkDCQMJAwkDCQMJAwkD
CQMJAwkDCQMJAwkDCQMJAwkDCQMJAwkDCQMJAwkDCQMJAwkDCQMJAwkDCQMJAwkDCQNNw9y9uc57
TwkDCQMJAwkDCQMJAwkDCQMJAwkDCQMJAwkDCQMJA90GZq7aBY4zYaCbYf6nCQMJAwkDCQMJAwkD
CQMJAwkzWdf9Cyu4as4r3pwqAAAAAElFTkSuQmCC
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="0"
              isprivate="0"
          >
            <attachid>197978</attachid>
            <date>2013-04-13 23:59:04 -0700</date>
            <delta_ts>2013-04-13 23:59:04 -0700</delta_ts>
            <desc>Test case</desc>
            <filename>testcase.html</filename>
            <type>text/html</type>
            <size>824</size>
            <attacher name="Stuart P. Bentley">stuart</attacher>
            
              <data encoding="base64">PCFET0NUWVBFIGh0bWw+DQo8aHRtbD4NCjxoZWFkPg0KPG1ldGEgY2hhcnNldD11dGYtOCAvPg0K
PHRpdGxlPnZlY3Rvci1lZmZlY3Q6IG5vbi1zY2FsaW5nLXN0cm9rZSBub3QgcmVjb2duaXplZCB3
aGVuIGFuaW1hdGluZzwvdGl0bGU+DQo8c3R5bGU+DQpzdmcgew0KICBkaXNwbGF5OiBibG9jazsN
CiAgaGVpZ2h0OiAyNTZweDsNCiAgd2lkdGg6IDI1NnB4Ow0KfQ0KDQpzdmcgcGF0aCB7DQogIHN0
cm9rZTogYmxhY2s7DQogIHN0cm9rZS13aWR0aDogMXB4Ow0KICBzdHJva2UtbGluZWNhcDogcm91
bmQ7DQogIHZlY3Rvci1lZmZlY3Q6IG5vbi1zY2FsaW5nLXN0cm9rZTsNCn0NCjwvc3R5bGU+DQo8
L2hlYWQ+DQo8Ym9keT4NCiAgPGgyPkFjdHVhbCBiZWhhdmlvcjwvaDI+DQogIDxzdmcgaWQ9ImFj
dHVhbCIgcHJlc2VydmVBc3BlY3RSYXRpbz0ibm9uZSI+DQogICAgPHBhdGggZD0iTSAxLDAgTDEs
NSIgLz4NCiAgICA8YW5pbWF0ZSBhdHRyaWJ1dGVOYW1lPSJ2aWV3Qm94IiBmcm9tPSIwIDAgNSA1
IiB0bz0iMCAwIDIgNSIgZHVyPSI1cyIgYmVnaW49IjBzIiBmaWxsPSJmcmVlemUiLz4NCiAgPC9z
dmc+DQogIDxoMj5EZXNpcmVkIGJlaGF2aW9yPC9oMj4NCiAgPHN2ZyBpZD0iZXhwZWN0ZWQiIHBy
ZXNlcnZlQXNwZWN0UmF0aW89Im5vbmUiIHZpZXdCb3g9IiI+DQogICAgPHBhdGggZD0iTSAxLDAg
TDEsNSIgLz4NCiAgICA8YW5pbWF0ZSBhdHRyaWJ1dGVOYW1lPSJ2aWV3Qm94IiBmcm9tPSIwIDAg
NSA1IiB0bz0iMCAwIDIgNSIgZHVyPSI1cyIgYmVnaW49IjBzIiBmaWxsPSJmcmVlemUiLz4NCiAg
PC9zdmc+DQo8L2JvZHk+DQo8L2h0bWw+DQo=
</data>

          </attachment>
      

    </bug>

</bugzilla>