Bug 6042 - "S" and "T" path types control points errors
Summary: "S" and "T" path types control points errors
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: SVG (show other bugs)
Version: 420+
Hardware: Mac OS X 10.4
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-12-11 07:25 PST by Julien Palmas
Modified: 2005-12-12 09:36 PST (History)
1 user (show)

See Also:


Attachments
Test case (362 bytes, image/svg+xml)
2005-12-11 07:26 PST, Julien Palmas
no flags Details
Expected result (6.49 KB, image/png)
2005-12-11 07:31 PST, Julien Palmas
no flags Details
Actuall result (7.25 KB, image/png)
2005-12-11 07:32 PST, Julien Palmas
no flags Details
Should fix this path problem (1.27 KB, patch)
2005-12-11 08:53 PST, Rob Buis
no flags Details | Formatted Diff | Diff
Another attempt atfixing this bug (1.79 KB, patch)
2005-12-11 09:33 PST, Rob Buis
eric: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
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.