Chromium Code Reviews

Unified Diff: components/autofill/core/browser/form_field_unittest.cc

Issue 1515553005: Label non-capturing groups if no capturing needed (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years ago
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
« components/autofill/core/browser/autofill_regex_constants.cc ('K') | « components/autofill/core/browser/autofill_regex_constants.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/autofill/core/browser/form_field_unittest.cc
diff --git a/components/autofill/core/browser/form_field_unittest.cc b/components/autofill/core/browser/form_field_unittest.cc
index aac94d07dfbd082bbaeb48b60d45f1a14323fee9..61ebaa3eb3b4edc7ae15c82c9c1e6e0b2e19cd8d 100644
--- a/components/autofill/core/browser/form_field_unittest.cc
+++ b/components/autofill/core/browser/form_field_unittest.cc
@@ -88,7 +88,7 @@ TEST(FormFieldTest, Match) {
EXPECT_TRUE(FormField::Match(&field, "head_tail", FormField::MATCH_LABEL));
// Word boundaries.
- const std::string kWordBoundary = "(\\A|\\z|\\PL)";
+ const std::string kWordBoundary = "(?:\\A|\\z|\\PL)";
field.label = ASCIIToUTF16("contains word:");
EXPECT_TRUE(FormField::Match(&field, kWordBoundary + "word" + kWordBoundary,
FormField::MATCH_LABEL));
« components/autofill/core/browser/autofill_regex_constants.cc ('K') | « components/autofill/core/browser/autofill_regex_constants.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine