[webkit-reviews] review denied: [Bug 27430] adds glib / gobject bindings to webkit gtk port : [Attachment 33059] adds use of glib / gobject bindings to webkitwebframe

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Jul 31 20:05:47 PDT 2009


Eric Seidel <eric at webkit.org> has denied Luke Kenneth Casson Leighton
<lkcl at lkcl.net>'s request for review:
Bug 27430: adds glib / gobject bindings to webkit gtk port
https://bugs.webkit.org/show_bug.cgi?id=27430

Attachment 33059: adds use of glib / gobject bindings to webkitwebframe
https://bugs.webkit.org/attachment.cgi?id=33059&action=review

------- Additional Comments from Eric Seidel <eric at webkit.org>
WebCore avoids using superfluous argument names in headers:
 91	gpointer toGDOM(DOMWindow* doc);
 92	gpointer toGDOM(Document* doc);
 93	gpointer toGDOM(XMLHttpRequest* obj);

We don't commit commented out code:
 407	 //WebKitWebFramePrivate* priv = frame->priv;

Name:
 2536 2008-11-30  lkcl	<lkcl at lkcl.net>

Otherwise looks fine.  I mean, I'm not really a gtk expert.  There are style
violations, but I assume this stuff is more supposed to be more gtk style than
webkit style, not sure.

r- for the above nits.	If you were a committer I could just r+ it and let you
fix it as you landed.


More information about the webkit-reviews mailing list