[webkit-reviews] review granted: [Bug 57984] WebKitTestRunner needs layoutTestController.pageSizeAndMarginsInPixels : [Attachment 88514] fix patch

[bugzilla-daemon at webkit.org]

Darin Adler <darin at apple.com> has granted Chang Shu <cshu at webkit.org>'s request
for review:
Bug 57984: WebKitTestRunner needs
layoutTestController.pageSizeAndMarginsInPixels
https://bugs.webkit.org/show_bug.cgi?id=57984

Attachment 88514: fix patch
https://bugs.webkit.org/attachment.cgi?id=88514&action=review

------- Additional Comments from Darin Adler <darin at apple.com>
View in context: https://bugs.webkit.org/attachment.cgi?id=88514&action=review

> Source/WebKit2/WebProcess/InjectedBundle/InjectedBundle.cpp:183
> +	   return "";

I suggest String() instead of "".

> Source/WebKit2/WebProcess/InjectedBundle/InjectedBundle.cpp:186
> +    return PrintContext::pageSizeAndMarginsInPixels(coreFrame, pageIndex,
width, height,
> +						       marginTop, marginRight,
marginBottom, marginLeft);

We don’t line up lines like this in WebKit. I suggest either doing a 4 pixel
indent rather than lining things up or just putting all the arguments into one
line.