Bug 22957

Summary: RenderPath clean-up for strokeContains
Product: WebKit Reporter: Dirk Schulze <krit>
Component: SVGAssignee: Nobody <webkit-unassigned>
Severity: Normal    
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: OS X 10.5   
Description Flags
Move strokeContains back to RenderPath.cpp darin: review+

Description Dirk Schulze 2008-12-21 22:54:54 PST
Move the rest of the RenderPath*.cpp code out of svg to RenderPath and Path.
Comment 1 Dirk Schulze 2008-12-22 03:52:41 PST
Created attachment 26199 [details]
Move strokeContains back to RenderPath.cpp

This patch moves the functionality of strokeContains to RenderPath.cpp, Path and cleares many unneeded parts like CgSupport and platform dependent code like RenderPath*.cpp
Comment 2 Darin Adler 2008-12-23 08:44:37 PST
Comment on attachment 26199 [details]
Move strokeContains back to RenderPath.cpp

> +        bool strokeContains(StrokeStyleApplier* applier, const FloatPoint&) const;

Our style is to omit the argument name when the type makes it clear. Hence, no "applier" needed here.

>          FloatRect strokeBoundingRect(StrokeStyleApplier* applier = 0);

Same here (not changed in this patch).

Comment 3 Dirk Schulze 2009-01-21 23:57:08 PST
landed in r39805.