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

Unified Diff: views/controls/label_unittest.cc

Issue 8113031: Change std::wstring to string16 for views::Label and views::Link (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase Created 9 years, 2 months 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 | « views/controls/label.cc ('k') | views/controls/link.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: views/controls/label_unittest.cc
diff --git a/views/controls/label_unittest.cc b/views/controls/label_unittest.cc
index db3d491773e1ff02592093f2be684c25b5ad6ffb..68fc29650ee76be13f3b73ae3e7fb99d70d98fee 100644
--- a/views/controls/label_unittest.cc
+++ b/views/controls/label_unittest.cc
@@ -41,7 +41,7 @@ TEST(LabelTest, FontPropertyArial) {
TEST(LabelTest, TextProperty) {
Label label;
- std::wstring test_text(L"A random string.");
+ string16 test_text(ASCIIToUTF16("A random string."));
label.SetText(test_text);
EXPECT_EQ(test_text, label.GetText());
}
@@ -52,7 +52,7 @@ TEST(LabelTest, UrlProperty) {
GURL url(my_url);
label.SetURL(url);
EXPECT_EQ(my_url, label.GetURL().spec());
- EXPECT_EQ(UTF8ToWide(my_url), label.GetText());
+ EXPECT_EQ(UTF8ToUTF16(my_url), label.GetText());
}
TEST(LabelTest, ColorProperty) {
@@ -111,7 +111,7 @@ TEST(LabelTest, MultiLineProperty) {
TEST(LabelTest, TooltipProperty) {
Label label;
string16 test_text(ASCIIToUTF16("My cool string."));
- label.SetText(UTF16ToWideHack(test_text));
+ label.SetText(test_text);
string16 tooltip;
EXPECT_TRUE(label.GetTooltipText(gfx::Point(), &tooltip));
@@ -158,7 +158,7 @@ TEST(LabelTest, TooltipProperty) {
TEST(LabelTest, Accessibility) {
Label label;
string16 test_text(ASCIIToUTF16("My special text."));
- label.SetText(UTF16ToWideHack(test_text));
+ label.SetText(test_text);
ui::AccessibleViewState state;
label.GetAccessibleState(&state);
@@ -169,7 +169,7 @@ TEST(LabelTest, Accessibility) {
TEST(LabelTest, SingleLineSizing) {
Label label;
- std::wstring test_text(L"A not so random string in one line.");
+ string16 test_text(ASCIIToUTF16("A not so random string in one line."));
label.SetText(test_text);
// GetPreferredSize
@@ -196,7 +196,8 @@ TEST(LabelTest, SingleLineSizing) {
TEST(LabelTest, MultiLineSizing) {
Label label;
label.set_focusable(false);
- std::wstring test_text(L"A random string\nwith multiple lines\nand returns!");
+ string16 test_text(
+ ASCIIToUTF16("A random string\nwith multiple lines\nand returns!"));
label.SetText(test_text);
label.SetMultiLine(true);
@@ -284,7 +285,7 @@ TEST(LabelTest, DrawSingleLineString) {
// align right really means align left.
label.set_rtl_alignment_mode(Label::AUTO_DETECT_ALIGNMENT);
- std::wstring test_text(L"Here's a string with no returns.");
+ string16 test_text(ASCIIToUTF16("Here's a string with no returns."));
label.SetText(test_text);
gfx::Size required_size(label.GetPreferredSize());
gfx::Size extra(22, 8);
@@ -294,7 +295,7 @@ TEST(LabelTest, DrawSingleLineString) {
required_size.height() + extra.height());
// Do some basic verifications for all three alignments.
- std::wstring paint_text;
+ string16 paint_text;
gfx::Rect text_bounds;
int flags;
@@ -402,7 +403,7 @@ TEST(LabelTest, DrawMultiLineString) {
// align right really means align left.
label.set_rtl_alignment_mode(Label::AUTO_DETECT_ALIGNMENT);
- std::wstring test_text(L"Another string\nwith returns\n\n!");
+ string16 test_text(ASCIIToUTF16("Another string\nwith returns\n\n!"));
label.SetText(test_text);
label.SetMultiLine(true);
label.SizeToFit(0);
@@ -413,7 +414,7 @@ TEST(LabelTest, DrawMultiLineString) {
label.height() + extra.height());
// Do some basic verifications for all three alignments.
- std::wstring paint_text;
+ string16 paint_text;
gfx::Rect text_bounds;
int flags;
label.CalculateDrawStringParams(&paint_text, &text_bounds, &flags);
@@ -542,7 +543,7 @@ TEST(LabelTest, DrawSingleLineStringInRTL) {
std::string locale = l10n_util::GetApplicationLocale("");
base::i18n::SetICUDefaultLocale("he");
- std::wstring test_text(L"Here's a string with no returns.");
+ string16 test_text(ASCIIToUTF16("Here's a string with no returns."));
label.SetText(test_text);
gfx::Size required_size(label.GetPreferredSize());
gfx::Size extra(22, 8);
@@ -552,7 +553,7 @@ TEST(LabelTest, DrawSingleLineStringInRTL) {
required_size.height() + extra.height());
// Do some basic verifications for all three alignments.
- std::wstring paint_text;
+ string16 paint_text;
gfx::Rect text_bounds;
int flags;
@@ -664,7 +665,7 @@ TEST(LabelTest, DrawMultiLineStringInRTL) {
std::string locale = l10n_util::GetApplicationLocale("");
base::i18n::SetICUDefaultLocale("he");
- std::wstring test_text(L"Another string\nwith returns\n\n!");
+ string16 test_text(ASCIIToUTF16("Another string\nwith returns\n\n!"));
label.SetText(test_text);
label.SetMultiLine(true);
label.SizeToFit(0);
@@ -675,7 +676,7 @@ TEST(LabelTest, DrawMultiLineStringInRTL) {
label.height() + extra.height());
// Do some basic verifications for all three alignments.
- std::wstring paint_text;
+ string16 paint_text;
gfx::Rect text_bounds;
int flags;
label.CalculateDrawStringParams(&paint_text, &text_bounds, &flags);
« no previous file with comments | « views/controls/label.cc ('k') | views/controls/link.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698