Revert of autofill: switch autofill_regexes to RE2 library

components/autofill/core/browser/address_field.cc

Index: components/autofill/core/browser/address_field.cc
diff --git a/components/autofill/core/browser/address_field.cc b/components/autofill/core/browser/address_field.cc
index b9948dbcdda7a76b668afd1348e20114cccf44cb..bf96dc2e75860ca64d95a8be1e20ca9be5360e7a 100644
--- a/components/autofill/core/browser/address_field.cc
+++ b/components/autofill/core/browser/address_field.cc
@@ -8,12 +8,16 @@
 
 #include "base/logging.h"
 #include "base/memory/scoped_ptr.h"
+#include "base/strings/string16.h"
 #include "base/strings/string_util.h"
+#include "base/strings/utf_string_conversions.h"
 #include "components/autofill/core/browser/autofill_field.h"
 #include "components/autofill/core/browser/autofill_regex_constants.h"
 #include "components/autofill/core/browser/autofill_scanner.h"
 #include "components/autofill/core/browser/field_types.h"
 
+using base::UTF8ToUTF16;
+
 namespace autofill {
 
 namespace {
@@ -43,6 +47,9 @@
   scoped_ptr<AddressField> address_field(new AddressField);
   const AutofillField* const initial_field = scanner->Cursor();
   size_t saved_cursor = scanner->SaveCursor();
+
+  base::string16 attention_ignored = UTF8ToUTF16(kAttentionIgnoredRe);
+  base::string16 region_ignored = UTF8ToUTF16(kRegionIgnoredRe);
 
   // Allow address fields to appear in any order.
   size_t begin_trailing_non_labeled_fields = 0;
@@ -55,8 +62,8 @@
         address_field->ParseCompany(scanner)) {
       has_trailing_non_labeled_fields = false;
       continue;
-    } else if (ParseField(scanner, kAttentionIgnoredRe, NULL) ||
-               ParseField(scanner, kRegionIgnoredRe, NULL)) {
+    } else if (ParseField(scanner, attention_ignored, NULL) ||
+               ParseField(scanner, region_ignored, NULL)) {
       // We ignore the following:
       // * Attention.
       // * Province/Region/Other.
@@ -141,7 +148,7 @@
   if (company_ && !company_->IsEmpty())
     return false;
 
-  return ParseField(scanner, kCompanyRe, &company_);
+  return ParseField(scanner, UTF8ToUTF16(kCompanyRe), &company_);
 }
 
 bool AddressField::ParseAddressLines(AutofillScanner* scanner) {
@@ -157,17 +164,19 @@
     return false;
 
   // Ignore "Address Lookup" field. http://crbug.com/427622
-  if (ParseField(scanner, kAddressLookupRe, NULL))
-    return false;
-
-  if (!ParseFieldSpecifics(scanner, kAddressLine1Re, MATCH_DEFAULT,
+  if (ParseField(scanner, base::UTF8ToUTF16(kAddressLookupRe), NULL))
+    return false;
+
+  base::string16 pattern = UTF8ToUTF16(kAddressLine1Re);
+  base::string16 label_pattern = UTF8ToUTF16(kAddressLine1LabelRe);
+  if (!ParseFieldSpecifics(scanner, pattern, MATCH_DEFAULT, &address1_) &&
+      !ParseFieldSpecifics(scanner, label_pattern, MATCH_LABEL | MATCH_TEXT,
                            &address1_) &&
-      !ParseFieldSpecifics(scanner, kAddressLine1LabelRe,
-                           MATCH_LABEL | MATCH_TEXT, &address1_) &&
-      !ParseFieldSpecifics(scanner, kAddressLine1Re,
-                           MATCH_DEFAULT | MATCH_TEXT_AREA, &street_address_) &&
-      !ParseFieldSpecifics(scanner, kAddressLine1LabelRe,
-                           MATCH_LABEL | MATCH_TEXT_AREA, &street_address_))
+      !ParseFieldSpecifics(scanner, pattern, MATCH_DEFAULT | MATCH_TEXT_AREA,
+                           &street_address_) &&
+      !ParseFieldSpecifics(scanner, label_pattern,
+                           MATCH_LABEL | MATCH_TEXT_AREA,
+                           &street_address_))
     return false;
 
   if (street_address_)
@@ -176,16 +185,19 @@
   // This code may not pick up pages that have an address field consisting of a
   // sequence of unlabeled address fields. If we need to add this, see
   // discussion on https://codereview.chromium.org/741493003/
-  if (!ParseField(scanner, kAddressLine2Re, &address2_) &&
-      !ParseFieldSpecifics(scanner, kAddressLine2LabelRe,
-                           MATCH_LABEL | MATCH_TEXT, &address2_))
+  pattern = UTF8ToUTF16(kAddressLine2Re);
+  label_pattern = UTF8ToUTF16(kAddressLine2LabelRe);
+  if (!ParseField(scanner, pattern, &address2_) &&
+      !ParseFieldSpecifics(scanner, label_pattern, MATCH_LABEL | MATCH_TEXT,
+                           &address2_))
     return true;
 
   // Optionally parse address line 3. This uses the same label regexp as
   // address 2 above.
-  if (!ParseField(scanner, kAddressLinesExtraRe, &address3_) &&
-      !ParseFieldSpecifics(scanner, kAddressLine2LabelRe,
-                           MATCH_LABEL | MATCH_TEXT, &address3_))
+  pattern = UTF8ToUTF16(kAddressLinesExtraRe);
+  if (!ParseField(scanner, pattern, &address3_) &&
+      !ParseFieldSpecifics(scanner, label_pattern, MATCH_LABEL | MATCH_TEXT,
+                           &address3_))
     return true;
 
   // Try for surplus lines, which we will promptly discard. Some pages have 4
@@ -193,7 +205,8 @@
   //
   // Since these are rare, don't bother considering unlabeled lines as extra
   // address lines.
-  while (ParseField(scanner, kAddressLinesExtraRe, NULL)) {
+  pattern = UTF8ToUTF16(kAddressLinesExtraRe);
+  while (ParseField(scanner, pattern, NULL)) {
     // Consumed a surplus line, try for another.
   }
   return true;
@@ -204,7 +217,9 @@
     return false;
 
   scanner->SaveCursor();
-  if (ParseFieldSpecifics(scanner, kCountryRe, MATCH_DEFAULT | MATCH_SELECT,
+  if (ParseFieldSpecifics(scanner,
+                          UTF8ToUTF16(kCountryRe),
+                          MATCH_DEFAULT | MATCH_SELECT,
                           &country_)) {
     return true;
   }
@@ -212,7 +227,8 @@
   // The occasional page (e.g. google account registration page) calls this a
   // "location". However, this only makes sense for select tags.
   scanner->Rewind();
-  return ParseFieldSpecifics(scanner, kCountryLocationRe,
+  return ParseFieldSpecifics(scanner,
+                             UTF8ToUTF16(kCountryLocationRe),
                              MATCH_LABEL | MATCH_NAME | MATCH_SELECT,
                              &country_);
 }
@@ -221,13 +237,16 @@
   if (zip_)
     return false;
 
-  if (!ParseFieldSpecifics(scanner, kZipCodeRe, kZipCodeMatchType, &zip_)) {
+  if (!ParseFieldSpecifics(scanner,
+                           UTF8ToUTF16(kZipCodeRe),
+                           kZipCodeMatchType,
+                           &zip_)) {
     return false;
   }
 
   // Look for a zip+4, whose field name will also often contain
   // the substring "zip".
-  ParseFieldSpecifics(scanner, kZip4Re, kZipCodeMatchType, &zip4_);
+  ParseFieldSpecifics(scanner, UTF8ToUTF16(kZip4Re), kZipCodeMatchType, &zip4_);
   return true;
 }
 
@@ -235,21 +254,20 @@
   if (city_)
     return false;
 
-  return ParseFieldSpecifics(scanner, kCityRe, kCityMatchType, &city_);
+  return ParseFieldSpecifics(scanner,
+                             UTF8ToUTF16(kCityRe),
+                             kCityMatchType,
+                             &city_);
 }
 
 bool AddressField::ParseState(AutofillScanner* scanner) {
   if (state_)
     return false;
 
-  // Ignore spurious matches for "United States".
-  size_t saved_cursor = scanner->SaveCursor();
-  if (ParseFieldSpecifics(scanner, "United States", kStateMatchType, nullptr)) {
-    scanner->RewindTo(saved_cursor);
-    return false;
-  }
-
-  return ParseFieldSpecifics(scanner, kStateRe, kStateMatchType, &state_);
+  return ParseFieldSpecifics(scanner,
+                             UTF8ToUTF16(kStateRe),
+                             kStateMatchType,
+                             &state_);
 }
 
 bool AddressField::ParseCityStateZipCode(AutofillScanner* scanner) {
@@ -311,7 +329,7 @@
     return RESULT_MATCH_NONE;
 
   ParseNameLabelResult result = ParseNameAndLabelSeparately(
-      scanner, kZipCodeRe, kZipCodeMatchType, &zip_);
+      scanner, UTF8ToUTF16(kZipCodeRe), kZipCodeMatchType, &zip_);
 
   if (result != RESULT_MATCH_NAME_LABEL || scanner->IsEnd())
     return result;
@@ -331,7 +349,10 @@
   if (!found_non_zip4) {
     // Look for a zip+4, whose field name will also often contain
     // the substring "zip".
-    ParseFieldSpecifics(scanner, kZip4Re, kZipCodeMatchType, &zip4_);
+    ParseFieldSpecifics(scanner,
+                        UTF8ToUTF16(kZip4Re),
+                        kZipCodeMatchType,
+                        &zip4_);
   }
   return result;
 }
@@ -341,7 +362,8 @@
   if (city_)
     return RESULT_MATCH_NONE;
 
-  return ParseNameAndLabelSeparately(scanner, kCityRe, kCityMatchType, &city_);
+  return ParseNameAndLabelSeparately(
+      scanner, UTF8ToUTF16(kCityRe), kCityMatchType, &city_);
 }
 
 AddressField::ParseNameLabelResult AddressField::ParseNameAndLabelForState(
@@ -349,14 +371,8 @@
   if (state_)
     return RESULT_MATCH_NONE;
 
-  size_t saved_cursor = scanner->SaveCursor();
-  if (ParseFieldSpecifics(scanner, "United States", kStateMatchType, nullptr)) {
-    scanner->RewindTo(saved_cursor);
-    return RESULT_MATCH_NONE;
-  }
-
-  return ParseNameAndLabelSeparately(scanner, kStateRe, kStateMatchType,
-                                     &state_);
+  return ParseNameAndLabelSeparately(
+      scanner, UTF8ToUTF16(kStateRe), kStateMatchType, &state_);
 }
 
 }  // namespace autofill