Summary: | [GTK] Don't use the C API internally in WebKitCookieManager | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Carlos Garcia Campos <cgarcia> | ||||
Component: | WebKit2 | Assignee: | Nobody <webkit-unassigned> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | mario | ||||
Priority: | P2 | Keywords: | Gtk | ||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | PC | ||||||
OS: | Linux | ||||||
Bug Depends on: | 96770 | ||||||
Bug Blocks: | 96766, 96773 | ||||||
Attachments: |
|
Description
Carlos Garcia Campos
2012-09-14 06:52:36 PDT
Created attachment 164132 [details]
Patch
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 (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 Committed r131607: <http://trac.webkit.org/changeset/131607> |