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

Unified Diff: ui/accessibility/ax_text_utils_unittest.cc

Issue 1780473002: Workaround for navigating by word in the omnibox on Windows. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Removed base from deps and fixed build.gn file (hopefully). Created 4 years, 9 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/accessibility/ax_text_utils.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/accessibility/ax_text_utils_unittest.cc
diff --git a/ui/accessibility/ax_text_utils_unittest.cc b/ui/accessibility/ax_text_utils_unittest.cc
index 1c2a4a1e4e11dc0cd71596336d511b3cee649e89..2c7f491d619aefbb1147845c654ec8a868990a0e 100644
--- a/ui/accessibility/ax_text_utils_unittest.cc
+++ b/ui/accessibility/ax_text_utils_unittest.cc
@@ -10,6 +10,49 @@
namespace ui {
+TEST(AXTextUtils, FindAccessibleTextBoundaryWord) {
+ const base::string16 text =
+ base::UTF8ToUTF16("Hello there.This/is\ntesting.");
+ const size_t text_length = text.length();
+ std::vector<int> line_start_offsets;
+ line_start_offsets.push_back(19);
+ size_t result;
+
+ result = FindAccessibleTextBoundary(text, line_start_offsets, WORD_BOUNDARY,
+ 0, FORWARDS_DIRECTION);
+ EXPECT_EQ(6UL, result);
+ result = FindAccessibleTextBoundary(text, line_start_offsets, WORD_BOUNDARY,
+ 5, BACKWARDS_DIRECTION);
+ EXPECT_EQ(0UL, result);
+ result = FindAccessibleTextBoundary(text, line_start_offsets, WORD_BOUNDARY,
+ 6, FORWARDS_DIRECTION);
+ EXPECT_EQ(12UL, result);
+ result = FindAccessibleTextBoundary(text, line_start_offsets, WORD_BOUNDARY,
+ 11, BACKWARDS_DIRECTION);
+ EXPECT_EQ(6UL, result);
+ result = FindAccessibleTextBoundary(text, line_start_offsets, WORD_BOUNDARY,
+ 12, BACKWARDS_DIRECTION);
+ EXPECT_EQ(12UL, result);
+ result = FindAccessibleTextBoundary(text, line_start_offsets, WORD_BOUNDARY,
+ 15, FORWARDS_DIRECTION);
+ EXPECT_EQ(17UL, result);
+ result = FindAccessibleTextBoundary(text, line_start_offsets, WORD_BOUNDARY,
+ 15, BACKWARDS_DIRECTION);
+ EXPECT_EQ(12UL, result);
+ result = FindAccessibleTextBoundary(text, line_start_offsets, WORD_BOUNDARY,
+ 16, FORWARDS_DIRECTION);
+ EXPECT_EQ(17UL, result);
+ result = FindAccessibleTextBoundary(text, line_start_offsets, WORD_BOUNDARY,
+ 17, FORWARDS_DIRECTION);
+ EXPECT_EQ(20UL, result);
+ result = FindAccessibleTextBoundary(text, line_start_offsets, WORD_BOUNDARY,
+ 20, FORWARDS_DIRECTION);
+ EXPECT_EQ(text_length, result);
+ result = FindAccessibleTextBoundary(text, line_start_offsets, WORD_BOUNDARY,
+ text_length, BACKWARDS_DIRECTION);
+ EXPECT_EQ(20UL, result);
+}
+
TEST(AXTextUtils, FindAccessibleTextBoundaryLine) {
const base::string16 text = base::UTF8ToUTF16("Line 1.\nLine 2\n\t");
const size_t text_length = text.length();
@@ -18,7 +61,6 @@ TEST(AXTextUtils, FindAccessibleTextBoundaryLine) {
line_start_offsets.push_back(15);
size_t result;
-
// Basic cases.
result = FindAccessibleTextBoundary(text, line_start_offsets, LINE_BOUNDARY,
5, FORWARDS_DIRECTION);
@@ -30,9 +72,7 @@ TEST(AXTextUtils, FindAccessibleTextBoundaryLine) {
10, FORWARDS_DIRECTION);
EXPECT_EQ(15UL, result);
-
// Edge cases.
-
result = FindAccessibleTextBoundary(text, line_start_offsets, LINE_BOUNDARY,
text_length, BACKWARDS_DIRECTION);
EXPECT_EQ(15UL, result);
@@ -68,4 +108,4 @@ TEST(AXTextUtils, FindAccessibleTextBoundaryLine) {
EXPECT_EQ(text_length, result);
}
-} // Namespace ui.
+} // namespace ui
« no previous file with comments | « ui/accessibility/ax_text_utils.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698