diff options
author | avi@chromium.org <avi@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-01-14 21:15:35 +0000 |
---|---|---|
committer | avi@chromium.org <avi@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-01-14 21:15:35 +0000 |
commit | a61c5c9bb6524d77c30b6a18f542c0e4616ee7bc (patch) | |
tree | c1afa473cffdaea857f181ba0788e60212649f30 /net/base | |
parent | 0eb90137036cf765397a69f04f5c6695679fd8c0 (diff) | |
download | chromium_src-a61c5c9bb6524d77c30b6a18f542c0e4616ee7bc.zip chromium_src-a61c5c9bb6524d77c30b6a18f542c0e4616ee7bc.tar.gz chromium_src-a61c5c9bb6524d77c30b6a18f542c0e4616ee7bc.tar.bz2 |
Remove wstring from TemplateURL and friends.
BUG=23581
TEST=no visible changes; all tests pass
Review URL: http://codereview.chromium.org/6322001
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@71485 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'net/base')
-rw-r--r-- | net/base/escape.cc | 8 | ||||
-rw-r--r-- | net/base/escape.h | 6 | ||||
-rw-r--r-- | net/base/escape_unittest.cc | 14 |
3 files changed, 14 insertions, 14 deletions
diff --git a/net/base/escape.cc b/net/base/escape.cc index 5479b82..ae8411b 100644 --- a/net/base/escape.cc +++ b/net/base/escape.cc @@ -1,4 +1,4 @@ -// Copyright (c) 2010 The Chromium Authors. All rights reserved. +// Copyright (c) 2011 The Chromium Authors. All rights reserved. // Use of this source code is governed by a BSD-style license that can be // found in the LICENSE file. @@ -192,9 +192,9 @@ std::string EscapeQueryParamValue(const std::string& text, bool use_plus) { // Convert the string to a sequence of bytes and then % escape anything // except alphanumerics and !'()*-._~ -std::wstring EscapeQueryParamValueUTF8(const std::wstring& text, - bool use_plus) { - return UTF8ToWide(Escape(WideToUTF8(text), kQueryCharmap, use_plus)); +string16 EscapeQueryParamValueUTF8(const string16& text, + bool use_plus) { + return UTF8ToUTF16(Escape(UTF16ToUTF8(text), kQueryCharmap, use_plus)); } // non-printable, non-7bit, and (including space) "#%:<>?[\]^`{|} diff --git a/net/base/escape.h b/net/base/escape.h index 39447a5..faa7bd3 100644 --- a/net/base/escape.h +++ b/net/base/escape.h @@ -1,4 +1,4 @@ -// Copyright (c) 2009 The Chromium Authors. All rights reserved. +// Copyright (c) 2011 The Chromium Authors. All rights reserved. // Use of this source code is governed by a BSD-style license that can be // found in the LICENSE file. @@ -125,8 +125,8 @@ std::string EscapeQueryParamValue(const std::string& text, bool use_plus); bool EscapeQueryParamValue(const string16& text, const char* codepage, bool use_plus, string16* escaped); -// A specialized version of EscapeQueryParamValue for wide strings that +// A specialized version of EscapeQueryParamValue for string16s that // assumes the codepage is UTF8. This is provided as a convenience. -std::wstring EscapeQueryParamValueUTF8(const std::wstring& text, bool use_plus); +string16 EscapeQueryParamValueUTF8(const string16& text, bool use_plus); #endif // NET_BASE_ESCAPE_H_ diff --git a/net/base/escape_unittest.cc b/net/base/escape_unittest.cc index 04a040e..60d4ae3 100644 --- a/net/base/escape_unittest.cc +++ b/net/base/escape_unittest.cc @@ -1,4 +1,4 @@ -// Copyright (c) 2009 The Chromium Authors. All rights reserved. +// Copyright (c) 2011 The Chromium Authors. All rights reserved. // Use of this source code is governed by a BSD-style license that can be // found in the LICENSE file. @@ -66,7 +66,8 @@ TEST(EscapeTest, EscapeTextForFormSubmission) { }; for (size_t i = 0; i < arraysize(escape_cases); ++i) { EscapeCase value = escape_cases[i]; - EXPECT_EQ(value.output, EscapeQueryParamValueUTF8(value.input, true)); + EXPECT_EQ(WideToUTF16Hack(value.output), + EscapeQueryParamValueUTF8(WideToUTF16Hack(value.input), true)); } const EscapeCase escape_cases_no_plus[] = { @@ -76,7 +77,8 @@ TEST(EscapeTest, EscapeTextForFormSubmission) { }; for (size_t i = 0; i < arraysize(escape_cases_no_plus); ++i) { EscapeCase value = escape_cases_no_plus[i]; - EXPECT_EQ(value.output, EscapeQueryParamValueUTF8(value.input, false)); + EXPECT_EQ(WideToUTF16Hack(value.output), + EscapeQueryParamValueUTF8(WideToUTF16Hack(value.input), false)); } // Test all the values in we're supposed to be escaping. @@ -114,12 +116,10 @@ TEST(EscapeTest, EscapeTextForFormSubmission) { string16 wide; EXPECT_TRUE(EscapeQueryParamValue(test_str, base::kCodepageUTF8, true, &wide)); - EXPECT_EQ(UTF16ToWideHack(wide), - EscapeQueryParamValueUTF8(UTF16ToWideHack(test_str), true)); + EXPECT_EQ(wide, EscapeQueryParamValueUTF8(test_str, true)); EXPECT_TRUE(EscapeQueryParamValue(test_str, base::kCodepageUTF8, false, &wide)); - EXPECT_EQ(UTF16ToWideHack(wide), - EscapeQueryParamValueUTF8(UTF16ToWideHack(test_str), false)); + EXPECT_EQ(wide, EscapeQueryParamValueUTF8(test_str, false)); } TEST(EscapeTest, EscapePath) { |