Bug 6042

Summary:

"S" and "T" path types control points errors

Product:

WebKit

Reporter:

Julien Palmas <julien.palmas>

Component:

SVG

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

rwlbuis

Priority:

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+

Julien Palmas

Reported 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

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
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.

Julien Palmas

Comment 1 2005-12-11 07:26:00 PST

Created attachment 5031 [details] Test case

Julien Palmas

Comment 2 2005-12-11 07:31:55 PST

Created attachment 5032 [details] Expected result

Julien Palmas

Comment 3 2005-12-11 07:32:15 PST

Created attachment 5033 [details] Actuall result

Rob Buis

Comment 4 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.

Rob Buis

Comment 5 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.

Julien Palmas

Comment 6 2005-12-11 09:39:09 PST

This patch looks good to me. Nice Rob!

Eric Seidel (no email)

Comment 7 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.

Note You need to log in before you can comment on or make changes to this bug.