RESOLVED FIXED 6074
WebKit+SVG and FireFox disagree on invalid "transform" handling 
https://bugs.webkit.org/show_bug.cgi?id=6074
Summary WebKit+SVG and FireFox disagree on invalid "transform" handling
Eric Seidel (no email)
Reported Wednesday, December 14, 2005 9:39:38 AM UTC
WebKit+SVG and FireFox disagree on invalid "transform" handling After my fixes as part of 6061, there is now one case in which WebKit+SVG and FireFox still disagree on transform handling. WebKit+SVG will respect any valid transforms up-to, but not including the first invalid transform instruction in a transform="..." attribute. FireFox will ignore the entire transform attribute if any of the transform instructions are invalid.
Attachments
Test case (596 bytes, image/svg+xml)
2005-12-14 01:41 PST, Eric Seidel (no email) 		no flags
Details
First attempt (17.46 KB, patch)
2006-12-04 13:29 PST, Rob Buis 		no flags
DetailsFormatted DiffDiff
Better patch (23.57 KB, patch)
2006-12-06 12:27 PST, Rob Buis 		darin: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Eric Seidel (no email)
Comment 1 Wednesday, December 14, 2005 9:41:07 AM UTC
Created attachment 5075 [details] Test case Safari+SVG "passes" this test and FireFox "fails". Eventually we should resolve this discrepancy. The w3c does not specify which should be correct, however I may ask them to as part of Tiny 1.2 last call.
Ian 'Hixie' Hickson
Comment 2 Wednesday, December 28, 2005 9:44:38 PM UTC
Per SVG Tiny 1.2, the entire attribute should be ignored. TESTCASE: http://www.hixie.ch/tests/adhoc/svg/transform/001.xml
Rob Buis
Comment 3 Monday, December 4, 2006 9:29:01 PM UTC
Created attachment 11729 [details] First attempt This patch makes us match FF behaviour here. Cheers, Rob.
Rob Buis
Comment 4 Wednesday, December 6, 2006 8:27:10 PM UTC
Created attachment 11753 [details] Better patch I had one big logic error in SVGTextElement.cpp, fixed now. Also added updated results for transform-ignore-after-invalid.svg. Finally some code cleanups. Cheers, Rob.
Darin Adler
Comment 5 Thursday, December 7, 2006 11:02:20 PM UTC
Comment on attachment 11753 [details] Better patch Looks good. r=me
Rob Buis
Comment 6 Friday, December 8, 2006 8:38:47 AM UTC
Landed in r18074.
Note You need to log in before you can comment on or make changes to this bug.