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

Unified Diff: net/http/http_util_unittest.cc

Issue 10365012: Update net::HttpUtil::IsSafeHeader() with two new headers (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: merge Created 8 years, 7 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
« no previous file with comments | « net/http/http_util.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/http/http_util_unittest.cc
diff --git a/net/http/http_util_unittest.cc b/net/http/http_util_unittest.cc
index 8293c8031517fe4a3ecb0dd55448f4065cfb0156..fc38137e54f387f7ebf3da4175a68eb764681475 100644
--- a/net/http/http_util_unittest.cc
+++ b/net/http/http_util_unittest.cc
@@ -27,6 +27,8 @@ TEST(HttpUtilTest, IsSafeHeader) {
"pRoXy-FoO",
"accept-charset",
"accept-encoding",
+ "access-control-request-headers",
+ "access-control-request-method",
"connection",
"content-length",
"cookie",
@@ -61,6 +63,11 @@ TEST(HttpUtilTest, IsSafeHeader) {
"accept_charset",
"accept-encodinga",
"accept_encoding",
+ "access-control-request-headersa",
+ "access-control-request-header",
+ "access_control_request_header",
+ "access-control-request-methoda",
+ "access_control_request_method",
"connectiona",
"content-lengtha",
"content_length",
« no previous file with comments | « net/http/http_util.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698