Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(116)

Unified Diff: third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/access-control-and-redirects-async.html

Issue 2421093003: Allow redirects for requests that require preflight. (Closed)
Patch Set: Rebase on custom headers CL Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/access-control-and-redirects-async.html
diff --git a/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/access-control-and-redirects-async.html b/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/access-control-and-redirects-async.html
index 83fc3b650b9786d4ccaff354a084a4c6e90f5f7b..be7338892c01c2fe7745e38c5520bb5cf2e944c5 100644
--- a/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/access-control-and-redirects-async.html
+++ b/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/access-control-and-redirects-async.html
@@ -49,20 +49,18 @@ var tests = [
// Receives a redirect response with CORS headers. The redirect response passes the access check and the resource response
// passes the access check.
-// FIXME: this test fails because the redirect is vetoed. There are continued bugs with redirects when the original
-// request was cross-origin.
["http://localhost:8000/xmlhttprequest/resources/redirect-cors.php?url=http://localhost:8000/xmlhttprequest/resources/access-control-basic-allow-star.cgi&\
- access-control-allow-origin=http://localhost:8000",
+ access-control-allow-origin=http://127.0.0.1:8000",
withoutCredentials, noCustomHeader, succeeds],
// Receives a redirect response with a URL containing the userinfo production.
["http://localhost:8000/xmlhttprequest/resources/redirect-cors.php?url=http://username:password@localhost:8000/xmlhttprequest/resources/access-control-basic-allow-star.cgi&\
- access-control-allow-origin=http://localhost:8000",
+ access-control-allow-origin=http://127.0.0.1:8000",
withoutCredentials, noCustomHeader, fails],
// Receives a redirect response with a URL with an unsupported scheme.
["http://localhost:8000/xmlhttprequest/resources/redirect-cors.php?url=foo://bar.cgi&\
- access-control-allow-origin=http://localhost:8000",
+ access-control-allow-origin=http://127.0.0.1:8000",
withoutCredentials, noCustomHeader, fails],
// 2) Test preflighted cross origin requests that receive redirects.

Powered by Google App Engine
This is Rietveld 408576698