Autofill: Apply the same field match mask to all phone field subtypes.

Autofill: Apply the same field match mask to all phone field subtypes.

- Modify unit tests to test relevant field types.
- Cleanup code.
- Update heuristics test expectations.

Committed: https://crrev.com/70c9d025b846e35d60d6581e9b2a39428796535b
Cr-Commit-Position: refs/heads/master@{#322312}

[Lei Zhang, 2015-03-26 00:20:26]

Patchset #1 (id:1) has been deleted

[Lei Zhang, 2015-03-26 02:07:53]

Patchset #1 (id:20001) has been deleted

[Lei Zhang, 2015-03-26 02:09:16]

https://codereview.chromium.org/1010743007/diff/40001/components/autofill/cor...
File components/autofill/core/browser/phone_field_unittest.cc (right):

https://codereview.chromium.org/1010743007/diff/40001/components/autofill/cor...
components/autofill/core/browser/phone_field_unittest.cc:22: "tel",
FYI, I intend to add "number" in a follow up CL.

[Evan Stade, 2015-03-26 02:13:12]

estade@chromium.org changed reviewers:
+ estade@chromium.org

[Evan Stade, 2015-03-26 02:13:13]

I cannot parse this sentence: "Make PhoneField parse fields with the same match
type."

[Lei Zhang, 2015-03-26 02:14:54]

On 2015/03/26 02:13:13, Evan Stade wrote:
> I cannot parse this sentence: "Make PhoneField parse fields with the same
match
> type."

is "Make PhoneField parse fields using the same field match type" better?

[Evan Stade, 2015-03-26 02:30:45]

On 2015/03/26 02:14:54, Lei Zhang wrote:
> On 2015/03/26 02:13:13, Evan Stade wrote:
> > I cannot parse this sentence: "Make PhoneField parse fields with the same
> match
> > type."
> 
> is "Make PhoneField parse fields using the same field match type" better?

Apply same field match mask to all phone field subtypes.

[Evan Stade, 2015-03-26 02:32:01]

now that i understand that, lgtm

[Lei Zhang, 2015-03-26 02:32:57]

On 2015/03/26 02:30:45, Evan Stade wrote:
> Apply same field match mask to all phone field subtypes.

Done.

[Lei Zhang, 2015-03-26 02:33:00]

The CQ bit was checked by thestig@chromium.org

[commit-bot: I haz the power, 2015-03-26 02:33:47]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1010743007/40001

[commit-bot: I haz the power, 2015-03-26 03:54:15]

Committed patchset #1 (id:40001)

[commit-bot: I haz the power, 2015-03-26 03:55:04]

Patchset 1 (id:??) landed as
https://crrev.com/70c9d025b846e35d60d6581e9b2a39428796535b
Cr-Commit-Position: refs/heads/master@{#322312}