[webkit-reviews] review denied: [Bug 12221] Webkit ToT fails to render this pattern example : [Attachment 16232] Initial pattern rework patch

[bugzilla-daemon at webkit.org]

Oliver Hunt <oliver at apple.com> has denied Nikolas Zimmermann
<zimmermann at kde.org>'s request for review:
Bug 12221: Webkit ToT fails to render this pattern example
http://bugs.webkit.org/show_bug.cgi?id=12221

Attachment 16232: Initial pattern rework patch
http://bugs.webkit.org/attachment.cgi?id=16232&action=edit

------- Additional Comments from Oliver Hunt <oliver at apple.com>
Is this safe?
	 if (!attributes.hasX() && current->hasAttribute(SVGNames::xAttr))

This seems really bad to me:

+    CGContextSetPatternPhase(contextRef, CGSizeMake(0.0, -0.01));
What happens under heavy scaling?



-	     attributes.setX(current->x().valueAsPercentage());
+	     attributes.setX(current->x());