Bug 10866

Summary: Code in kcanvas/ should not use Private
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
Cleanup patch eric: review+

Rob Buis
Reported 2006-09-15 01:14:30 PDT
The old kcanvas code uses a d pointer to minimize changes to the header files. This is not used in webkit, so remove it and conform to webkit style.
Attachments
Cleanup patch (33.32 KB, patch)
2006-09-15 01:24 PDT, Rob Buis 		eric: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Rob Buis
Comment 1 2006-09-15 01:24:20 PDT
Created attachment 10571 [details] Cleanup patch This should be a simple and useful patch... Cheers, Rob.
Eric Seidel (no email)
Comment 2 2006-09-15 14:56:27 PDT
Comment on attachment 10571 [details] Cleanup patch Looks fine. Assuming it passes the layout tests r=me.
Rob Buis
Comment 3 2006-09-16 10:19:19 PDT
Landed in r16393.
Note You need to log in before you can comment on or make changes to this bug.