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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: ui/views/controls/label_unittest.cc

Issue 259073004: Revert of retry r266042: (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 8 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 | « ui/views/controls/label.cc ('k') | ui/views/controls/scroll_view.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/views/controls/label_unittest.cc
diff --git a/ui/views/controls/label_unittest.cc b/ui/views/controls/label_unittest.cc
index 2ce9b0741e067865c8b4b95e6a8b253ad21f5901..aac99840ddc1f4f74353c771e953fe50fdd716ab 100644
--- a/ui/views/controls/label_unittest.cc
+++ b/ui/views/controls/label_unittest.cc
@@ -528,10 +528,11 @@
int expected_flags = gfx::Canvas::MULTI_LINE |
gfx::Canvas::TEXT_ALIGN_CENTER |
gfx::Canvas::FORCE_LTR_DIRECTIONALITY;
-#if !defined(OS_WIN)
- expected_flags |= gfx::Canvas::NO_ELLIPSIS;
-#endif
- EXPECT_EQ(expected_flags, expected_flags & flags);
+#if defined(OS_WIN)
+ EXPECT_EQ(expected_flags, flags);
+#else
+ EXPECT_EQ(expected_flags | gfx::Canvas::NO_ELLIPSIS, flags);
+#endif
gfx::Rect center_bounds(text_bounds);
label.SetHorizontalAlignment(gfx::ALIGN_LEFT);
@@ -546,10 +547,11 @@
expected_flags = gfx::Canvas::MULTI_LINE |
gfx::Canvas::TEXT_ALIGN_LEFT |
gfx::Canvas::FORCE_LTR_DIRECTIONALITY;
-#if !defined(OS_WIN)
- expected_flags |= gfx::Canvas::NO_ELLIPSIS;
-#endif
- EXPECT_EQ(expected_flags, expected_flags & flags);
+#if defined(OS_WIN)
+ EXPECT_EQ(expected_flags, flags);
+#else
+ EXPECT_EQ(expected_flags | gfx::Canvas::NO_ELLIPSIS, flags);
+#endif
label.SetHorizontalAlignment(gfx::ALIGN_RIGHT);
paint_text.clear();
@@ -563,10 +565,11 @@
expected_flags = gfx::Canvas::MULTI_LINE |
gfx::Canvas::TEXT_ALIGN_RIGHT |
gfx::Canvas::FORCE_LTR_DIRECTIONALITY;
-#if !defined(OS_WIN)
- expected_flags |= gfx::Canvas::NO_ELLIPSIS;
-#endif
- EXPECT_EQ(expected_flags, expected_flags & flags);
+#if defined(OS_WIN)
+ EXPECT_EQ(expected_flags, flags);
+#else
+ EXPECT_EQ(expected_flags | gfx::Canvas::NO_ELLIPSIS, flags);
+#endif
// Test multiline drawing with a border.
gfx::Insets border(19, 92, 23, 2);
@@ -590,10 +593,11 @@
expected_flags = gfx::Canvas::MULTI_LINE |
gfx::Canvas::TEXT_ALIGN_CENTER |
gfx::Canvas::FORCE_LTR_DIRECTIONALITY;
-#if !defined(OS_WIN)
- expected_flags |= gfx::Canvas::NO_ELLIPSIS;
-#endif
- EXPECT_EQ(expected_flags, expected_flags & flags);
+#if defined(OS_WIN)
+ EXPECT_EQ(expected_flags, flags);
+#else
+ EXPECT_EQ(expected_flags | gfx::Canvas::NO_ELLIPSIS, flags);
+#endif
label.SetHorizontalAlignment(gfx::ALIGN_LEFT);
paint_text.clear();
@@ -607,10 +611,11 @@
expected_flags = gfx::Canvas::MULTI_LINE |
gfx::Canvas::TEXT_ALIGN_LEFT |
gfx::Canvas::FORCE_LTR_DIRECTIONALITY;
-#if !defined(OS_WIN)
- expected_flags |= gfx::Canvas::NO_ELLIPSIS;
-#endif
- EXPECT_EQ(expected_flags, expected_flags & flags);
+#if defined(OS_WIN)
+ EXPECT_EQ(expected_flags, flags);
+#else
+ EXPECT_EQ(expected_flags | gfx::Canvas::NO_ELLIPSIS, flags);
+#endif
label.SetHorizontalAlignment(gfx::ALIGN_RIGHT);
paint_text.clear();
@@ -624,10 +629,11 @@
expected_flags = gfx::Canvas::MULTI_LINE |
gfx::Canvas::TEXT_ALIGN_RIGHT |
gfx::Canvas::FORCE_LTR_DIRECTIONALITY;
-#if !defined(OS_WIN)
- expected_flags |= gfx::Canvas::NO_ELLIPSIS;
-#endif
- EXPECT_EQ(expected_flags, expected_flags & flags);
+#if defined(OS_WIN)
+ EXPECT_EQ(expected_flags, flags);
+#else
+ EXPECT_EQ(expected_flags | gfx::Canvas::NO_ELLIPSIS, flags);
+#endif
}
TEST(LabelTest, DrawSingleLineStringInRTL) {
@@ -795,10 +801,14 @@
EXPECT_EQ(extra.height() / 2, text_bounds.y());
EXPECT_GT(text_bounds.width(), kMinTextDimension);
EXPECT_GT(text_bounds.height(), kMinTextDimension);
- EXPECT_TRUE(gfx::Canvas::MULTI_LINE & flags);
- EXPECT_TRUE(gfx::Canvas::TEXT_ALIGN_CENTER & flags);
-#if !defined(OS_WIN)
- EXPECT_TRUE(gfx::Canvas::NO_ELLIPSIS & flags);
+#if defined(OS_WIN)
+ EXPECT_EQ(gfx::Canvas::MULTI_LINE | gfx::Canvas::TEXT_ALIGN_CENTER, flags);
+#else
+ EXPECT_EQ(
+ gfx::Canvas::MULTI_LINE |
+ gfx::Canvas::TEXT_ALIGN_CENTER |
+ gfx::Canvas::NO_ELLIPSIS,
+ flags);
#endif
gfx::Rect center_bounds(text_bounds);
@@ -811,10 +821,14 @@
EXPECT_EQ(extra.height() / 2, text_bounds.y());
EXPECT_GT(text_bounds.width(), kMinTextDimension);
EXPECT_GT(text_bounds.height(), kMinTextDimension);
- EXPECT_TRUE(gfx::Canvas::MULTI_LINE & flags);
- EXPECT_TRUE(gfx::Canvas::TEXT_ALIGN_RIGHT & flags);
-#if !defined(OS_WIN)
- EXPECT_TRUE(gfx::Canvas::NO_ELLIPSIS & flags);
+#if defined(OS_WIN)
+ EXPECT_EQ(gfx::Canvas::MULTI_LINE | gfx::Canvas::TEXT_ALIGN_RIGHT, flags);
+#else
+ EXPECT_EQ(
+ gfx::Canvas::MULTI_LINE |
+ gfx::Canvas::TEXT_ALIGN_RIGHT |
+ gfx::Canvas::NO_ELLIPSIS,
+ flags);
#endif
label.SetHorizontalAlignment(gfx::ALIGN_RIGHT);
@@ -826,10 +840,14 @@
EXPECT_EQ(extra.height() / 2, text_bounds.y());
EXPECT_GT(text_bounds.width(), kMinTextDimension);
EXPECT_GT(text_bounds.height(), kMinTextDimension);
- EXPECT_TRUE(gfx::Canvas::MULTI_LINE & flags);
- EXPECT_TRUE(gfx::Canvas::TEXT_ALIGN_LEFT & flags);
-#if !defined(OS_WIN)
- EXPECT_TRUE(gfx::Canvas::NO_ELLIPSIS & flags);
+#if defined(OS_WIN)
+ EXPECT_EQ(gfx::Canvas::MULTI_LINE | gfx::Canvas::TEXT_ALIGN_LEFT, flags);
+#else
+ EXPECT_EQ(
+ gfx::Canvas::MULTI_LINE |
+ gfx::Canvas::TEXT_ALIGN_LEFT |
+ gfx::Canvas::NO_ELLIPSIS,
+ flags);
#endif
// Test multiline drawing with a border.
@@ -851,10 +869,14 @@
EXPECT_EQ(border.top() + extra.height() / 2, text_bounds.y());
EXPECT_EQ(center_bounds.width(), text_bounds.width());
EXPECT_EQ(center_bounds.height(), text_bounds.height());
- EXPECT_TRUE(gfx::Canvas::MULTI_LINE & flags);
- EXPECT_TRUE(gfx::Canvas::TEXT_ALIGN_CENTER & flags);
-#if !defined(OS_WIN)
- EXPECT_TRUE(gfx::Canvas::NO_ELLIPSIS & flags);
+#if defined(OS_WIN)
+ EXPECT_EQ(gfx::Canvas::MULTI_LINE | gfx::Canvas::TEXT_ALIGN_CENTER, flags);
+#else
+ EXPECT_EQ(
+ gfx::Canvas::MULTI_LINE |
+ gfx::Canvas::TEXT_ALIGN_CENTER |
+ gfx::Canvas::NO_ELLIPSIS,
+ flags);
#endif
label.SetHorizontalAlignment(gfx::ALIGN_LEFT);
@@ -866,10 +888,14 @@
EXPECT_EQ(border.top() + extra.height() / 2, text_bounds.y());
EXPECT_EQ(center_bounds.width(), text_bounds.width());
EXPECT_EQ(center_bounds.height(), text_bounds.height());
- EXPECT_TRUE(gfx::Canvas::MULTI_LINE & flags);
- EXPECT_TRUE(gfx::Canvas::TEXT_ALIGN_RIGHT & flags);
-#if !defined(OS_WIN)
- EXPECT_TRUE(gfx::Canvas::NO_ELLIPSIS & flags);
+#if defined(OS_WIN)
+ EXPECT_EQ(gfx::Canvas::MULTI_LINE | gfx::Canvas::TEXT_ALIGN_RIGHT, flags);
+#else
+ EXPECT_EQ(
+ gfx::Canvas::MULTI_LINE |
+ gfx::Canvas::TEXT_ALIGN_RIGHT |
+ gfx::Canvas::NO_ELLIPSIS,
+ flags);
#endif
label.SetHorizontalAlignment(gfx::ALIGN_RIGHT);
@@ -881,10 +907,14 @@
EXPECT_EQ(border.top() + extra.height() / 2, text_bounds.y());
EXPECT_EQ(center_bounds.width(), text_bounds.width());
EXPECT_EQ(center_bounds.height(), text_bounds.height());
- EXPECT_TRUE(gfx::Canvas::MULTI_LINE & flags);
- EXPECT_TRUE(gfx::Canvas::TEXT_ALIGN_LEFT & flags);
-#if !defined(OS_WIN)
- EXPECT_TRUE(gfx::Canvas::NO_ELLIPSIS & flags);
+#if defined(OS_WIN)
+ EXPECT_EQ(gfx::Canvas::MULTI_LINE | gfx::Canvas::TEXT_ALIGN_LEFT, flags);
+#else
+ EXPECT_EQ(
+ gfx::Canvas::MULTI_LINE |
+ gfx::Canvas::TEXT_ALIGN_LEFT |
+ gfx::Canvas::NO_ELLIPSIS,
+ flags);
#endif
// Reset Locale
« no previous file with comments | « ui/views/controls/label.cc ('k') | ui/views/controls/scroll_view.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698