Autofill: Recognize phone numbers in input fields of type number.

Improve PhoneFieldTest to test this. Review URL: https://codereview.chromium.org/1031923003 Cr-Commit-Position: refs/heads/master@{#322446}

Autofill: Recognize phone numbers in input fields of type number.
Improve PhoneFieldTest to test this. Review URL: https://codereview.chromium.org/1031923003 Cr-Commit-Position: refs/heads/master@{#322446}
70deb4f4 · thestig · Commit bot · 1275e815 · 70deb4f4 · 70deb4f4
Commit 70deb4f4 authored Mar 26, 2015 by thestig Committed by Commit bot Mar 26, 2015
3 changed files
--- a/chrome/test/data/autofill/heuristics/output/bug_469012.out
+++ b/chrome/test/data/autofill/heuristics/output/bug_469012.out
@@ -11,4 +11,4 @@ ADDRESS_HOME_CITY | city | City * |  | cardissuer_1-default
 ADDRESS_HOME_STATE | state | State * |  | cardissuer_1-default
 ADDRESS_HOME_ZIP | zip | State * |  | cardissuer_1-default
 ADDRESS_HOME_COUNTRY | country | Country * | us | cardissuer_1-default
-UNKNOWN_TYPE | phonenumber | Mobile Phone* |  | cardissuer_1-default
+PHONE_HOME_WHOLE_NUMBER | phonenumber | Mobile Phone* |  | cardissuer_1-default
--- a/components/autofill/core/browser/phone_field.cc
+++ b/components/autofill/core/browser/phone_field.cc
@@ -278,7 +278,7 @@ bool PhoneField::ParsePhoneField(AutofillScanner* scanner,
                                 AutofillField** field) {
  return ParseFieldSpecifics(scanner,
                             base::UTF8ToUTF16(regex),
-                             MATCH_DEFAULT | MATCH_TELEPHONE,
+                             MATCH_DEFAULT | MATCH_TELEPHONE | MATCH_NUMBER,
                             field);
 }

--- a/components/autofill/core/browser/phone_field_unittest.cc
+++ b/components/autofill/core/browser/phone_field_unittest.cc
@@ -20,6 +20,7 @@ namespace {
 const char* const kFieldTypes[] = {
  "text",
  "tel",
+  "number",
 };
 }  // namespace