Minor style cleanup in the PhoneField class:

Minor style cleanup in the PhoneField class:

* Inline ParseInternal() into Parse()
* Rename a constant to have kConstantNamingStyle
* Make GetRegExp a private static method (note that this can't be moved into the anonymous namespace since it depends on the private RegexType enum).
* Remove unnecessary PhoneField:: prefixes
* Adjust ordering of declarations and definitions within header and implementation files.

BUG=none


Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=177095

[Ilya Sherman, 2013-01-15 22:17:33]

https://chromiumcodereview.appspot.com/11889029/diff/1/chrome/browser/autofil...
File chrome/browser/autofill/phone_field.cc (right):

https://chromiumcodereview.appspot.com/11889029/diff/1/chrome/browser/autofil...
chrome/browser/autofill/phone_field.cc:194:
&phone_field->parsed_phone_fields_[FIELD_EXTENSION]);
Lines 125-194 were just copy/pasted, with the only change being "return false"
-> "return NULL".

[dhollowa, 2013-01-16 00:45:33]

lgtm

[commit-bot: I haz the power, 2013-01-16 00:52:38]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/isherman@chromium.org/11889029/1

[commit-bot: I haz the power, 2013-01-16 03:12:30]

Retried try job too often on win for step(s) compile

[commit-bot: I haz the power, 2013-01-16 04:12:23]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/isherman@chromium.org/11889029/1

[commit-bot: I haz the power, 2013-01-16 05:42:10]

Change committed as 177095