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

Unified Diff: components/autofill/core/common/autofill_util_unittest.cc

Issue 962673004: [Autofill/Autocomplete Feature] Substring matching instead of prefix matching. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 6 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
Index: components/autofill/core/common/autofill_util_unittest.cc
diff --git a/components/autofill/core/common/autofill_util_unittest.cc b/components/autofill/core/common/autofill_util_unittest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..1df4057bc0ee63716a384b93c994fc4fb4f62925
--- /dev/null
+++ b/components/autofill/core/common/autofill_util_unittest.cc
@@ -0,0 +1,96 @@
+// Copyright 2015 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "components/autofill/core/common/autofill_util.h"
+
+#include "base/command_line.h"
+#include "base/strings/utf_string_conversions.h"
+#include "components/autofill/core/common/autofill_switches.h"
+#include "testing/gtest/include/gtest/gtest.h"
+
+namespace autofill {
+
+// Tests for ContainsTokenThatStartsWith().
+TEST(AutofillUtilTest, ContainsTokenThatStartsWith) {
+ // ContainsTokenThatStartsWith should not work yet without a flag.
+ EXPECT_FALSE(ContainsTokenThatStartsWith(base::ASCIIToUTF16("ab@cd.b"),
+ base::ASCIIToUTF16("a"), false));
+
+ // Token matching is currently behind a flag.
+ base::CommandLine::ForCurrentProcess()->AppendSwitch(
+ autofill::switches::kEnableSuggestionsWithSubstringMatch);
+
+ const struct {
+ const char* const field_suggestion;
+ const char* const field_contents;
+ bool case_sensitive;
+ bool expected_result;
+ } test_cases[] = {
+ {"ab@cd.b", "a", false, true},
+ {"ab@cd.b", "b", false, true},
+ {"ab@cd.b", "Ab", false, true},
+ {"ab@cd.b", "Ab", true, false},
+ {"ab@cd.b", "cd", true, true},
+ {"ab@cd.b", "d", false, false},
+ {"ab@cd.b", "b@", true, false},
+ {"ab@cd.b", "ab", false, true},
+ {"ab@cd.b", "cd.b", true, true},
+ {"ab@cd.b", "b@cd", false, false},
+ {"", "ab", false, false},
+ {"", "ab", true, false},
+ {"ab", "", false, true},
+ {"ab", "", true, true},
+ };
+
+ for (size_t i = 0; i < arraysize(test_cases); ++i) {
+ SCOPED_TRACE(testing::Message()
+ << "suggestion = " << test_cases[i].field_suggestion
+ << ", contents = " << test_cases[i].field_contents
+ << ", case_sensitive = " << test_cases[i].case_sensitive);
+
+ EXPECT_EQ(test_cases[i].expected_result,
+ ContainsTokenThatStartsWith(
+ base::ASCIIToUTF16(test_cases[i].field_suggestion),
+ base::ASCIIToUTF16(test_cases[i].field_contents),
+ test_cases[i].case_sensitive));
+ }
+}
+
+// Tests for GetTextSelectionStart().
+TEST(AutofillUtilTest, GetTextSelectionStart) {
+ const size_t kInvalid = base::string16::npos;
+ const struct {
+ const char* const field_suggestion;
+ const char* const field_contents;
+ bool case_sensitive;
+ size_t expected_start;
+ } test_cases[] = {
+ {"ab@cd.b", "a", false, 1},
+ {"ab@cd.b", "A", true, kInvalid},
+ {"ab@cd.b", "Ab", false, 2},
+ {"ab@cd.b", "Ab", true, kInvalid},
+ {"ab@cd.b", "cd", false, 5},
+ {"ab@cd.b", "ab@c", false, 4},
+ {"ab@cd.b", "cd.b", false, 7},
+ {"ab@cd.b", "b@cd", false, kInvalid},
+ {"ab@cd.b", "b", false, 7},
+ {"ba.a.ab", "a.a", false, 6},
+ {"texample@example.com", "example", false, 16},
+ };
+
+ for (size_t i = 0; i < arraysize(test_cases); ++i) {
+ SCOPED_TRACE(testing::Message()
+ << "suggestion = " << test_cases[i].field_suggestion
+ << ", contents = " << test_cases[i].field_contents
+ << ", case_sensitive = " << test_cases[i].case_sensitive);
+
+ EXPECT_EQ(test_cases[i].expected_start,
+ GetTextSelectionStart(
+ base::ASCIIToUTF16(test_cases[i].field_suggestion),
+ base::ASCIIToUTF16(test_cases[i].field_contents),
+ test_cases[i].case_sensitive));
+ }
+}
+
+} // autofill

Powered by Google App Engine
This is Rietveld 408576698