[webkit-reviews] review denied: [Bug 12013] REGRESSION: <pattern> in objectBoundingBox mode are broken. : [Attachment 12099] Updated patch

[bugzilla-request-daemon at macosforge.org]

Eric Seidel <macdome at opendarwin.org> has denied Eric Seidel
<macdome at opendarwin.org>'s request for review:
Bug 12013: REGRESSION: <pattern> in objectBoundingBox mode are broken.
http://bugs.webkit.org/show_bug.cgi?id=12013

Attachment 12099: Updated patch
http://bugs.webkit.org/attachment.cgi?id=12099&action=edit

------- Additional Comments from Eric Seidel <macdome at opendarwin.org>
Ok, the code looks good.

1.  Need to remove the 1x1 pruning in SVGRenderContainer
2. Need to file a bug about the /100 issue
3.  You said you were going to fix gradients to be lazy
4. Need to add some test cases, including one for failed mask allocations.  You
said you created a bunch of crazy tests, but I don't see them here.  The test
cases are almost more important than the code changes as once we have the
tests, we'll never break them.
5.  
+	 IntRect textBoundary =
const_cast<RenderObject*>(object)->absoluteBoundingBoxRect();
+	 targetRect =
object->absoluteTransform().inverse().mapRect(textBoundary);
seems kinda hacky.  I'm surprise there isn't a method to get you what you want
there.

Otherwise looks great!