RESOLVED FIXED 96772
[GTK] Don't use the C API internally in WebKitCookieManager 
https://bugs.webkit.org/show_bug.cgi?id=96772
Summary [GTK] Don't use the C API internally in WebKitCookieManager
Carlos Garcia Campos
Reported 2012-09-14 06:52:36 PDT
Use the C++ classes instead.
Attachments
Patch (9.78 KB, patch)
2012-09-14 06:54 PDT, Carlos Garcia Campos 		gustavo: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Carlos Garcia Campos
Comment 1 2012-09-14 06:54:20 PDT
Created attachment 164132 [details] Patch
Mario Sanchez Prada
Comment 2 2012-10-03 01:34:49 PDT
Comment on attachment 164132 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=164132&action=review Looks good to me > Source/WebKit2/UIProcess/API/gtk/WebKitCookieManager.cpp:37 > + RefPtr<WebCookieManagerProxy> webCookieManager; As I commented in another patch, I would probably not use the "web" prefix here. Just cookieManager would be enough, IMO
Carlos Garcia Campos
Comment 3 2012-10-03 01:38:42 PDT
(In reply to comment #2) > (From update of attachment 164132 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=164132&action=review > > Looks good to me > > > Source/WebKit2/UIProcess/API/gtk/WebKitCookieManager.cpp:37 > > + RefPtr<WebCookieManagerProxy> webCookieManager; > > As I commented in another patch, I would probably not use the "web" prefix here. Just cookieManager would be enough, IMO Class name is WebCookieManagerProxy, I don't see the problem of suing the web prefix
Carlos Garcia Campos
Comment 4 2012-10-17 08:56:40 PDT
Committed r131607: <http://trac.webkit.org/changeset/131607>
Note You need to log in before you can comment on or make changes to this bug.