TestWebKitAPI.WebKit.CookieCacheSyncAcrossProcess /Volumes/Data/slave/catalina-release/build/Tools/TestWebKitAPI/Tests/WebKitCocoa/CookiePrivateBrowsing.mm:110 Expected equality of these values: "foo=bar" (NSString *)cookie Which is: "" /Volumes/Data/slave/catalina-release/build/Tools/TestWebKitAPI/Tests/WebKitCocoa/CookiePrivateBrowsing.mm:130 Expected equality of these values: "foo=bar" cookieString Which is: "" TestWebKitAPI.WebKit.RelaxThirdPartyCookieBlocking /Volumes/Data/slave/catalina-release/build/Tools/TestWebKitAPI/Tests/WebKitCocoa/Proxy.mm:232 Value of: strstr(request.data(), "Cookie: a=b\r\n") Actual: false Expected: true /Volumes/Data/slave/catalina-release/build/Tools/TestWebKitAPI/Tests/WebKitCocoa/Proxy.mm:232 Value of: strstr(request.data(), "Cookie: a=b\r\n") Actual: false Expected: true /Volumes/Data/slave/catalina-release/build/Tools/TestWebKitAPI/Tests/WebKitCocoa/Proxy.mm:232 Value of: strstr(request.data(), "Cookie: a=b\r\n") Actual: false Expected: true
<rdar://problem/70006475>
Created attachment 410950 [details] Patch
Created attachment 410952 [details] Patch
Comment on attachment 410952 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=410952&action=review > Tools/TestWebKitAPI/Tests/WebKitCocoa/CookiePrivateBrowsing.mm:81 > + [[WKWebsiteDataStore defaultDataStore].httpCookieStore _setCookieAcceptPolicy:static_cast<NSHTTPCookieAcceptPolicy>(NSHTTPCookieAcceptPolicyExclusivelyFromMainDocumentDomain) completionHandler:^{ This test does not use the default data store at all. It uses an ephemeral session so why does setting the default data store cookie policy help? Doesn't this indicate a WebKit bug of some sort?
Comment on attachment 410952 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=410952&action=review >> Tools/TestWebKitAPI/Tests/WebKitCocoa/CookiePrivateBrowsing.mm:81 >> + [[WKWebsiteDataStore defaultDataStore].httpCookieStore _setCookieAcceptPolicy:static_cast<NSHTTPCookieAcceptPolicy>(NSHTTPCookieAcceptPolicyExclusivelyFromMainDocumentDomain) completionHandler:^{ > > This test does not use the default data store at all. It uses an ephemeral session so why does setting the default data store cookie policy help? Doesn't this indicate a WebKit bug of some sort? CFNetwork ephemeral cookie stores inherit their initial cookie accept policy from the default store.
Created attachment 410964 [details] Patch
Committed r268286: <https://trac.webkit.org/changeset/268286> All reviewed patches have been landed. Closing bug and clearing flags on attachment 410964 [details].
https://results.webkit.org/?suite=api-tests&test=TestWebKitAPI.WebKit.CookieCacheSyncAcrossProcess https://results.webkit.org/?suite=api-tests&test=TestWebKitAPI.WebKit.RelaxThirdPartyCookieBlocking Seems to have fixed it. Hooray!