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

See Also:

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

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