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

Unified Diff: third_party/WebKit/LayoutTests/external/wpt/fetch/api/redirect/redirect-location.js

Issue 2785123002: Make no-location redirect response to be "opaque redirect" when redirect mode is manual. (Closed)
Patch Set: add link to crbug.com/707185 Created 3 years, 8 months 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/external/wpt/fetch/api/redirect/redirect-location.js
diff --git a/third_party/WebKit/LayoutTests/external/wpt/fetch/api/redirect/redirect-location.js b/third_party/WebKit/LayoutTests/external/wpt/fetch/api/redirect/redirect-location.js
index 0297250b535545cf647238080f0c3325052d4676..cb038a6812ce3e19580e82099675923f10031a97 100644
--- a/third_party/WebKit/LayoutTests/external/wpt/fetch/api/redirect/redirect-location.js
+++ b/third_party/WebKit/LayoutTests/external/wpt/fetch/api/redirect/redirect-location.js
@@ -14,7 +14,7 @@ function redirectLocation(desc, redirectUrl, redirectLocation, redirectStatus, r
promise_test(function(test) {
if (redirectMode === "error" || !shouldPass)
return promise_rejects(test, new TypeError(), fetch(url + urlParameters, requestInit));
- if (redirectLocation && redirectMode === "manual")
+ if (redirectMode === "manual")
return fetch(url + urlParameters, requestInit).then(function(resp) {
assert_equals(resp.status, 0, "Response's status is 0");
assert_equals(resp.type, "opaqueredirect", "Response's type is opaqueredirect");
@@ -22,7 +22,7 @@ function redirectLocation(desc, redirectUrl, redirectLocation, redirectStatus, r
assert_true(resp.headers.entries().next().done, "Headers should be empty");
});
- if (redirectMode === "manual" || redirectMode === "follow")
+ if (redirectMode === "follow")
return fetch(url + urlParameters, requestInit).then(function(resp) {
assert_equals(resp.status, redirectStatus, "Response's status is " + redirectStatus);
});

Powered by Google App Engine
This is Rietveld 408576698