| Index: ui/gfx/render_text_unittest.cc
|
| diff --git a/ui/gfx/render_text_unittest.cc b/ui/gfx/render_text_unittest.cc
|
| index 75e985e01a8dc48870f31bb66397007e14adc145..350df9178e21629a6c5f75b28290e9d088f53b21 100644
|
| --- a/ui/gfx/render_text_unittest.cc
|
| +++ b/ui/gfx/render_text_unittest.cc
|
| @@ -5,6 +5,7 @@
|
| #include "ui/gfx/render_text.h"
|
|
|
| #include <algorithm>
|
| +#include <limits>
|
|
|
| #include "base/format_macros.h"
|
| #include "base/memory/scoped_ptr.h"
|
| @@ -1139,6 +1140,8 @@ TEST_F(RenderTextTest, StringSizeEmptyString) {
|
| const FontList font_list("Arial,Symbol, 16px");
|
| scoped_ptr<RenderText> render_text(RenderText::CreateInstance());
|
| render_text->SetFontList(font_list);
|
| + render_text->SetDisplayRect(Rect(0, 0, std::numeric_limits<int>::max(),
|
| + font_list.GetHeight()));
|
|
|
| // The empty string respects FontList metrics for non-zero height
|
| // and baseline.
|
| @@ -1180,7 +1183,9 @@ TEST_F(RenderTextTest, StringSizeRespectsFontListMetrics) {
|
| // Check |smaller_font_text| is rendered with the smaller font.
|
| scoped_ptr<RenderText> render_text(RenderText::CreateInstance());
|
| render_text->SetText(UTF8ToUTF16(smaller_font_text));
|
| - render_text->SetFont(smaller_font);
|
| + render_text->SetFontList(FontList(smaller_font));
|
| + render_text->SetDisplayRect(Rect(0, 0, std::numeric_limits<int>::max(),
|
| + render_text->font_list().GetHeight()));
|
| EXPECT_EQ(smaller_font.GetHeight(), render_text->GetStringSize().height());
|
| EXPECT_EQ(smaller_font.GetBaseline(), render_text->GetBaseline());
|
|
|
| @@ -1192,6 +1197,8 @@ TEST_F(RenderTextTest, StringSizeRespectsFontListMetrics) {
|
| fonts.push_back(larger_font);
|
| const FontList font_list(fonts);
|
| render_text->SetFontList(font_list);
|
| + render_text->SetDisplayRect(Rect(0, 0, std::numeric_limits<int>::max(),
|
| + render_text->font_list().GetHeight()));
|
| EXPECT_LT(smaller_font.GetHeight(), render_text->GetStringSize().height());
|
| EXPECT_LT(smaller_font.GetBaseline(), render_text->GetBaseline());
|
| EXPECT_EQ(font_list.GetHeight(), render_text->GetStringSize().height());
|
| @@ -1321,17 +1328,6 @@ TEST_F(RenderTextTest, GetTextOffset) {
|
| offset = render_text->GetLineOffset(0);
|
| EXPECT_EQ(kEnlargement, offset.x());
|
|
|
| - // Check explicitly setting the vertical alignment.
|
| - render_text->SetVerticalAlignment(ALIGN_TOP);
|
| - offset = render_text->GetLineOffset(0);
|
| - EXPECT_EQ(0, offset.y());
|
| - render_text->SetVerticalAlignment(ALIGN_VCENTER);
|
| - offset = render_text->GetLineOffset(0);
|
| - EXPECT_EQ(kEnlargement / 2, offset.y());
|
| - render_text->SetVerticalAlignment(ALIGN_BOTTOM);
|
| - offset = render_text->GetLineOffset(0);
|
| - EXPECT_EQ(kEnlargement, offset.y());
|
| -
|
| SetRTL(was_rtl);
|
| }
|
|
|
|
|