Bug 12620 - "values" attribute of <animateTransform type="rotate"> is divided by 10.
Summary: "values" attribute of <animateTransform type="rotate"> is divided by 10.
Status: RESOLVED DUPLICATE of bug 12565
Alias: None
Product: WebKit
Classification: Unclassified
Component: SVG (show other bugs)
Version: 420+
Hardware: Mac (PowerPC) OS X 10.4
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-02-05 13:13 PST by Francis Avila
Modified: 2007-02-10 17:31 PST (History)
0 users

See Also:


Attachments
Test Case (383 bytes, image/svg+xml)
2007-02-05 13:15 PST, Francis Avila
no flags Details
fix values= parsing (1.98 KB, patch)
2007-02-06 05:36 PST, Eric Seidel (no email)
mjs: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Francis Avila 2007-02-05 13:13:35 PST
The values of the "values" attribute are read as divided by 10, at least during a rotation transform.  Thus in the following SVG, the stroke rotates not 30, 60, and 90 degrees but 3, 6, and 9.  ("values='0;300;600;900'" will give the behavior desired here.)

<?xml version="1.0" standalone="yes"?>
<!DOCTYPE svg PUBLIC "-//W3C//DTD SVG 1.1//EN" 
		 "http://www.w3.org/Graphics/SVG/1.1/DTD/svg11.dtd">
<svg xmlns="http://www.w3.org/2000/svg">
     <line x2="50" stroke="black" stroke-width="10" >
 		<animateTransform
     		type="rotate" calcMode="discrete" repeatCount="indefinite" dur="1s"
     		values="0;300;600;900"/>
   	</line>
</svg>
Comment 1 Francis Avila 2007-02-05 13:15:01 PST
Created attachment 12942 [details]
Test Case
Comment 2 Eric Seidel (no email) 2007-02-05 20:00:52 PST
This hits an ASSERT and crashes, so it's hard to verify on a debug build.

ASSERTION FAILED: m_type == toSVGTransform.type()
(/Stuff/Projects/WebKit/WebCore/ksvg2/svg/SVGTransformDistance.cpp:55 WebCore::SVGTransformDistance::SVGTransformDistance(const WebCore::SVGTransform&, const WebCore::SVGTransform&))
Comment 3 Eric Seidel (no email) 2007-02-06 05:36:23 PST
Created attachment 12969 [details]
fix values= parsing
Comment 4 Maciej Stachowiak 2007-02-07 02:56:59 PST
Comment on attachment 12969 [details]
fix values= parsing

r=me
Comment 5 David Kilzer (:ddkilzer) 2007-02-10 17:31:27 PST

*** This bug has been marked as a duplicate of 12565 ***