Autofill: Remove fax number completely.

chrome/browser/autofill/phone_number.h

Index: chrome/browser/autofill/phone_number.h
diff --git a/chrome/browser/autofill/phone_number.h b/chrome/browser/autofill/phone_number.h
index 652cb709f8b348a1ac01117f59139b0c257ee04b..1eda750e3c559edb0e6d92831f487567f272e7f8 100644
--- a/chrome/browser/autofill/phone_number.h
+++ b/chrome/browser/autofill/phone_number.h
@@ -49,14 +49,14 @@ class PhoneNumber : public FormGroup {
   static const size_t kSuffixLength = 4;
 
   // The following functions should return the field type for each part of the
-  // phone number.  Currently, these are either fax or home phone number types.
+  // phone number.
   AutofillFieldType GetNumberType() const;
   AutofillFieldType GetCityCodeType() const;
   AutofillFieldType GetCountryCodeType() const;
   AutofillFieldType GetCityAndNumberType() const;
   AutofillFieldType GetWholeNumberType() const;

[Ilya Sherman, 2011/09/15 03:47:08]

Let's just get rid of these, as well as the whole HomePhoneNumber class.

[James Hawkins, 2011/09/16 03:23:28]

Done.

 
-  // The class used to combine home phone or fax parts into a whole number.
+  // The class used to combine home phone parts into a whole number.
   class PhoneCombineHelper {
    public:
     explicit PhoneCombineHelper(AutofillType::FieldTypeGroup phone_group);
@@ -96,7 +96,7 @@ class PhoneNumber : public FormGroup {
   // since the last time the cache was updated.
   void UpdateCacheIfNeeded() const;
 
-  // Phone group -  currently it is PHONE_HOME and PHONE_FAX.
+  // Phone group -  currently it is PHONE_HOME.
   AutofillType::FieldTypeGroup phone_group_;

[Ilya Sherman, 2011/09/15 03:47:08]

There's no reason to store the |phone_group_| here any more.

[James Hawkins, 2011/09/16 03:23:28]

Done.

   // The phone number.
   string16 number_;