Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1285)

Unified Diff: chrome/browser/autofill/autofill_manager.h

Issue 7892048: Autofill: Remove fax number completely. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/autofill/autofill_manager.h
diff --git a/chrome/browser/autofill/autofill_manager.h b/chrome/browser/autofill/autofill_manager.h
index 21ded1c5428d7a6bf56780ea68d565fbf887eb32..6a6b4b645949cb024a275f51d444636ba370d757 100644
--- a/chrome/browser/autofill/autofill_manager.h
+++ b/chrome/browser/autofill/autofill_manager.h
@@ -194,17 +194,17 @@ class AutofillManager : public TabContentsObserver,
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,
@@ -218,13 +218,13 @@ class AutofillManager : public TabContentsObserver,
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 c
James Hawkins 2011/09/16 03:23:28 Done. I could swear we made the style rule of one
+ // |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;
@@ -232,7 +232,7 @@ class AutofillManager : public TabContentsObserver,
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_;

Powered by Google App Engine
This is Rietveld 408576698