Bug 12004

Summary: Webkit doesnt handle omitting M at the start of path syntax well
Product: WebKit Reporter: Rob Buis <rwlbuis>
Component: SVGAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal    
Priority: P2    
Version: 420+   
Hardware: Mac   
OS: OS X 10.4   
Attachments:
Description Flags
First attempt eric: review+

Rob Buis
Reported 2006-12-27 13:33:44 PST
A path syntax like d='L200 0 L 200 200 L 0 200 Z L asdlkfjasdfkja' causes these warnings: CGPathAddLineToPoint: no current point. CGPathAddLineToPoint: no current point. CGPathAddLineToPoint: no current point. CGPathCloseSubpath: no current point. CGPathAddLineToPoint: no current point. CGPathAddCurveToPoint: no current point. CGPathAddLineToPoint: no current point. CGPathAddLineToPoint: no current point. CGPathAddLineToPoint: no current point. CGPathCloseSubpath: no current point. CGPathAddLineToPoint: no current point. CGPathAddCurveToPoint: no current point. Cheers, Rob.
Attachments
First attempt (47.50 KB, patch)
2006-12-30 12:59 PST, Rob Buis 		eric: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Rob Buis
Comment 1 2006-12-30 12:59:01 PST
Created attachment 12127 [details] First attempt The code changes are ok now I think, though the testcase may need some work. Cheers, Rob.
Eric Seidel (no email)
Comment 2 2006-12-30 14:40:41 PST
Comment on attachment 12127 [details] First attempt The patch looks great. We could always use more tests though.
Rob Buis
Comment 3 2006-12-31 07:10:51 PST
Landed in r18489.
Note You need to log in before you can comment on or make changes to this bug.