Bug 172345

Summary: REGRESSION(r216977): [GTK] Ephemeral sessions broken after r216977
Product: WebKit Reporter: Carlos Garcia Campos <cgarcia>
Component: WebKitGTKAssignee: Nobody <webkit-unassigned>
Severity: Normal CC: achristensen, beidson, bugs-noreply, mcatanzaro
Priority: P2 Keywords: Gtk, Regression
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Description Flags
Patch mcatanzaro: review+

Description Carlos Garcia Campos 2017-05-18 23:48:48 PDT
Since r216977, WebProcessPool sends the page session ID to the web process, but the WebsiteDataStoreParameters session ID to the network process. WebsiteDataStoreParameters is only implemented for Cocoa, so we are sending 0 session ID to the network process, while using the right session ID everywhere else.

$ MiniBrowser -p
Attempted to create a NetworkLoad with a session (id=9223372036854775810) that does not exist.

This has also broken two unit tests.
Comment 1 Carlos Garcia Campos 2017-05-18 23:52:27 PDT
Created attachment 310625 [details]
Comment 2 Michael Catanzaro 2017-05-19 09:04:20 PDT
Comment on attachment 310625 [details]

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

Glad the API tests caught this.

Technically you need an owner to approve the cross-platform assertion, but since it's one line I assume you can get approval on IRC on something.

> Source/WebKit2/UIProcess/WebsiteData/WebsiteDataStore.cpp:1302
> +    // FIXME: Implement cookies.

What needs to happen here?
Comment 3 Carlos Garcia Campos 2017-05-19 09:11:44 PDT
Committed r217128: <http://trac.webkit.org/changeset/217128>