Autofill: Remove fax number completely.

chrome/browser/autofill/autofill_manager.h

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_AUTOFILL_AUTOFILL_MANAGER_H_
 #define CHROME_BROWSER_AUTOFILL_AUTOFILL_MANAGER_H_
 #pragma once
 
 #include <list>
 #include <map>
@@ skipping 176 matching lines @@
                                 std::vector<string16>* labels,
                                 std::vector<string16>* icons,
                                 std::vector<int>* unique_ids) const;
 
   // Set |field|'s value based on |type| and contents of the |credit_card|.
   void FillCreditCardFormField(const CreditCard& credit_card,
                                AutofillFieldType type,
                                webkit_glue::FormField* field);
 
   // Set |field|'s value based on |cached_field|'s type and contents of the
-  // |profile|.  The |variant| parameter specifies which value in a multi-valued
+  // |profile|. The |variant| parameter specifies which value in a multi-valued
   // profile.
   void FillFormField(const AutofillProfile& profile,
                      const AutofillField& cached_field,
                      size_t variant,
                      webkit_glue::FormField* field);
 
-  // Set |field|'s value for phone/fax number based on contents of the
-  // |profile|.  The |cached_field| specifies the type of the phone and whether
-  // this is a phone prefix or suffix.  The |variant| parameter specifies which
-  // value in a multi-valued profile.
+  // Set |field|'s value for phone number based on contents of the |profile|.
+  // The |cached_field| specifies the type of the phone and whether this is a
+  // phone prefix or suffix.  The |variant| parameter specifies which value in a
+  // multi-valued profile.
   void FillPhoneNumberField(const AutofillProfile& profile,
                             const AutofillField& cached_field,
                             size_t variant,
                             webkit_glue::FormField* field);
 
   // Parses the forms using heuristic matching and querying the Autofill server.
   void ParseForms(const std::vector<webkit_glue::FormData>& forms);
 
   // Uses existing personal data to determine possible field types for the
   // |submitted_form|.
   void DeterminePossibleFieldTypesForUpload(FormStructure* submitted_form);
 
   // If |initial_interaction_timestamp_| is unset or is set to a later time than
-  // |interaction_timestamp|, updates the cached timestamp.  The latter check is

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

nit: Please preserve two spaces after a sentence closing period in Autofill code
(throughout).

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

Done.

I could swear we made the style rule of one space, but it turns out it hasn't
been codified.

+  // |interaction_timestamp|, updates the cached timestamp. The latter check is
   // needed because IPC messages can arrive out of order.
   void UpdateInitialInteractionTimestamp(
       const base::TimeTicks& interaction_timestamp);
 
-  // Send our current field type predictions to the renderer.
-  // This is a no-op if the appropriate command-line flag is not set.
+  // Send our current field type predictions to the renderer. This is a no-op if
+  // the appropriate command-line flag is not set.
   void SendAutofillTypePredictions(
       const std::vector<FormStructure*>& forms) const;
 
   // The owning TabContentsWrapper.
   TabContentsWrapper* tab_contents_wrapper_;
 
   // The personal data manager, used to save and load personal data to/from the
-  // web database.  This is overridden by the AutofillManagerTest.
+  // web database. This is overridden by the AutofillManagerTest.
   // Weak reference.
   // May be NULL.  NULL indicates OTR.
   PersonalDataManager* personal_data_;
 
   std::list<std::string> autofilled_form_signatures_;
   // Handles queries and uploads to Autofill servers.
   AutofillDownloadManager download_manager_;
 
   // Should be set to true in AutofillManagerTest and other tests, false in
   // AutofillDownloadManagerTest and in non-test environment. Is false by
@@ skipping 64 matching lines @@
   FRIEND_TEST_ALL_PREFIXES(AutofillMetricsTest, SaneMetricsWithCacheMismatch);
   FRIEND_TEST_ALL_PREFIXES(AutofillMetricsTest,
                            UserHappinessFormLoadAndSubmission);
   FRIEND_TEST_ALL_PREFIXES(AutofillMetricsTest, UserHappinessFormInteraction);
   FRIEND_TEST_ALL_PREFIXES(AutofillMetricsTest, FormFillDuration);
 
   DISALLOW_COPY_AND_ASSIGN(AutofillManager);
 };
 
 #endif  // CHROME_BROWSER_AUTOFILL_AUTOFILL_MANAGER_H_