From 6235541680600aecf07e7e85a63dc0843f1b7084 Mon Sep 17 00:00:00 2001 From: "avi@chromium.org" Date: Fri, 14 Jan 2011 22:18:25 +0000 Subject: 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 --- net/base/escape.cc | 8 ++++---- net/base/escape.h | 6 +++--- net/base/escape_unittest.cc | 14 +++++++------- 3 files changed, 14 insertions(+), 14 deletions(-) (limited to 'net/base') 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) { -- cgit v1.1