RESOLVED FIXED 96781
[GTK] Don't use the C API internally in PolicyClient classes 
https://bugs.webkit.org/show_bug.cgi?id=96781
Summary [GTK] Don't use the C API internally in PolicyClient classes
Carlos Garcia Campos
Reported 2012-09-14 07:21:23 PDT
Use the C++ classes instead.
Attachments
Patch (19.29 KB, patch)
2012-09-14 07:22 PDT, Carlos Garcia Campos 		xan.lopez: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Carlos Garcia Campos
Comment 1 2012-09-14 07:22:37 PDT
Created attachment 164144 [details] Patch
WebKit Review Bot
Comment 2 2012-09-14 07:26:29 PDT
Thanks for the patch. If this patch contains new public API please make sure it follows the guidelines for new WebKit2 GTK+ API. See http://trac.webkit.org/wiki/WebKitGTK/AddingNewWebKit2API
Mario Sanchez Prada
Comment 3 2012-10-03 02:08:05 PDT
Comment on attachment 164144 [details] Patch Looks good to me
Xan Lopez
Comment 4 2012-10-18 05:09:54 PDT
Comment on attachment 164144 [details] Patch Yeah, looks good. Same comment about the header stuff.
Carlos Garcia Campos
Comment 5 2012-10-18 05:22:51 PDT
Committed r131731: <http://trac.webkit.org/changeset/131731>
Note You need to log in before you can comment on or make changes to this bug.