[webkit-reviews] review granted: [Bug 59082] [GTK] DRT: implement eventSender.scalePageBy : [Attachment 90504] proposed patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Apr 21 08:59:57 PDT 2011


Martin Robinson <mrobinson at webkit.org> has granted Philippe Normand
<pnormand at igalia.com>'s request for review:
Bug 59082: [GTK] DRT: implement eventSender.scalePageBy
https://bugs.webkit.org/show_bug.cgi?id=59082

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

------- Additional Comments from Martin Robinson <mrobinson at webkit.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=90504&action=review

Awesome! Thanks for implementing this.

> Source/WebKit/gtk/WebCoreSupport/DumpRenderTreeSupportGtk.cpp:787
> +void DumpRenderTreeSupportGtk::scale(WebKitWebView* webView, float
scaleFactor, float x, float y)

Please name this method scalePageBy to match the name of the callback in DRT.
This will make it more obvious why this is needed above the scaling options we
already have in our API.


More information about the webkit-reviews mailing list