Bug 6042

Summary: "S" and "T" path types control points errors
Product: WebKit Reporter: Julien Palmas <julien.palmas>
Component: SVGAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: rwlbuis
Priority: P2    
Version: 420+   
Hardware: Mac   
OS: OS X 10.4   
Attachments:
Description Flags
Test case
none
Expected result
none
Actuall result
none
Should fix this path problem
none
Another attempt atfixing this bug eric: review+

Description Julien Palmas 2005-12-11 07:25:02 PST 
The spec is not respected.

In the attached test case, the "S" command should set its first control point to be the current point and the 
"T" command should set its only control point to be the current point as well.

FF renders the test case correctly
Comment 1 Julien Palmas 2005-12-11 07:26:00 PST 
Created attachment 5031 [details]
Test case
Comment 2 Julien Palmas 2005-12-11 07:31:55 PST 
Created attachment 5032 [details]
Expected result
Comment 3 Julien Palmas 2005-12-11 07:32:15 PST 
Created attachment 5033 [details]
Actuall result
Comment 4 Rob Buis 2005-12-11 08:53:29 PST 
Created attachment 5035 [details]
Should fix this path problem

This should fix it, again pleasetest well :)
It doesnt seem to regress on my system.
Cheers,

Rob.
Comment 5 Rob Buis 2005-12-11 09:33:41 PST 
Created attachment 5036 [details]
Another attempt atfixing this bug

Previous one was wrong, since the test was done afterwards.
This one should work.
Cheers,

Rob.
Comment 6 Julien Palmas 2005-12-11 09:39:09 PST 
This patch looks good to me.
Nice Rob!
Comment 7 Eric Seidel (no email) 2005-12-12 09:26:22 PST 
Comment on attachment 5036 [details]
Another attempt atfixing this bug

I dislike the existing variable names, but the patch itself looks fine.