Bug 171987

Summary: Interacting with WKHTTPCookieStores before creating WKWebViews and WKProcessPools should affect cookies used
Product: WebKit Reporter: Alex Christensen <achristensen>
Component: New BugsAssignee: Alex Christensen <achristensen>
Status: RESOLVED FIXED    
Severity: Normal CC: beidson, ddkilzer, ggaren, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 172558    
Attachments:
Description Flags
Patch
none
Patch
none
Patch beidson: review+

Alex Christensen
Reported 2017-05-11 14:12:52 PDT
Interacting with WKHTTPCookieStores before creating WKWebViews and WKProcessPools should effect cookies used
Attachments
Patch (2.77 KB, patch)
2017-05-11 14:13 PDT, Alex Christensen 		no flags
DetailsFormatted DiffDiff
Patch (31.91 KB, patch)
2017-05-16 17:20 PDT, Alex Christensen 		no flags
DetailsFormatted DiffDiff
Patch (33.21 KB, patch)
2017-05-16 22:14 PDT, Alex Christensen 		beidson: review+
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Alex Christensen
Comment 1 2017-05-11 14:13:05 PDT
Created attachment 309781 [details] Patch
Radar WebKit Bug Importer
Comment 2 2017-05-11 14:14:22 PDT
<rdar://problem/32141505>
Geoffrey Garen
Comment 3 2017-05-11 14:14:42 PDT
Comment on attachment 309781 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=309781&action=review Did you mean to include a patch too, or is that in another bug? > Tools/ChangeLog:3 > + Interacting with WKHTTPCookieStores before creating WKWebViews and WKProcessPools should effect cookies used effect => affect
Alex Christensen
Comment 4 2017-05-11 14:16:13 PDT
I haven't written the patch yet. This is just the failing test. I just needed to file the radar before I've finished the patch.
Alex Christensen
Comment 5 2017-05-16 17:20:34 PDT
Created attachment 310325 [details] Patch
Alex Christensen
Comment 6 2017-05-16 22:14:51 PDT
Created attachment 310349 [details] Patch
Alex Christensen
Comment 7 2017-05-17 09:52:12 PDT
http://trac.webkit.org/r216977
Alex Christensen
Comment 8 2017-05-17 16:11:55 PDT
http://trac.webkit.org/r217010
David Kilzer (:ddkilzer)
Comment 9 2017-05-24 15:59:47 PDT
This caused leaks in TestWebKitAPI: Bug 172558: REGRESSION (r216977): 4 leaks introduced in new WebKit2_WKHTTPCookieStoreWithoutProcessPool_Test <https://bugs.webkit.org/show_bug.cgi?id=172558>
Note You need to log in before you can comment on or make changes to this bug.