Bug 18927

Summary: Unify path data debug output
Product: WebKit Reporter: Rob Buis <rwlbuis>
Component: SVGAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal    
Priority: P3    
Version: 528+ (Nightly build)   
Hardware: Mac   
OS: OS X 10.5   
Attachments:
Description Flags
Unified code and text dumps for Mac Tiger eric: review+

Rob Buis
Reported 2008-05-07 12:02:34 PDT
It turns out that dump to text for path data is different on the various platforms, as noted by Ariya. Together we settled on a mixed version between how mac and how Qt outputs the path data, namely using a space after each path command: old mac (M100.00,100.00L200.00,100.00) new proposed (M100.00,100.00 L200.00,100.00) This allow a bit more readable output, but does not add too much extra space(s).
Attachments
Unified code and text dumps for Mac Tiger (974.29 KB, patch)
2008-05-07 12:20 PDT, Rob Buis 		eric: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Rob Buis
Comment 1 2008-05-07 12:20:59 PDT
Created attachment 21005 [details] Unified code and text dumps for Mac Tiger I hope I got the CFString stuff right. Unfortunately the patch is big... Cheers, Rob.
Eric Seidel (no email)
Comment 2 2008-05-20 14:06:38 PDT
Comment on attachment 21005 [details] Unified code and text dumps for Mac Tiger I'm OK with this. You'll of course need to re-baseline again.
Rob Buis
Comment 3 2008-05-25 03:44:18 PDT
Fixed by r34117.
Note You need to log in before you can comment on or make changes to this bug.