Title: [238972] trunk/Tools
Revision
238972
Author
[email protected]
Date
2018-12-07 15:56:07 -0800 (Fri, 07 Dec 2018)

Log Message

ProcessSwap.UseSessionCookiesAfterProcessSwapInPrivateBrowsing API test is failing
https://bugs.webkit.org/show_bug.cgi?id=192362
<rdar://problem/46460360>

Reviewed by Ryosuke Niwa.

Move the call to _setCookieAcceptPolicy *after* we've constructed a WKWebView and thus
*after* we know for sure we have a network process to communicate the policy to.
This should address the failures on iOS.

* TestWebKitAPI/Tests/WebKitCocoa/ProcessSwapOnNavigation.mm:

Modified Paths

Diff

Modified: trunk/Tools/ChangeLog (238971 => 238972)


--- trunk/Tools/ChangeLog	2018-12-07 23:38:38 UTC (rev 238971)
+++ trunk/Tools/ChangeLog	2018-12-07 23:56:07 UTC (rev 238972)
@@ -1,3 +1,17 @@
+2018-12-07  Chris Dumez  <[email protected]>
+
+        ProcessSwap.UseSessionCookiesAfterProcessSwapInPrivateBrowsing API test is failing
+        https://bugs.webkit.org/show_bug.cgi?id=192362
+        <rdar://problem/46460360>
+
+        Reviewed by Ryosuke Niwa.
+
+        Move the call to _setCookieAcceptPolicy *after* we've constructed a WKWebView and thus
+        *after* we know for sure we have a network process to communicate the policy to.
+        This should address the failures on iOS.
+
+        * TestWebKitAPI/Tests/WebKitCocoa/ProcessSwapOnNavigation.mm:
+
 2018-12-07  Jiewen Tan  <[email protected]>
 
         Add keychain access entitlement to WebKitTestRunnerApp

Modified: trunk/Tools/TestWebKitAPI/Tests/WebKitCocoa/ProcessSwapOnNavigation.mm (238971 => 238972)


--- trunk/Tools/TestWebKitAPI/Tests/WebKitCocoa/ProcessSwapOnNavigation.mm	2018-12-07 23:38:38 UTC (rev 238971)
+++ trunk/Tools/TestWebKitAPI/Tests/WebKitCocoa/ProcessSwapOnNavigation.mm	2018-12-07 23:56:07 UTC (rev 238972)
@@ -2993,7 +2993,6 @@
     auto processPoolConfiguration = adoptNS([[_WKProcessPoolConfiguration alloc] init]);
     processPoolConfiguration.get().processSwapsOnNavigation = YES;
     auto processPool = adoptNS([[WKProcessPool alloc] _initWithConfiguration:processPoolConfiguration.get()]);
-    [processPool _setCookieAcceptPolicy:NSHTTPCookieAcceptPolicyAlways];
     RetainPtr<WKWebsiteDataStore> ephemeralStore = [WKWebsiteDataStore nonPersistentDataStore];
 
     auto webViewConfiguration = adoptNS([[WKWebViewConfiguration alloc] init]);
@@ -3010,6 +3009,8 @@
     auto delegate = adoptNS([[PSONNavigationDelegate alloc] init]);
     [webView setNavigationDelegate:delegate.get()];
 
+    [processPool _setCookieAcceptPolicy:NSHTTPCookieAcceptPolicyAlways];
+
     NSURLRequest *request = [NSURLRequest requestWithURL:[[NSBundle mainBundle] URLForResource:@"SetSessionCookie" withExtension:@"html" subdirectory:@"TestWebKitAPI.resources"]];
     [webView loadRequest:request];
 
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to