Bug 207449

Summary: [WK2][Soup] Implement NetworkStorageSession::getAllCookies
Product: WebKit Reporter: Pavel Feldman <pfeldman>
Component: WebKit2Assignee: Pavel Feldman <pfeldman>
Status: RESOLVED FIXED    
Severity: Normal CC: aperez, berto, cgarcia, commit-queue, ews-watchlist, gustavo, webkit-bug-importer, yurys
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Description Pavel Feldman 2020-02-09 20:29:51 PST
Getting is implemented for GTK/WPE ports.
Comment 1 Pavel Feldman 2020-02-09 20:31:30 PST
Created attachment 390228 [details]
Patch
Comment 2 Carlos Garcia Campos 2020-02-10 00:15:18 PST
Comment on attachment 390228 [details]
Patch

This looks good, but it seems to be deadcode for ports using libsoup. Why do you need this?  This is only used by cocoa API, I think.
Comment 3 Pavel Feldman 2020-02-11 13:10:27 PST
Thanks for looking into it.

This achieves feature parity with the Cocoa API in terms of raw cookie management. We'd like to be able to manage cookies on the datastore / processpool level over the remote debugging protocol. The harness / web inspector client is not there (yet) and it'll take time, so this does not make any tests green yet. But it looks innocent in terms of adding simple yet functional implementations for those.
Comment 4 Carlos Garcia Campos 2020-02-12 01:13:11 PST
Comment on attachment 390228 [details]
Patch

Ok, then. Thanks!
Comment 5 WebKit Commit Bot 2020-02-12 18:13:16 PST
Comment on attachment 390228 [details]
Patch

Clearing flags on attachment: 390228

Committed r256496: <https://trac.webkit.org/changeset/256496>
Comment 6 WebKit Commit Bot 2020-02-12 18:13:18 PST
All reviewed patches have been landed.  Closing bug.
Comment 7 Radar WebKit Bug Importer 2020-02-12 18:14:17 PST
<rdar://problem/59409694>