diff options
author | brettw@chromium.org <brettw@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-08-13 16:23:25 +0000 |
---|---|---|
committer | brettw@chromium.org <brettw@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-08-13 16:25:00 +0000 |
commit | eee3522979ae33c9ddc7ec33c769b2c3f0a48320 (patch) | |
tree | 46c85b8ad29772c08e4a423303128e976cb4b20f /net/http/http_util_unittest.cc | |
parent | 67c87495f7b272a74ab1944af19202531b766b5b (diff) | |
download | chromium_src-eee3522979ae33c9ddc7ec33c769b2c3f0a48320.zip chromium_src-eee3522979ae33c9ddc7ec33c769b2c3f0a48320.tar.gz chromium_src-eee3522979ae33c9ddc7ec33c769b2c3f0a48320.tar.bz2 |
Move StringToUpperASCII and LowerCaseEqualsASCII to the base namespace
Convert LowerCaseEqualsASCII to take StringPiece. In the current patch this is
generally a NOP but will allow me to delete the other 4 variants in a followup
(wanted to do that separately since that will require more review, since
callsites will be changed in nontrivial ways).
In some cases, LowerCaseEqualsASCII is called with a WebString, which no
longer is implicitly converted. I added base::string16(...) around such
calls to force the right conversion. It happened in these files:
window_container_type.cc
savable_resources.cc
render_view_impl.cc
blink_ax_tree_source.cc
password_form_conversion_utils.cc
translate_helper.cc
chrome_render_view_observer.cc
dom_serializer_browsertest.cc
R=jamesr@chromium.org
Review URL: https://codereview.chromium.org/448143008
Cr-Commit-Position: refs/heads/master@{#289312}
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@289312 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'net/http/http_util_unittest.cc')
-rw-r--r-- | net/http/http_util_unittest.cc | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/net/http/http_util_unittest.cc b/net/http/http_util_unittest.cc index 54acf68..8c08df2 100644 --- a/net/http/http_util_unittest.cc +++ b/net/http/http_util_unittest.cc @@ -50,7 +50,7 @@ TEST(HttpUtilTest, IsSafeHeader) { for (size_t i = 0; i < arraysize(unsafe_headers); ++i) { EXPECT_FALSE(HttpUtil::IsSafeHeader(unsafe_headers[i])) << unsafe_headers[i]; - EXPECT_FALSE(HttpUtil::IsSafeHeader(StringToUpperASCII(std::string( + EXPECT_FALSE(HttpUtil::IsSafeHeader(base::StringToUpperASCII(std::string( unsafe_headers[i])))) << unsafe_headers[i]; } static const char* safe_headers[] = { @@ -95,7 +95,7 @@ TEST(HttpUtilTest, IsSafeHeader) { }; for (size_t i = 0; i < arraysize(safe_headers); ++i) { EXPECT_TRUE(HttpUtil::IsSafeHeader(safe_headers[i])) << safe_headers[i]; - EXPECT_TRUE(HttpUtil::IsSafeHeader(StringToUpperASCII(std::string( + EXPECT_TRUE(HttpUtil::IsSafeHeader(base::StringToUpperASCII(std::string( safe_headers[i])))) << safe_headers[i]; } } |