Commit 472d7eec authored by yhirano@chromium.org's avatar yhirano@chromium.org

Update and enable PPApiTest.URLLoaderTest3

The test was disabled[1] when XMLHttpRequest forbidden header names changed[2].
This CL updates and enables it.

1: https://src.chromium.org/viewvc/chrome?view=rev&revision=282507
2: https://src.chromium.org/viewvc/blink?view=rev&revision=178023

BUG=373120

Review URL: https://codereview.chromium.org/391213003

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283709 0039d316-1c4b-4281-b951-d872f2087c98
parent 9e05a8d3
......@@ -439,6 +439,7 @@ TEST_PPAPI_NACL(HostResolverPrivate_ResolveIPv4)
#define RUN_URLLOADER_SUBTESTS_3 \
RunTestViaHTTP( \
LIST_TEST(URLLoader_UntrustedHttpRequests) \
LIST_TEST(URLLoader_FollowURLRedirect) \
LIST_TEST(URLLoader_AuditURLRedirect) \
LIST_TEST(URLLoader_AbortCalls) \
......@@ -446,11 +447,6 @@ TEST_PPAPI_NACL(HostResolverPrivate_ResolveIPv4)
LIST_TEST(URLLoader_PrefetchBufferThreshold) \
)
// TODO(yhirano): I remove this subtest from RUN_URLLOADER_SUBTESTS_3
// in order to change the behavior in Blink.
// Restore the subtest once it is landed.
// LIST_TEST(URLLoader_UntrustedHttpRequests)
// Note: we do not support Trusted APIs in NaCl, so these will be skipped.
#define RUN_URLLOADER_TRUSTED_SUBTESTS \
RunTestViaHTTP( \
......
......@@ -593,9 +593,8 @@ std::string TestURLLoader::TestUntrustedHttpRequests() {
ASSERT_EQ(PP_ERROR_NOACCESS, OpenUntrusted("GET", "Content-Length:\n"));
ASSERT_EQ(PP_ERROR_NOACCESS, OpenUntrusted("GET", "Cookie:\n"));
ASSERT_EQ(PP_ERROR_NOACCESS, OpenUntrusted("GET", "Cookie2:\n"));
ASSERT_EQ(PP_ERROR_NOACCESS,
OpenUntrusted("GET", "Content-Transfer-Encoding:\n"));
ASSERT_EQ(PP_ERROR_NOACCESS, OpenUntrusted("GET", "Date:\n"));
ASSERT_EQ(PP_ERROR_NOACCESS, OpenUntrusted("GET", "Dnt:\n"));
ASSERT_EQ(PP_ERROR_NOACCESS, OpenUntrusted("GET", "Expect:\n"));
ASSERT_EQ(PP_ERROR_NOACCESS, OpenUntrusted("GET", "Host:\n"));
ASSERT_EQ(PP_ERROR_NOACCESS, OpenUntrusted("GET", "Keep-Alive:\n"));
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment