Bug 204230
| Summary: | [GTK] http/tests/xmlhttprequest/cross-origin-cookie-storage.html [since r249619] [Remove calls to WKContextGetWebsiteDataStore missed in r249614] | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Pablo Saavedra <psaavedra> |
| Component: | WebKitGTK | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | achristensen, bugs-noreply, cgarcia |
| Priority: | P2 | ||
| Version: | WebKit Nightly Build | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| See Also: | https://bugs.webkit.org/show_bug.cgi?id=210106 | ||
Pablo Saavedra
Introduced by:
commit cdfb1cd5224630a87d2d14fd389bafc45747e1f9
Author: achristensen@apple.com <achristensen@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Date: Sat Sep 7 22:54:22 2019 +0000
Remove calls to WKContextGetWebsiteDataStore missed in r249614
https://bugs.webkit.org/show_bug.cgi?id=200050
Source/WebKit:
It turns out I didn't need to introduce another way of clearing all cookies between tests.
I just needed to make the call to WKHTTPCookieStoreDeleteAllCookies use the right WebsiteDataStore.
* NetworkProcess/NetworkProcess.cpp:
(WebKit::NetworkProcess::clearCachedCredentials):
* NetworkProcess/NetworkSession.h:
(WebKit::NetworkSession::clearCredentials):
(WebKit::NetworkSession::clearCookies): Deleted.
* NetworkProcess/cocoa/NetworkSessionCocoa.h:
* NetworkProcess/cocoa/NetworkSessionCocoa.mm:
(WebKit::NetworkSessionCocoa::clearCookies): Deleted.
Tools:
* WebKitTestRunner/TestController.cpp:
(WTR::TestController::resetPreferencesToConsistentValues):
(WTR::TestController::didReceiveSynchronousMessageFromInjectedBundle):
* WebKitTestRunner/TestInvocation.cpp:
(WTR::TestInvocation::invoke):
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@249619 268f45cc-cd09-0410-ab3c-d52691b4dbfc
Diff:
--- /home/igalia/psaavedra/repos/webkit-gtk/WebKitBuild/Release/layout-test-results/http/tests/xmlhttprequest/cross-origin-cookie-storage-expected.txt
+++ /home/igalia/psaavedra/repos/webkit-gtk/WebKitBuild/Release/layout-test-results/http/tests/xmlhttprequest/cross-origin-cookie-storage-actual.txt
@@ -7,6 +7,6 @@
PASS: No cookies set.
Cross-origin XMLHttpRequest (sync), testing that cookies are set when sending credentials.
PASS: Finished sync xhr.
-PASS: Cookie set.
+FAIL: no cookie set.
DONE
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Carlos Garcia Campos
This is passing since r259705.