diff options
author | avi@chromium.org <avi@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-01-14 22:18:25 +0000 |
---|---|---|
committer | avi@chromium.org <avi@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-01-14 22:18:25 +0000 |
commit | 6235541680600aecf07e7e85a63dc0843f1b7084 (patch) | |
tree | 76304d95b2638e9aa3eebfd5db6b2f2198e4da5d /net | |
parent | fd4aafc80ad6b4fc7dcbbe36dfec3b7e1bcbd707 (diff) | |
download | chromium_src-6235541680600aecf07e7e85a63dc0843f1b7084.zip chromium_src-6235541680600aecf07e7e85a63dc0843f1b7084.tar.gz chromium_src-6235541680600aecf07e7e85a63dc0843f1b7084.tar.bz2 |
Revert 71485 - Remove wstring from TemplateURL and friends.
BUG=23581
TEST=no visible changes; all tests pass
Review URL: http://codereview.chromium.org/6322001
TBR=avi@chromium.org
Review URL: http://codereview.chromium.org/6291003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@71500 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'net')
-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 ae8411b..5479b82 100644 --- a/net/base/escape.cc +++ b/net/base/escape.cc @@ -1,4 +1,4 @@ -// Copyright (c) 2011 The Chromium Authors. All rights reserved. +// Copyright (c) 2010 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 !'()*-._~ -string16 EscapeQueryParamValueUTF8(const string16& text, - bool use_plus) { - return UTF8ToUTF16(Escape(UTF16ToUTF8(text), kQueryCharmap, use_plus)); +std::wstring EscapeQueryParamValueUTF8(const std::wstring& text, + bool use_plus) { + return UTF8ToWide(Escape(WideToUTF8(text), kQueryCharmap, use_plus)); } // non-printable, non-7bit, and (including space) "#%:<>?[\]^`{|} diff --git a/net/base/escape.h b/net/base/escape.h index faa7bd3..39447a5 100644 --- a/net/base/escape.h +++ b/net/base/escape.h @@ -1,4 +1,4 @@ -// Copyright (c) 2011 The Chromium Authors. All rights reserved. +// Copyright (c) 2009 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 string16s that +// A specialized version of EscapeQueryParamValue for wide strings that // assumes the codepage is UTF8. This is provided as a convenience. -string16 EscapeQueryParamValueUTF8(const string16& text, bool use_plus); +std::wstring EscapeQueryParamValueUTF8(const std::wstring& text, bool use_plus); #endif // NET_BASE_ESCAPE_H_ diff --git a/net/base/escape_unittest.cc b/net/base/escape_unittest.cc index 60d4ae3..04a040e 100644 --- a/net/base/escape_unittest.cc +++ b/net/base/escape_unittest.cc @@ -1,4 +1,4 @@ -// Copyright (c) 2011 The Chromium Authors. All rights reserved. +// Copyright (c) 2009 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,8 +66,7 @@ TEST(EscapeTest, EscapeTextForFormSubmission) { }; for (size_t i = 0; i < arraysize(escape_cases); ++i) { EscapeCase value = escape_cases[i]; - EXPECT_EQ(WideToUTF16Hack(value.output), - EscapeQueryParamValueUTF8(WideToUTF16Hack(value.input), true)); + EXPECT_EQ(value.output, EscapeQueryParamValueUTF8(value.input, true)); } const EscapeCase escape_cases_no_plus[] = { @@ -77,8 +76,7 @@ TEST(EscapeTest, EscapeTextForFormSubmission) { }; for (size_t i = 0; i < arraysize(escape_cases_no_plus); ++i) { EscapeCase value = escape_cases_no_plus[i]; - EXPECT_EQ(WideToUTF16Hack(value.output), - EscapeQueryParamValueUTF8(WideToUTF16Hack(value.input), false)); + EXPECT_EQ(value.output, EscapeQueryParamValueUTF8(value.input, false)); } // Test all the values in we're supposed to be escaping. @@ -116,10 +114,12 @@ TEST(EscapeTest, EscapeTextForFormSubmission) { string16 wide; EXPECT_TRUE(EscapeQueryParamValue(test_str, base::kCodepageUTF8, true, &wide)); - EXPECT_EQ(wide, EscapeQueryParamValueUTF8(test_str, true)); + EXPECT_EQ(UTF16ToWideHack(wide), + EscapeQueryParamValueUTF8(UTF16ToWideHack(test_str), true)); EXPECT_TRUE(EscapeQueryParamValue(test_str, base::kCodepageUTF8, false, &wide)); - EXPECT_EQ(wide, EscapeQueryParamValueUTF8(test_str, false)); + EXPECT_EQ(UTF16ToWideHack(wide), + EscapeQueryParamValueUTF8(UTF16ToWideHack(test_str), false)); } TEST(EscapeTest, EscapePath) { |