[webkit-reviews] review granted: [Bug 73631] [chromium] Add layout test support for autosize code in FrameView. : [Attachment 121485] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Jan 6 14:03:24 PST 2012


Dmitry Titov <dimich at chromium.org> has granted David Levin
<levin at chromium.org>'s request for review:
Bug 73631: [chromium] Add layout test support for autosize code in FrameView.
https://bugs.webkit.org/show_bug.cgi?id=73631

Attachment 121485: Patch
https://bugs.webkit.org/attachment.cgi?id=121485&action=review

------- Additional Comments from Dmitry Titov <dimich at chromium.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=121485&action=review


r+ with a nit. If there will be more tests in this dir, lets share things like
shouldBe() between them.

> Tools/DumpRenderTree/chromium/LayoutTestController.cpp:1107
> +	   return;

result->set(false); ?


More information about the webkit-reviews mailing list