Bug 82441 - [GTK] Initial cookies API for WebKit2 GTK+ API
Summary: [GTK] Initial cookies API for WebKit2 GTK+ API
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit2 (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC Linux
: P2 Normal
Assignee: Nobody
URL:
Keywords: Gtk
Depends on:
Blocks: 82598
  Show dependency treegraph
 
Reported: 2012-03-28 02:44 PDT by Carlos Garcia Campos
Modified: 2012-04-16 08:32 PDT (History)
4 users (show)

See Also:


Attachments
Patch (34.78 KB, patch)
2012-03-28 02:51 PDT, Carlos Garcia Campos
gustavo: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Carlos Garcia Campos 2012-03-28 02:44:26 PDT
Add an initial WebKitCookieManager object to be able to set the acceptance policy, get domains and delete cookies.
Comment 1 Carlos Garcia Campos 2012-03-28 02:51:27 PDT
Created attachment 134247 [details]
Patch
Comment 2 WebKit Review Bot 2012-03-28 02:54:18 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
Comment 3 Gustavo Noronha (kov) 2012-04-11 05:15:13 PDT
Comment on attachment 134247 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=134247&action=review

> Source/WebKit2/UIProcess/API/gtk/WebKitCookieManager.cpp:210
> +    WKCookieManagerDeleteCookiesForHostname(manager->priv->wkCookieManager.get(), wkDomain.get());

Are the delete calls synchronous, or do we just not care about the result in this case? Also what about deleting a single cookie? That's something Ephy does today, something we want to keep supporting?
Comment 4 Carlos Garcia Campos 2012-04-11 06:24:09 PDT
(In reply to comment #3)
> (From update of attachment 134247 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=134247&action=review

Thanks for reviewing!

> > Source/WebKit2/UIProcess/API/gtk/WebKitCookieManager.cpp:210
> > +    WKCookieManagerDeleteCookiesForHostname(manager->priv->wkCookieManager.get(), wkDomain.get());
> 
> Are the delete calls synchronous, or do we just not care about the result in this case?

It's async, but the C API doesn't provide any result nor callback to be emitted when the operation finishes.

> Also what about deleting a single cookie?

Not supported by the C API.

> That's something Ephy does today, something we want to keep supporting?

As I commented in the mailing list when I proposed this API, it's not a common operation, people usually remove the cookies for a domain rather than a single cookie. However, since bugs #82081 and #82082 landed already, it's now possible to see the details of all cookies using the inspector, as well as deleting a single cookie. So, we could leave that as an advanced feature available through the inspector. Or we can add the API if you think it's really important. In any case, I think it would be batter to discuss about it in the mailing list, see this thread:

https://lists.webkit.org/pipermail/webkit-gtk/2012-March/001007.html
Comment 5 Carlos Garcia Campos 2012-04-11 09:15:05 PDT
Committed r113870: <http://trac.webkit.org/changeset/113870>
Comment 6 Gustavo Noronha (kov) 2012-04-16 08:32:39 PDT
I think using the inspector for these advanced features is a great idea, yeah =)