[webkit-reviews] review denied: [Bug 57944] [GTK] Implement WKView with a common widget that can be used by both C and gtk APIs : [Attachment 89758] Updated patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Apr 26 16:05:57 PDT 2011


Martin Robinson <mrobinson at webkit.org> has denied Carlos Garcia Campos
<cgarcia at igalia.com>'s request for review:
Bug 57944: [GTK] Implement WKView with a common widget that can be used by both
C and gtk APIs
https://bugs.webkit.org/show_bug.cgi?id=57944

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

------- Additional Comments from Martin Robinson <mrobinson at webkit.org>
I'm going to r- this patch. On discussing this with Alex, it seems that we
should be able to leave our current WebKit2 C-API design mostly unchanged. Why
remove the WebView class?  It seems there'd be much less code churn if we
simply turn the existing WebViewWidget into the base widget. If that's not
possible, let's discuss alternatives from there.


More information about the webkit-reviews mailing list