Bug 210106

Summary: [GTK][WPE] WTR: fix handling of WebsiteDataStore
Product: WebKit Reporter: Carlos Garcia Campos <cgarcia>
Component: Tools / TestsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: aperez, bugs-noreply, webkit-bug-importer
Priority: P2 Keywords: Gtk, InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
See Also: https://bugs.webkit.org/show_bug.cgi?id=202236
https://bugs.webkit.org/show_bug.cgi?id=202237
https://bugs.webkit.org/show_bug.cgi?id=202238
https://bugs.webkit.org/show_bug.cgi?id=202240
https://bugs.webkit.org/show_bug.cgi?id=202234
https://bugs.webkit.org/show_bug.cgi?id=202241
https://bugs.webkit.org/show_bug.cgi?id=204230
https://bugs.webkit.org/show_bug.cgi?id=208180
https://bugs.webkit.org/show_bug.cgi?id=212044
Attachments:
Description Flags
Patch aperez: review+

Description Carlos Garcia Campos 2020-04-07 05:33:19 PDT 
The TestController::defaultWebsiteDataStore() is created and used but never really assigned to the context. Since we are not assigning a WebsiteDataStore to the context, and we don't use the default WebsiteDataStore, when a new page is created, we end up creating the default one (with the default config, instead of the one created by WTR). We should set the TestController::defaultWebsiteDataStore() as the primary data store of the context like the GLib API does.
Comment 1 Carlos Garcia Campos 2020-04-07 05:36:01 PDT 
Created attachment 395672 [details]
Patch
Comment 2 Adrian Perez 2020-04-07 05:53:42 PDT 
Comment on attachment 395672 [details]
Patch

Please make sure that tests pass before landing, otherwise LGTM.
Comment 3 Carlos Garcia Campos 2020-04-07 23:19:30 PDT 
Committed r259705: <https://trac.webkit.org/changeset/259705>
Comment 4 Radar WebKit Bug Importer 2020-04-07 23:20:15 PDT 
<rdar://problem/61433923>