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

Issues Search
    My Issues | Starred     Open | Closed | All

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

Issue 867003002: Cache gfx::RenderText instances in views::Label. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: do not overuse the memory Created 5 years, 10 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
« ui/views/controls/label.cc ('K') | « ui/views/controls/label.cc ('k') | no next file » | 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 56fd144a13abba2cfdadae87ff48ba2575660ce8..a89be9eb5a952976cbc9e20f6f4f15ea44967c5e 100644
--- a/ui/views/controls/label_unittest.cc
+++ b/ui/views/controls/label_unittest.cc
@@ -77,19 +77,18 @@ TEST_F(LabelTest, AlignmentProperty) {
// The alignment should be flipped in RTL UI.
label.SetHorizontalAlignment(gfx::ALIGN_RIGHT);
EXPECT_EQ(reverse_alignment ? gfx::ALIGN_LEFT : gfx::ALIGN_RIGHT,
- label.GetHorizontalAlignment());
+ label.horizontal_alignment());
label.SetHorizontalAlignment(gfx::ALIGN_LEFT);
EXPECT_EQ(reverse_alignment ? gfx::ALIGN_RIGHT : gfx::ALIGN_LEFT,
- label.GetHorizontalAlignment());
+ label.horizontal_alignment());
label.SetHorizontalAlignment(gfx::ALIGN_CENTER);
- EXPECT_EQ(gfx::ALIGN_CENTER, label.GetHorizontalAlignment());
+ EXPECT_EQ(gfx::ALIGN_CENTER, label.horizontal_alignment());
for (size_t j = 0; j < 2; ++j) {
label.SetHorizontalAlignment(gfx::ALIGN_TO_HEAD);
const bool rtl = j == 0;
label.SetText(rtl ? base::WideToUTF16(L"\x5d0") : ASCIIToUTF16("A"));
- EXPECT_EQ(rtl ? gfx::ALIGN_RIGHT : gfx::ALIGN_LEFT,
- label.GetHorizontalAlignment());
+ EXPECT_EQ(gfx::ALIGN_TO_HEAD, label.horizontal_alignment());
}
}
@@ -212,6 +211,7 @@ TEST_F(LabelTest, TooltipProperty) {
// Making the label multiline shouldn't eliminate the tooltip.
label.SetMultiLine(true);
EXPECT_TRUE(label.GetTooltipText(gfx::Point(), &tooltip));
+
// Expanding the multiline label bounds should eliminate the tooltip.
label.SetBounds(0, 0, 1000, 1000);
EXPECT_FALSE(label.GetTooltipText(gfx::Point(), &tooltip));
@@ -357,510 +357,6 @@ TEST_F(LabelTest, MultiLineSizing) {
required_size.width() + border.width());
}
-TEST_F(LabelTest, DirectionalityFromText) {
- Label label;
- label.SetBounds(0, 0, 1000, 1000);
-
- // Test text starts with RTL character.
- label.SetText(base::WideToUTF16(L" \x5d0\x5d1\x5d2 abc"));
- const Label::DrawStringParams* params = label.CalculateDrawStringParams();
- EXPECT_EQ(gfx::Canvas::FORCE_RTL_DIRECTIONALITY,
- params->flags & (gfx::Canvas::FORCE_RTL_DIRECTIONALITY |
- gfx::Canvas::FORCE_LTR_DIRECTIONALITY));
-
- // Test text starts with LTR character.
- label.SetText(base::WideToUTF16(L"ltr \x5d0\x5d1\x5d2 abc"));
- params = label.CalculateDrawStringParams();
- EXPECT_EQ(gfx::Canvas::FORCE_LTR_DIRECTIONALITY,
- params->flags & (gfx::Canvas::FORCE_RTL_DIRECTIONALITY |
- gfx::Canvas::FORCE_LTR_DIRECTIONALITY));
-}
-
-TEST_F(LabelTest, DrawSingleLineString) {
- Label label;
- label.SetFocusable(false);
-
- label.SetText(ASCIIToUTF16("Here's a string with no returns."));
- gfx::Size required_size(label.GetPreferredSize());
- gfx::Size extra(22, 8);
- label.SetBounds(0, 0, required_size.width() + extra.width(),
- required_size.height() + extra.height());
-
- // Do some basic verifications for all three alignments.
- // Centered text.
- const Label::DrawStringParams* params = label.CalculateDrawStringParams();
- EXPECT_EQ(label.text(), params->text);
- // The text should be centered horizontally and vertically.
- EXPECT_EQ(extra.width() / 2, params->bounds.x());
- EXPECT_EQ(0, params->bounds.y());
- EXPECT_EQ(required_size.width(), params->bounds.width());
- EXPECT_EQ(label.height(), params->bounds.height());
- EXPECT_EQ(gfx::Canvas::TEXT_ALIGN_CENTER,
- params->flags & (gfx::Canvas::TEXT_ALIGN_LEFT |
- gfx::Canvas::TEXT_ALIGN_CENTER |
- gfx::Canvas::TEXT_ALIGN_RIGHT));
-
- // Left aligned text.
- label.SetHorizontalAlignment(gfx::ALIGN_LEFT);
- label.ResetLayoutCache();
- params = label.CalculateDrawStringParams();
- EXPECT_EQ(label.text(), params->text);
- // The text should be left aligned horizontally and centered vertically.
- EXPECT_EQ(0, params->bounds.x());
- EXPECT_EQ(0, params->bounds.y());
- EXPECT_EQ(required_size.width(), params->bounds.width());
- EXPECT_EQ(label.height(), params->bounds.height());
- EXPECT_EQ(gfx::Canvas::TEXT_ALIGN_LEFT,
- params->flags & (gfx::Canvas::TEXT_ALIGN_LEFT |
- gfx::Canvas::TEXT_ALIGN_CENTER |
- gfx::Canvas::TEXT_ALIGN_RIGHT));
-
- // Right aligned text.
- label.SetHorizontalAlignment(gfx::ALIGN_RIGHT);
- label.ResetLayoutCache();
- params = label.CalculateDrawStringParams();
- EXPECT_EQ(label.text(), params->text);
- // The text should be right aligned horizontally and centered vertically.
- EXPECT_EQ(extra.width(), params->bounds.x());
- EXPECT_EQ(0, params->bounds.y());
- EXPECT_EQ(required_size.width(), params->bounds.width());
- EXPECT_EQ(label.height(), params->bounds.height());
- EXPECT_EQ(gfx::Canvas::TEXT_ALIGN_RIGHT,
- params->flags & (gfx::Canvas::TEXT_ALIGN_LEFT |
- gfx::Canvas::TEXT_ALIGN_CENTER |
- gfx::Canvas::TEXT_ALIGN_RIGHT));
-
- // Test single line drawing with a border.
- gfx::Insets border(39, 34, 8, 96);
- label.SetBorder(Border::CreateEmptyBorder(
- border.top(), border.left(), border.bottom(), border.right()));
-
- gfx::Size required_size_with_border(label.GetPreferredSize());
- EXPECT_EQ(required_size.width() + border.width(),
- required_size_with_border.width());
- EXPECT_EQ(required_size.height() + border.height(),
- required_size_with_border.height());
- label.SetBounds(0, 0, required_size_with_border.width() + extra.width(),
- required_size_with_border.height() + extra.height());
-
- // Centered text with border.
- label.SetHorizontalAlignment(gfx::ALIGN_CENTER);
- label.ResetLayoutCache();
- params = label.CalculateDrawStringParams();
- EXPECT_EQ(label.text(), params->text);
- // The text should be centered horizontally and vertically within the border.
- EXPECT_EQ(border.left() + extra.width() / 2, params->bounds.x());
- EXPECT_EQ(border.top(), params->bounds.y());
- EXPECT_EQ(required_size.width(), params->bounds.width());
- EXPECT_EQ(label.GetContentsBounds().height(), params->bounds.height());
- EXPECT_EQ(gfx::Canvas::TEXT_ALIGN_CENTER,
- params->flags & (gfx::Canvas::TEXT_ALIGN_LEFT |
- gfx::Canvas::TEXT_ALIGN_CENTER |
- gfx::Canvas::TEXT_ALIGN_RIGHT));
-
- // Left aligned text with border.
- label.SetHorizontalAlignment(gfx::ALIGN_LEFT);
- label.ResetLayoutCache();
- params = label.CalculateDrawStringParams();
- EXPECT_EQ(label.text(), params->text);
- // The text should be left aligned horizontally and centered vertically.
- EXPECT_EQ(border.left(), params->bounds.x());
- EXPECT_EQ(border.top(), params->bounds.y());
- EXPECT_EQ(required_size.width(), params->bounds.width());
- EXPECT_EQ(label.GetContentsBounds().height(), params->bounds.height());
- EXPECT_EQ(gfx::Canvas::TEXT_ALIGN_LEFT,
- params->flags & (gfx::Canvas::TEXT_ALIGN_LEFT |
- gfx::Canvas::TEXT_ALIGN_CENTER |
- gfx::Canvas::TEXT_ALIGN_RIGHT));
-
- // Right aligned text with border.
- label.SetHorizontalAlignment(gfx::ALIGN_RIGHT);
- label.ResetLayoutCache();
- params = label.CalculateDrawStringParams();
- EXPECT_EQ(label.text(), params->text);
- // The text should be right aligned horizontally and centered vertically.
- EXPECT_EQ(border.left() + extra.width(), params->bounds.x());
- EXPECT_EQ(border.top(), params->bounds.y());
- EXPECT_EQ(required_size.width(), params->bounds.width());
- EXPECT_EQ(label.GetContentsBounds().height(), params->bounds.height());
- EXPECT_EQ(gfx::Canvas::TEXT_ALIGN_RIGHT,
- params->flags & (gfx::Canvas::TEXT_ALIGN_LEFT |
- gfx::Canvas::TEXT_ALIGN_CENTER |
- gfx::Canvas::TEXT_ALIGN_RIGHT));
-}
-
-// Pango needs a max height to elide multiline text; that is not supported here.
-TEST_F(LabelTest, DrawMultiLineString) {
- Label label;
- label.SetFocusable(false);
- // Set a background color to prevent gfx::Canvas::NO_SUBPIXEL_RENDERING flags.
- label.SetBackgroundColor(SK_ColorWHITE);
-
- label.SetText(ASCIIToUTF16("Another string\nwith returns\n\n!"));
- label.SetMultiLine(true);
- label.SizeToFit(0);
- gfx::Size extra(50, 10);
- label.SetBounds(label.x(), label.y(),
- label.width() + extra.width(),
- label.height() + extra.height());
-
- // Do some basic verifications for all three alignments.
- const Label::DrawStringParams* params = label.CalculateDrawStringParams();
- EXPECT_EQ(label.text(), params->text);
- EXPECT_EQ(extra.width() / 2, params->bounds.x());
- EXPECT_EQ(extra.height() / 2, params->bounds.y());
- EXPECT_GT(params->bounds.width(), kMinTextDimension);
- EXPECT_GT(params->bounds.height(), kMinTextDimension);
- 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);
- gfx::Rect center_bounds(params->bounds);
-
- label.SetHorizontalAlignment(gfx::ALIGN_LEFT);
- label.ResetLayoutCache();
- params = label.CalculateDrawStringParams();
- EXPECT_EQ(label.text(), params->text);
- EXPECT_EQ(0, params->bounds.x());
- EXPECT_EQ(extra.height() / 2, params->bounds.y());
- EXPECT_GT(params->bounds.width(), kMinTextDimension);
- EXPECT_GT(params->bounds.height(), kMinTextDimension);
- 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);
-
- label.SetHorizontalAlignment(gfx::ALIGN_RIGHT);
- label.ResetLayoutCache();
- params = label.CalculateDrawStringParams();
- EXPECT_EQ(label.text(), params->text);
- EXPECT_EQ(extra.width(), params->bounds.x());
- EXPECT_EQ(extra.height() / 2, params->bounds.y());
- EXPECT_GT(params->bounds.width(), kMinTextDimension);
- EXPECT_GT(params->bounds.height(), kMinTextDimension);
- 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);
-
- // Test multiline drawing with a border.
- gfx::Insets border(19, 92, 23, 2);
- label.SetBorder(Border::CreateEmptyBorder(
- border.top(), border.left(), border.bottom(), border.right()));
- label.SizeToFit(0);
- label.SetBounds(label.x(), label.y(),
- label.width() + extra.width(),
- label.height() + extra.height());
-
- label.SetHorizontalAlignment(gfx::ALIGN_CENTER);
- label.ResetLayoutCache();
- params = label.CalculateDrawStringParams();
- EXPECT_EQ(label.text(), params->text);
- EXPECT_EQ(border.left() + extra.width() / 2, params->bounds.x());
- EXPECT_EQ(border.top() + extra.height() / 2, params->bounds.y());
- EXPECT_EQ(center_bounds.width(), params->bounds.width());
- EXPECT_EQ(center_bounds.height(), params->bounds.height());
- 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);
-
- label.SetHorizontalAlignment(gfx::ALIGN_LEFT);
- label.ResetLayoutCache();
- params = label.CalculateDrawStringParams();
- EXPECT_EQ(label.text(), params->text);
- EXPECT_EQ(border.left(), params->bounds.x());
- EXPECT_EQ(border.top() + extra.height() / 2, params->bounds.y());
- EXPECT_EQ(center_bounds.width(), params->bounds.width());
- EXPECT_EQ(center_bounds.height(), params->bounds.height());
- 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);
-
- label.SetHorizontalAlignment(gfx::ALIGN_RIGHT);
- label.ResetLayoutCache();
- params = label.CalculateDrawStringParams();
- EXPECT_EQ(label.text(), params->text);
- EXPECT_EQ(extra.width() + border.left(), params->bounds.x());
- EXPECT_EQ(border.top() + extra.height() / 2, params->bounds.y());
- EXPECT_EQ(center_bounds.width(), params->bounds.width());
- EXPECT_EQ(center_bounds.height(), params->bounds.height());
- 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);
-}
-
-TEST_F(LabelTest, DrawSingleLineStringInRTL) {
- Label label;
- label.SetFocusable(false);
-
- std::string locale = l10n_util::GetApplicationLocale("");
- base::i18n::SetICUDefaultLocale("he");
-
- label.SetText(ASCIIToUTF16("Here's a string with no returns."));
- gfx::Size required_size(label.GetPreferredSize());
- gfx::Size extra(22, 8);
- label.SetBounds(0, 0, required_size.width() + extra.width(),
- required_size.height() + extra.height());
-
- // Do some basic verifications for all three alignments.
- // Centered text.
- const Label::DrawStringParams* params = label.CalculateDrawStringParams();
- EXPECT_EQ(label.text(), params->text);
- // The text should be centered horizontally and vertically.
- EXPECT_EQ(extra.width() / 2, params->bounds.x());
- EXPECT_EQ(0, params->bounds.y());
- EXPECT_EQ(required_size.width(), params->bounds.width());
- EXPECT_EQ(label.height(), params->bounds.height());
- EXPECT_EQ(gfx::Canvas::TEXT_ALIGN_CENTER,
- params->flags & (gfx::Canvas::TEXT_ALIGN_LEFT |
- gfx::Canvas::TEXT_ALIGN_CENTER |
- gfx::Canvas::TEXT_ALIGN_RIGHT));
-
- // ALIGN_LEFT label.
- label.SetHorizontalAlignment(gfx::ALIGN_LEFT);
- label.ResetLayoutCache();
- params = label.CalculateDrawStringParams();
- EXPECT_EQ(label.text(), params->text);
- // The text should be right aligned horizontally and centered vertically.
- EXPECT_EQ(extra.width(), params->bounds.x());
- EXPECT_EQ(0, params->bounds.y());
- EXPECT_EQ(required_size.width(), params->bounds.width());
- EXPECT_EQ(label.height(), params->bounds.height());
- EXPECT_EQ(gfx::Canvas::TEXT_ALIGN_RIGHT,
- params->flags & (gfx::Canvas::TEXT_ALIGN_LEFT |
- gfx::Canvas::TEXT_ALIGN_CENTER |
- gfx::Canvas::TEXT_ALIGN_RIGHT));
-
- // ALIGN_RIGHT label.
- label.SetHorizontalAlignment(gfx::ALIGN_RIGHT);
- label.ResetLayoutCache();
- params = label.CalculateDrawStringParams();
- EXPECT_EQ(label.text(), params->text);
- // The text should be left aligned horizontally and centered vertically.
- EXPECT_EQ(0, params->bounds.x());
- EXPECT_EQ(0, params->bounds.y());
- EXPECT_EQ(required_size.width(), params->bounds.width());
- EXPECT_EQ(label.height(), params->bounds.height());
- EXPECT_EQ(gfx::Canvas::TEXT_ALIGN_LEFT,
- params->flags & (gfx::Canvas::TEXT_ALIGN_LEFT |
- gfx::Canvas::TEXT_ALIGN_CENTER |
- gfx::Canvas::TEXT_ALIGN_RIGHT));
-
-
- // Test single line drawing with a border.
- gfx::Insets border(39, 34, 8, 96);
- label.SetBorder(Border::CreateEmptyBorder(
- border.top(), border.left(), border.bottom(), border.right()));
-
- gfx::Size required_size_with_border(label.GetPreferredSize());
- EXPECT_EQ(required_size.width() + border.width(),
- required_size_with_border.width());
- EXPECT_EQ(required_size.height() + border.height(),
- required_size_with_border.height());
- label.SetBounds(0, 0, required_size_with_border.width() + extra.width(),
- required_size_with_border.height() + extra.height());
-
- // Centered text with border.
- label.SetHorizontalAlignment(gfx::ALIGN_CENTER);
- label.ResetLayoutCache();
- params = label.CalculateDrawStringParams();
- EXPECT_EQ(label.text(), params->text);
- // The text should be centered horizontally and vertically within the border.
- EXPECT_EQ(border.left() + extra.width() / 2, params->bounds.x());
- EXPECT_EQ(border.top(), params->bounds.y());
- EXPECT_EQ(required_size.width(), params->bounds.width());
- EXPECT_EQ(label.GetContentsBounds().height(), params->bounds.height());
- EXPECT_EQ(gfx::Canvas::TEXT_ALIGN_CENTER,
- params->flags & (gfx::Canvas::TEXT_ALIGN_LEFT |
- gfx::Canvas::TEXT_ALIGN_CENTER |
- gfx::Canvas::TEXT_ALIGN_RIGHT));
-
- // ALIGN_LEFT text with border.
- label.SetHorizontalAlignment(gfx::ALIGN_LEFT);
- label.ResetLayoutCache();
- params = label.CalculateDrawStringParams();
- EXPECT_EQ(label.text(), params->text);
- // The text should be right aligned horizontally and centered vertically.
- EXPECT_EQ(border.left() + extra.width(), params->bounds.x());
- EXPECT_EQ(border.top(), params->bounds.y());
- EXPECT_EQ(required_size.width(), params->bounds.width());
- EXPECT_EQ(label.GetContentsBounds().height(), params->bounds.height());
- EXPECT_EQ(gfx::Canvas::TEXT_ALIGN_RIGHT,
- params->flags & (gfx::Canvas::TEXT_ALIGN_LEFT |
- gfx::Canvas::TEXT_ALIGN_CENTER |
- gfx::Canvas::TEXT_ALIGN_RIGHT));
-
- // ALIGN_RIGHT text.
- label.SetHorizontalAlignment(gfx::ALIGN_RIGHT);
- label.ResetLayoutCache();
- params = label.CalculateDrawStringParams();
- EXPECT_EQ(label.text(), params->text);
- // The text should be left aligned horizontally and centered vertically.
- EXPECT_EQ(border.left(), params->bounds.x());
- EXPECT_EQ(border.top(), params->bounds.y());
- EXPECT_EQ(required_size.width(), params->bounds.width());
- EXPECT_EQ(label.GetContentsBounds().height(), params->bounds.height());
- EXPECT_EQ(gfx::Canvas::TEXT_ALIGN_LEFT,
- params->flags & (gfx::Canvas::TEXT_ALIGN_LEFT |
- gfx::Canvas::TEXT_ALIGN_CENTER |
- gfx::Canvas::TEXT_ALIGN_RIGHT));
-
- // Reset locale.
- base::i18n::SetICUDefaultLocale(locale);
-}
-
-// On Linux the underlying pango routines require a max height in order to
-// ellide multiline text. So until that can be resolved, we set all
-// multiline lables to not ellide in Linux only.
-TEST_F(LabelTest, DrawMultiLineStringInRTL) {
- Label label;
- label.SetFocusable(false);
-
- // Test for RTL.
- std::string locale = l10n_util::GetApplicationLocale("");
- base::i18n::SetICUDefaultLocale("he");
-
- label.SetText(ASCIIToUTF16("Another string\nwith returns\n\n!"));
- label.SetMultiLine(true);
- label.SizeToFit(0);
- gfx::Size extra(50, 10);
- label.SetBounds(label.x(), label.y(),
- label.width() + extra.width(),
- label.height() + extra.height());
-
- // Do some basic verifications for all three alignments.
- const Label::DrawStringParams* params = label.CalculateDrawStringParams();
- EXPECT_EQ(label.text(), params->text);
- EXPECT_EQ(extra.width() / 2, params->bounds.x());
- EXPECT_EQ(extra.height() / 2, params->bounds.y());
- EXPECT_GT(params->bounds.width(), kMinTextDimension);
- EXPECT_GT(params->bounds.height(), kMinTextDimension);
- EXPECT_TRUE(gfx::Canvas::MULTI_LINE & params->flags);
- EXPECT_TRUE(gfx::Canvas::TEXT_ALIGN_CENTER & params->flags);
-#if !defined(OS_WIN)
- EXPECT_TRUE(gfx::Canvas::NO_ELLIPSIS & params->flags);
-#endif
- gfx::Rect center_bounds(params->bounds);
-
- label.SetHorizontalAlignment(gfx::ALIGN_LEFT);
- label.ResetLayoutCache();
- params = label.CalculateDrawStringParams();
- EXPECT_EQ(label.text(), params->text);
- EXPECT_EQ(extra.width(), params->bounds.x());
- EXPECT_EQ(extra.height() / 2, params->bounds.y());
- EXPECT_GT(params->bounds.width(), kMinTextDimension);
- EXPECT_GT(params->bounds.height(), kMinTextDimension);
- EXPECT_TRUE(gfx::Canvas::MULTI_LINE & params->flags);
- EXPECT_TRUE(gfx::Canvas::TEXT_ALIGN_RIGHT & params->flags);
-#if !defined(OS_WIN)
- EXPECT_TRUE(gfx::Canvas::NO_ELLIPSIS & params->flags);
-#endif
-
- label.SetHorizontalAlignment(gfx::ALIGN_RIGHT);
- label.ResetLayoutCache();
- params = label.CalculateDrawStringParams();
- EXPECT_EQ(label.text(), params->text);
- EXPECT_EQ(0, params->bounds.x());
- EXPECT_EQ(extra.height() / 2, params->bounds.y());
- EXPECT_GT(params->bounds.width(), kMinTextDimension);
- EXPECT_GT(params->bounds.height(), kMinTextDimension);
- EXPECT_TRUE(gfx::Canvas::MULTI_LINE & params->flags);
- EXPECT_TRUE(gfx::Canvas::TEXT_ALIGN_LEFT & params->flags);
-#if !defined(OS_WIN)
- EXPECT_TRUE(gfx::Canvas::NO_ELLIPSIS & params->flags);
-#endif
-
- // Test multiline drawing with a border.
- gfx::Insets border(19, 92, 23, 2);
- label.SetBorder(Border::CreateEmptyBorder(
- border.top(), border.left(), border.bottom(), border.right()));
- label.SizeToFit(0);
- label.SetBounds(label.x(), label.y(),
- label.width() + extra.width(),
- label.height() + extra.height());
-
- label.SetHorizontalAlignment(gfx::ALIGN_CENTER);
- label.ResetLayoutCache();
- params = label.CalculateDrawStringParams();
- EXPECT_EQ(label.text(), params->text);
- EXPECT_EQ(border.left() + extra.width() / 2, params->bounds.x());
- EXPECT_EQ(border.top() + extra.height() / 2, params->bounds.y());
- EXPECT_EQ(center_bounds.width(), params->bounds.width());
- EXPECT_EQ(center_bounds.height(), params->bounds.height());
- EXPECT_TRUE(gfx::Canvas::MULTI_LINE & params->flags);
- EXPECT_TRUE(gfx::Canvas::TEXT_ALIGN_CENTER & params->flags);
-#if !defined(OS_WIN)
- EXPECT_TRUE(gfx::Canvas::NO_ELLIPSIS & params->flags);
-#endif
-
- label.SetHorizontalAlignment(gfx::ALIGN_LEFT);
- label.ResetLayoutCache();
- params = label.CalculateDrawStringParams();
- EXPECT_EQ(label.text(), params->text);
- EXPECT_EQ(border.left() + extra.width(), params->bounds.x());
- EXPECT_EQ(border.top() + extra.height() / 2, params->bounds.y());
- EXPECT_EQ(center_bounds.width(), params->bounds.width());
- EXPECT_EQ(center_bounds.height(), params->bounds.height());
- EXPECT_TRUE(gfx::Canvas::MULTI_LINE & params->flags);
- EXPECT_TRUE(gfx::Canvas::TEXT_ALIGN_RIGHT & params->flags);
-#if !defined(OS_WIN)
- EXPECT_TRUE(gfx::Canvas::NO_ELLIPSIS & params->flags);
-#endif
-
- label.SetHorizontalAlignment(gfx::ALIGN_RIGHT);
- label.ResetLayoutCache();
- params = label.CalculateDrawStringParams();
- EXPECT_EQ(label.text(), params->text);
- EXPECT_EQ(border.left(), params->bounds.x());
- EXPECT_EQ(border.top() + extra.height() / 2, params->bounds.y());
- EXPECT_EQ(center_bounds.width(), params->bounds.width());
- EXPECT_EQ(center_bounds.height(), params->bounds.height());
- EXPECT_TRUE(gfx::Canvas::MULTI_LINE & params->flags);
- EXPECT_TRUE(gfx::Canvas::TEXT_ALIGN_LEFT & params->flags);
-#if !defined(OS_WIN)
- EXPECT_TRUE(gfx::Canvas::NO_ELLIPSIS & params->flags);
-#endif
-
- // Reset Locale
- base::i18n::SetICUDefaultLocale(locale);
-}
-
-// Ensure the subpixel rendering flag and background color alpha are respected.
-TEST_F(LabelTest, DisableSubpixelRendering) {
- Label label;
- label.SetBackgroundColor(SK_ColorWHITE);
- const int flag = gfx::Canvas::NO_SUBPIXEL_RENDERING;
- EXPECT_EQ(0, label.ComputeDrawStringFlags() & flag);
- label.SetSubpixelRenderingEnabled(false);
- EXPECT_EQ(flag, label.ComputeDrawStringFlags() & flag);
- label.SetSubpixelRenderingEnabled(true);
- EXPECT_EQ(0, label.ComputeDrawStringFlags() & flag);
- // Text cannot be drawn with subpixel rendering on transparent backgrounds.
- label.SetBackgroundColor(SkColorSetARGB(64, 255, 255, 255));
- EXPECT_EQ(flag, label.ComputeDrawStringFlags() & flag);
-}
-
// Check that labels support GetTooltipHandlerForPoint.
TEST_F(LabelTest, GetTooltipHandlerForPoint) {
// A root view must be defined for this test because the hit-testing
« ui/views/controls/label.cc ('K') | « ui/views/controls/label.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698