diff options
author | ckocagil@chromium.org <ckocagil@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-09-18 22:01:22 +0000 |
---|---|---|
committer | ckocagil@chromium.org <ckocagil@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-09-18 22:01:22 +0000 |
commit | 3dfe5c5b5688eef4b0a479c69b3a2e5afa0e4b0b (patch) | |
tree | 579a0525dc442209a00556b80152598dfa97020e /ui/gfx/render_text_unittest.cc | |
parent | 72221811013399d24bbcd89936878bd125c145ab (diff) | |
download | chromium_src-3dfe5c5b5688eef4b0a479c69b3a2e5afa0e4b0b.zip chromium_src-3dfe5c5b5688eef4b0a479c69b3a2e5afa0e4b0b.tar.gz chromium_src-3dfe5c5b5688eef4b0a479c69b3a2e5afa0e4b0b.tar.bz2 |
Remove explicit namespace resolution from code in gfx namespace
BUG=103304
R=sky@chromium.org
Review URL: https://codereview.chromium.org/24053002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@223952 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'ui/gfx/render_text_unittest.cc')
-rw-r--r-- | ui/gfx/render_text_unittest.cc | 74 |
1 files changed, 37 insertions, 37 deletions
diff --git a/ui/gfx/render_text_unittest.cc b/ui/gfx/render_text_unittest.cc index 5d1b248..52cb834 100644 --- a/ui/gfx/render_text_unittest.cc +++ b/ui/gfx/render_text_unittest.cc @@ -43,7 +43,7 @@ const wchar_t kRtlLtrRtl[] = L"\x5d0" L"a" L"\x5d1"; // Checks whether |range| contains |index|. This is not the same as calling // |range.Contains(gfx::Range(index))| - as that would return true when // |index| == |range.end()|. -bool IndexInRange(const gfx::Range& range, size_t index) { +bool IndexInRange(const Range& range, size_t index) { return index >= range.start() && index < range.end(); } @@ -125,8 +125,8 @@ TEST_F(RenderTextTest, ApplyColorAndStyle) { render_text->SetText(ASCIIToUTF16("012345678")); // Apply a ranged color and style and check the resulting breaks. - render_text->ApplyColor(SK_ColorRED, gfx::Range(1, 4)); - render_text->ApplyStyle(BOLD, true, gfx::Range(2, 5)); + render_text->ApplyColor(SK_ColorRED, Range(1, 4)); + render_text->ApplyStyle(BOLD, true, Range(2, 5)); std::vector<std::pair<size_t, SkColor> > expected_color; expected_color.push_back(std::pair<size_t, SkColor>(0, SK_ColorBLACK)); expected_color.push_back(std::pair<size_t, SkColor>(1, SK_ColorRED)); @@ -147,8 +147,8 @@ TEST_F(RenderTextTest, ApplyColorAndStyle) { // Apply a color and style over the text end and check the resulting breaks. // (INT_MAX should be used instead of the text length for the range end) const size_t text_length = render_text->text().length(); - render_text->ApplyColor(SK_ColorRED, gfx::Range(0, text_length)); - render_text->ApplyStyle(BOLD, true, gfx::Range(2, text_length)); + render_text->ApplyColor(SK_ColorRED, Range(0, text_length)); + render_text->ApplyStyle(BOLD, true, Range(2, text_length)); std::vector<std::pair<size_t, SkColor> > expected_color_end; expected_color_end.push_back(std::pair<size_t, SkColor>(0, SK_ColorRED)); EXPECT_TRUE(render_text->colors().EqualsForTesting(expected_color_end)); @@ -158,9 +158,9 @@ TEST_F(RenderTextTest, ApplyColorAndStyle) { EXPECT_TRUE(render_text->styles()[BOLD].EqualsForTesting(expected_style_end)); // Ensure ranged values adjust to accommodate text length changes. - render_text->ApplyStyle(ITALIC, true, gfx::Range(0, 2)); - render_text->ApplyStyle(ITALIC, true, gfx::Range(3, 6)); - render_text->ApplyStyle(ITALIC, true, gfx::Range(7, text_length)); + render_text->ApplyStyle(ITALIC, true, Range(0, 2)); + render_text->ApplyStyle(ITALIC, true, Range(3, 6)); + render_text->ApplyStyle(ITALIC, true, Range(7, text_length)); std::vector<std::pair<size_t, bool> > expected_italic; expected_italic.push_back(std::pair<size_t, bool>(0, true)); expected_italic.push_back(std::pair<size_t, bool>(2, false)); @@ -188,8 +188,8 @@ TEST_F(RenderTextTest, PangoAttributes) { render_text->SetText(ASCIIToUTF16("012345678")); // Apply ranged BOLD/ITALIC styles and check the resulting Pango attributes. - render_text->ApplyStyle(BOLD, true, gfx::Range(2, 4)); - render_text->ApplyStyle(ITALIC, true, gfx::Range(1, 3)); + render_text->ApplyStyle(BOLD, true, Range(2, 4)); + render_text->ApplyStyle(ITALIC, true, Range(1, 3)); struct { int start; @@ -240,22 +240,22 @@ void TestVisualCursorMotionInObscuredField(RenderText* render_text, render_text->SetText(text); int len = text.length(); render_text->MoveCursor(LINE_BREAK, CURSOR_RIGHT, select); - EXPECT_EQ(SelectionModel(gfx::Range(select ? 0 : len, len), CURSOR_FORWARD), + EXPECT_EQ(SelectionModel(Range(select ? 0 : len, len), CURSOR_FORWARD), render_text->selection_model()); render_text->MoveCursor(LINE_BREAK, CURSOR_LEFT, select); EXPECT_EQ(SelectionModel(0, CURSOR_BACKWARD), render_text->selection_model()); for (int j = 1; j <= len; ++j) { render_text->MoveCursor(CHARACTER_BREAK, CURSOR_RIGHT, select); - EXPECT_EQ(SelectionModel(gfx::Range(select ? 0 : j, j), CURSOR_BACKWARD), + EXPECT_EQ(SelectionModel(Range(select ? 0 : j, j), CURSOR_BACKWARD), render_text->selection_model()); } for (int j = len - 1; j >= 0; --j) { render_text->MoveCursor(CHARACTER_BREAK, CURSOR_LEFT, select); - EXPECT_EQ(SelectionModel(gfx::Range(select ? 0 : j, j), CURSOR_FORWARD), + EXPECT_EQ(SelectionModel(Range(select ? 0 : j, j), CURSOR_FORWARD), render_text->selection_model()); } render_text->MoveCursor(WORD_BREAK, CURSOR_RIGHT, select); - EXPECT_EQ(SelectionModel(gfx::Range(select ? 0 : len, len), CURSOR_FORWARD), + EXPECT_EQ(SelectionModel(Range(select ? 0 : len, len), CURSOR_FORWARD), render_text->selection_model()); render_text->MoveCursor(WORD_BREAK, CURSOR_LEFT, select); EXPECT_EQ(SelectionModel(0, CURSOR_BACKWARD), render_text->selection_model()); @@ -879,8 +879,8 @@ TEST_F(RenderTextTest, SelectAll) { // Ensure that SelectAll respects the |reversed| argument regardless of // application locale and text content directionality. scoped_ptr<RenderText> render_text(RenderText::CreateInstance()); - const SelectionModel expected_reversed(gfx::Range(3, 0), CURSOR_FORWARD); - const SelectionModel expected_forwards(gfx::Range(0, 3), CURSOR_BACKWARD); + const SelectionModel expected_reversed(Range(3, 0), CURSOR_FORWARD); + const SelectionModel expected_forwards(Range(0, 3), CURSOR_BACKWARD); const bool was_rtl = base::i18n::IsRTL(); for (size_t i = 0; i < 2; ++i) { @@ -907,39 +907,39 @@ TEST_F(RenderTextTest, SelectAll) { render_text->SetText(WideToUTF16(L"abc\x05d0\x05d1\x05d2")); // Left arrow on select ranging (6, 4). render_text->MoveCursor(LINE_BREAK, CURSOR_RIGHT, false); - EXPECT_EQ(gfx::Range(6), render_text->selection()); + EXPECT_EQ(Range(6), render_text->selection()); render_text->MoveCursor(CHARACTER_BREAK, CURSOR_LEFT, false); - EXPECT_EQ(gfx::Range(4), render_text->selection()); + EXPECT_EQ(Range(4), render_text->selection()); render_text->MoveCursor(CHARACTER_BREAK, CURSOR_LEFT, false); - EXPECT_EQ(gfx::Range(5), render_text->selection()); + EXPECT_EQ(Range(5), render_text->selection()); render_text->MoveCursor(CHARACTER_BREAK, CURSOR_LEFT, false); - EXPECT_EQ(gfx::Range(6), render_text->selection()); + EXPECT_EQ(Range(6), render_text->selection()); render_text->MoveCursor(CHARACTER_BREAK, CURSOR_RIGHT, true); - EXPECT_EQ(gfx::Range(6, 5), render_text->selection()); + EXPECT_EQ(Range(6, 5), render_text->selection()); render_text->MoveCursor(CHARACTER_BREAK, CURSOR_RIGHT, true); - EXPECT_EQ(gfx::Range(6, 4), render_text->selection()); + EXPECT_EQ(Range(6, 4), render_text->selection()); render_text->MoveCursor(CHARACTER_BREAK, CURSOR_LEFT, false); - EXPECT_EQ(gfx::Range(6), render_text->selection()); + EXPECT_EQ(Range(6), render_text->selection()); // Right arrow on select ranging (4, 6). render_text->MoveCursor(LINE_BREAK, CURSOR_LEFT, false); - EXPECT_EQ(gfx::Range(0), render_text->selection()); + EXPECT_EQ(Range(0), render_text->selection()); render_text->MoveCursor(CHARACTER_BREAK, CURSOR_RIGHT, false); - EXPECT_EQ(gfx::Range(1), render_text->selection()); + EXPECT_EQ(Range(1), render_text->selection()); render_text->MoveCursor(CHARACTER_BREAK, CURSOR_RIGHT, false); - EXPECT_EQ(gfx::Range(2), render_text->selection()); + EXPECT_EQ(Range(2), render_text->selection()); render_text->MoveCursor(CHARACTER_BREAK, CURSOR_RIGHT, false); - EXPECT_EQ(gfx::Range(3), render_text->selection()); + EXPECT_EQ(Range(3), render_text->selection()); render_text->MoveCursor(CHARACTER_BREAK, CURSOR_RIGHT, false); - EXPECT_EQ(gfx::Range(5), render_text->selection()); + EXPECT_EQ(Range(5), render_text->selection()); render_text->MoveCursor(CHARACTER_BREAK, CURSOR_RIGHT, false); - EXPECT_EQ(gfx::Range(4), render_text->selection()); + EXPECT_EQ(Range(4), render_text->selection()); render_text->MoveCursor(CHARACTER_BREAK, CURSOR_LEFT, true); - EXPECT_EQ(gfx::Range(4, 5), render_text->selection()); + EXPECT_EQ(Range(4, 5), render_text->selection()); render_text->MoveCursor(CHARACTER_BREAK, CURSOR_LEFT, true); - EXPECT_EQ(gfx::Range(4, 6), render_text->selection()); + EXPECT_EQ(Range(4, 6), render_text->selection()); render_text->MoveCursor(CHARACTER_BREAK, CURSOR_RIGHT, false); - EXPECT_EQ(gfx::Range(4), render_text->selection()); + EXPECT_EQ(Range(4), render_text->selection()); } #endif // !defined(OS_MACOSX) @@ -1226,7 +1226,7 @@ TEST_F(RenderTextTest, StringSizeBoldWidth) { EXPECT_GT(bold_width, plain_width); // Now, apply a plain style over the first word only. - render_text->ApplyStyle(BOLD, false, gfx::Range(0, 5)); + render_text->ApplyStyle(BOLD, false, Range(0, 5)); const int plain_bold_width = render_text->GetStringSize().width(); EXPECT_GT(plain_bold_width, plain_width); EXPECT_LT(plain_bold_width, bold_width); @@ -1465,7 +1465,7 @@ TEST_F(RenderTextTest, SelectWord) { for (size_t i = 0; i < ARRAYSIZE_UNSAFE(cases); ++i) { render_text->SetCursorPosition(cases[i].cursor); render_text->SelectWord(); - EXPECT_EQ(gfx::Range(cases[i].selection_start, cases[i].selection_end), + EXPECT_EQ(Range(cases[i].selection_start, cases[i].selection_end), render_text->selection()); } } @@ -1498,13 +1498,13 @@ TEST_F(RenderTextTest, SelectMultipleWords) { scoped_ptr<RenderText> render_text(RenderText::CreateInstance()); render_text->SetText(ASCIIToUTF16(kTestURL)); - render_text->SelectRange(gfx::Range(16, 20)); + render_text->SelectRange(Range(16, 20)); render_text->SelectWord(); EXPECT_EQ(ASCIIToUTF16("google.com"), GetSelectedText(render_text.get())); EXPECT_FALSE(render_text->selection().is_reversed()); // SelectWord should preserve the selection direction. - render_text->SelectRange(gfx::Range(20, 16)); + render_text->SelectRange(Range(20, 16)); render_text->SelectWord(); EXPECT_EQ(ASCIIToUTF16("google.com"), GetSelectedText(render_text.get())); EXPECT_TRUE(render_text->selection().is_reversed()); @@ -1643,7 +1643,7 @@ TEST_F(RenderTextTest, SelectionKeepsLigatures) { for (size_t i = 0; i < arraysize(kTestStrings); ++i) { render_text->SetText(WideToUTF16(kTestStrings[i])); const int expected_width = render_text->GetStringSize().width(); - render_text->MoveCursorTo(SelectionModel(gfx::Range(0, 1), CURSOR_FORWARD)); + render_text->MoveCursorTo(SelectionModel(Range(0, 1), CURSOR_FORWARD)); EXPECT_EQ(expected_width, render_text->GetStringSize().width()); // Draw the text. It shouldn't hit any DCHECKs or crash. // See http://crbug.com/214150 |