[webkit-reviews] review denied: [Bug 14998] Full page zooming support : [Attachment 16218] Small changes to conform to coding style, general tidying up

[bugzilla-daemon at webkit.org]

Oliver Hunt <oliver at apple.com> has denied George Wright
<george.wright at collabora.co.uk>'s request for review:
Bug 14998: Full page zooming support
http://bugs.webkit.org/show_bug.cgi?id=14998

Attachment 16218: Small changes to conform to coding style, general tidying up
http://bugs.webkit.org/attachment.cgi?id=16218&action=edit

------- Additional Comments from Oliver Hunt <oliver at apple.com>
Scaling the render region outside of the engine itself can cause also of
badness, especially with hit testing, region invalidation and what not.

I think you'd benefit from matching the WebView API from mac.

– canMakeTextLarger  
– makeTextLarger:  
– canMakeTextSmaller  
– makeTextSmaller:

Which would be on gtk WebView equivalent -- the Mac impl is kind of messy, look
at the windows ones as they do much less icky objc-fu.

makeTextSmaller/Larger are designed to allow buttons to be hooked up directly
-- i think that would mean you want them to be "slots"/"sockets" or whatever
the gtk name for something like that is.