[Webkit-unassigned] [Bug 175265] New: webkit_cookie_manager_delete_all_cookies doesn't delete the cookies before returning

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Aug 7 09:43:23 PDT 2017


            Bug ID: 175265
           Summary: webkit_cookie_manager_delete_all_cookies doesn't
                    delete the cookies before returning
           Product: WebKit
           Version: Other
          Hardware: All
                OS: All
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: WebKit Gtk
          Assignee: webkit-unassigned at lists.webkit.org
          Reporter: rishi.is at lostca.se
                CC: bugs-noreply at webkitgtk.org

For context, see:

webkit_cookie_manager_delete_all_cookies no longer deletes all the cookies before returning ever since it was re-written in terms of webkit_website_data_manager_clear in:

commit c968a9600329208662695273a6ce676f84fc657f
Author: carlosgc at webkit.org <carlosgc at webkit.org...>
Date:   Wed Feb 15 07:34:59 2017 +0000

    [GTK] Update cookie manager API to properly work with ephemeral sessions

This isn't surprising because now webkit_cookie_manager_delete_all_cookies asynchronously calls webkit_website_data_manager_clear but doesn't wait for it to complete without returning.

I don't know how WebProcessPool and the network process works, so I am not sure if webkit_cookie_manager_delete_all_cookies was working merely by accident.

I tried to work around this by wrapping webkit_website_data_manager_clear inside a new thread-default GMainContext (using g_main_context_push/pop_thread_default), but that doesn't work either. The GAsyncReadyCallback doesn't get invoked. Doesn't WTF::RunLoop, etc. not work with thread-default GMainContexts?

You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20170807/8d451533/attachment-0001.html>

More information about the webkit-unassigned mailing list