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

Unified Diff: app/l10n_util_unittest.cc

Issue 5742006: wstrings: make l10n_util::TruncateString use string16 (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 10 years 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 | « app/l10n_util.cc ('k') | chrome/browser/extensions/extension_menu_manager.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: app/l10n_util_unittest.cc
diff --git a/app/l10n_util_unittest.cc b/app/l10n_util_unittest.cc
index 8d488134e0f76d1a245bb740ce832775784b3617..fa4cc2401f74682c926527dbf1709f0816f17332 100644
--- a/app/l10n_util_unittest.cc
+++ b/app/l10n_util_unittest.cc
@@ -61,30 +61,33 @@ TEST_F(L10nUtilTest, DISABLED_GetString) {
#endif // defined(OS_WIN)
TEST_F(L10nUtilTest, TruncateString) {
- std::wstring string(L"foooooey bxxxar baz");
+ string16 string = ASCIIToUTF16("foooooey bxxxar baz");
// Make sure it doesn't modify the string if length > string length.
EXPECT_EQ(string, l10n_util::TruncateString(string, 100));
// Test no characters.
- EXPECT_EQ(L"", l10n_util::TruncateString(string, 0));
+ EXPECT_EQ(L"", UTF16ToWide(l10n_util::TruncateString(string, 0)));
// Test 1 character.
- EXPECT_EQ(L"\x2026", l10n_util::TruncateString(string, 1));
+ EXPECT_EQ(L"\x2026", UTF16ToWide(l10n_util::TruncateString(string, 1)));
// Test adds ... at right spot when there is enough room to break at a
// word boundary.
- EXPECT_EQ(L"foooooey\x2026", l10n_util::TruncateString(string, 14));
+ EXPECT_EQ(L"foooooey\x2026",
+ UTF16ToWide(l10n_util::TruncateString(string, 14)));
// Test adds ... at right spot when there is not enough space in first word.
- EXPECT_EQ(L"f\x2026", l10n_util::TruncateString(string, 2));
+ EXPECT_EQ(L"f\x2026", UTF16ToWide(l10n_util::TruncateString(string, 2)));
// Test adds ... at right spot when there is not enough room to break at a
// word boundary.
- EXPECT_EQ(L"foooooey\x2026", l10n_util::TruncateString(string, 11));
+ EXPECT_EQ(L"foooooey\x2026",
+ UTF16ToWide(l10n_util::TruncateString(string, 11)));
// Test completely truncates string if break is on initial whitespace.
- EXPECT_EQ(L"\x2026", l10n_util::TruncateString(L" ", 2));
+ EXPECT_EQ(L"\x2026",
+ UTF16ToWide(l10n_util::TruncateString(ASCIIToUTF16(" "), 2)));
}
void SetICUDefaultLocale(const std::string& locale_string) {
« no previous file with comments | « app/l10n_util.cc ('k') | chrome/browser/extensions/extension_menu_manager.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698