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

Unified Diff: third_party/WebKit/LayoutTests/http/tests/fetch/script-tests/thorough/redirect.js

Issue 2471533005: Preserve custom headers when following cross-origin redirects. (Closed)
Patch Set: 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/fetch/script-tests/thorough/redirect.js
diff --git a/third_party/WebKit/LayoutTests/http/tests/fetch/script-tests/thorough/redirect.js b/third_party/WebKit/LayoutTests/http/tests/fetch/script-tests/thorough/redirect.js
index 7f33e95b817d440bddb5505df85ce0ef1982c2ce..79c565d8985baa7c7a014e717d103e8424920bf7 100644
--- a/third_party/WebKit/LayoutTests/http/tests/fetch/script-tests/thorough/redirect.js
+++ b/third_party/WebKit/LayoutTests/http/tests/fetch/script-tests/thorough/redirect.js
@@ -133,6 +133,14 @@ var TEST_TARGETS = [
// [fetchResolved, noContentLength, noServerHeader, hasBody, typeCors],
// [methodIsPOST]],
+ // Custom header
+ [REDIRECT_URL +
+ encodeURIComponent(OTHER_BASE_URL +
+ 'ACAOrigin=*&ACAHeaders=x-serviceworker-test') +
+ '&mode=cors&credentials=same-origin&method=GET&headers=CUSTOM',
+ [fetchResolved, noContentLength, noServerHeader, hasBody, typeCors],
+ [methodIsGET, hasCustomHeader, authCheckNone]],
+
yhirano 2016/11/07 07:48:30 Please add a test to verify that ACAOrigin and ACA
Jack Bates 2016/11/12 21:13:30 Done.
// Server header
[REDIRECT_URL +
encodeURIComponent(

Powered by Google App Engine
This is Rietveld 408576698