Bug 12620

Summary: "values" attribute of <animateTransform type="rotate"> is divided by 10.
Product: WebKit Reporter: Francis Avila <francisga>
Component: SVGAssignee: Nobody <webkit-unassigned>
Status: RESOLVED DUPLICATE    
Severity: Normal    
Priority: P2    
Version: 420+   
Hardware: Mac (PowerPC)   
OS: OS X 10.4   
Attachments:
Description Flags
Test Case
none
fix values= parsing mjs: review+

Francis Avila
Reported 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>
Attachments
Test Case (383 bytes, image/svg+xml)
2007-02-05 13:15 PST, Francis Avila
no flags
fix values= parsing (1.98 KB, patch)
2007-02-06 05:36 PST, Eric Seidel (no email)
mjs: review+
Francis Avila
Comment 1 2007-02-05 13:15:01 PST
Created attachment 12942 [details] Test Case
Eric Seidel (no email)
Comment 2 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&))
Eric Seidel (no email)
Comment 3 2007-02-06 05:36:23 PST
Created attachment 12969 [details] fix values= parsing
Maciej Stachowiak
Comment 4 2007-02-07 02:56:59 PST
Comment on attachment 12969 [details] fix values= parsing r=me
David Kilzer (:ddkilzer)
Comment 5 2007-02-10 17:31:27 PST
*** This bug has been marked as a duplicate of 12565 ***
Note You need to log in before you can comment on or make changes to this bug.