[Webkit-unassigned] [Bug 96772] [GTK] Don't use the C API internally in WebKitCookieManager

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Oct 3 01:38:17 PDT 2012


https://bugs.webkit.org/show_bug.cgi?id=96772





--- Comment #3 from Carlos Garcia Campos <cgarcia at igalia.com>  2012-10-03 01:38:42 PST ---
(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

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.


More information about the webkit-unassigned mailing list