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

Unified Diff: chrome/browser/autofill/autofill_metrics.cc

Issue 7892048: Autofill: Remove fax number completely. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Comment fix. 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_metrics.cc
diff --git a/chrome/browser/autofill/autofill_metrics.cc b/chrome/browser/autofill/autofill_metrics.cc
index dae51a2816aba0818d0f2db60dd8dbf35823373d..af13ca4f1d451eeda5853ca5a3bd450e9a110cc4 100644
--- a/chrome/browser/autofill/autofill_metrics.cc
+++ b/chrome/browser/autofill/autofill_metrics.cc
@@ -42,7 +42,7 @@ enum FieldTypeGroupForMetrics {
ADDRESS_ZIP,
ADDRESS_COUNTRY,
PHONE,
- FAX,
+ FAX, // Deprecated.
EMAIL,
CREDIT_CARD_NAME,
CREDIT_CARD_NUMBER,
@@ -124,10 +124,6 @@ int GetFieldTypeGroupMetric(const AutofillFieldType field_type,
group = PHONE;
break;
- case AutofillType::PHONE_FAX:
- group = FAX;
- break;
-
case AutofillType::CREDIT_CARD:
switch (field_type) {
case ::CREDIT_CARD_NAME:
« no previous file with comments | « chrome/browser/autofill/autofill_merge_unittest.cc ('k') | chrome/browser/autofill/autofill_metrics_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698