Bug 4553 - QuartzSupport::CGPathToCFStringApplierFunction incorrect
Summary: QuartzSupport::CGPathToCFStringApplierFunction incorrect
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: Eric Seidel (no email)
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-08-21 03:21 PDT by Tobias Lidskog
Modified: 2005-08-22 02:29 PDT (History)
0 users

See Also:

Attachments
patch + updated tests (269.65 KB, patch)
2005-08-21 03:22 PDT, Tobias Lidskog 		no flags Details | Formatted Diff | Diff
updated diff (266.16 KB, patch)
2005-08-21 19:41 PDT, Tobias Lidskog 		eric: review+
Details | Formatted Diff | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Tobias Lidskog 2005-08-21 03:21:42 PDT 
Converting a CGPath to a string is done in an incorrect way.
Quadratic curves (kCGPathElementAddQuadCurveToPoint) should result in two points, currently four 
points are printed.
Cubic curves (kCGPathElementAddCurveToPoint) should result in three points, currently six points are 
printed.
Comment 1 Tobias Lidskog 2005-08-21 03:22:19 PDT 
Created attachment 3490 [details]
patch + updated tests
Comment 2 Tobias Lidskog 2005-08-21 19:41:40 PDT 
Created attachment 3500 [details]
updated diff

The previous diff accidently included a patch for another bug as well.
Comment 3 Eric Seidel (no email) 2005-08-22 02:14:00 PDT 
Comment on attachment 3500 [details]
updated diff

Wow. Yeah, serious oversight on my part.  This was causing problems with the
layout tests too...  Thanks for the nice fix.