Index: third_party/WebKit/LayoutTests/http/tests/fetch/script-tests/thorough/cors-preflight2.js |
diff --git a/third_party/WebKit/LayoutTests/http/tests/fetch/script-tests/thorough/cors-preflight2.js b/third_party/WebKit/LayoutTests/http/tests/fetch/script-tests/thorough/cors-preflight2.js |
index 675cd6bb217e62f44187127d775b0c04293cf221..8cb966913a36781c4f84121b2e6e6209fb10d85f 100644 |
--- a/third_party/WebKit/LayoutTests/http/tests/fetch/script-tests/thorough/cors-preflight2.js |
+++ b/third_party/WebKit/LayoutTests/http/tests/fetch/script-tests/thorough/cors-preflight2.js |
@@ -167,6 +167,15 @@ var TEST_TARGETS = []; |
'&PACRHeaders=x-serviceworker-test&PreflightTest=200', |
[fetchResolved, noContentLength, noServerHeader, hasBody, typeCors], |
[checkMethod, hasCustomHeader]], |
+ |
+ // Verify that Access-Control-Request-Headers: is not present in preflight |
+ // if its value is the empty list - https://crbug.com/633729 |
+ [OTHER_BASE_URL + 'mode=cors&credentials=same-origin&method=' + method + |
+ '&ACAOrigin=*&PACAOrigin=*&PACAMethods=' + method + |
+ '&PACRHeaders=missing&PACRMethod=' + method + '&PreflightTest=200', |
+ [fetchResolved, noContentLength, noServerHeader, hasBody, typeCors], |
+ [checkMethod]], |
tyoshino (SeeGerritForStatus)
2017/01/18 06:00:29
This test is ok to have, but what we should test i
sof
2017/01/18 06:26:49
You mean explicitly furnishing the request with a
tyoshino (SeeGerritForStatus)
2017/01/18 06:49:02
Right. Even before this fix, we skipped the Access
sof
2017/01/18 06:57:48
ok, that code path was already being exercised by
|
+ |
// Test Access-Control-Request-Headers is sorted https://crbug.com/452391 |
[OTHER_BASE_URL + 'mode=cors&credentials=same-origin&method=' + method + |
'&headers=CUSTOM2&ACAOrigin=*&PACAOrigin=*&PACAMethods=' + method + |