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

Unified Diff: chrome/browser/resources/options/autofill_edit_address_overlay.js

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/resources/options/autofill_edit_address_overlay.js
diff --git a/chrome/browser/resources/options/autofill_edit_address_overlay.js b/chrome/browser/resources/options/autofill_edit_address_overlay.js
index 2994d77ecf88d62d376bd5548d787d181d9bd8ef..2cb1b639c84a4b058fab3704ea128a86bbc8085e 100644
--- a/chrome/browser/resources/options/autofill_edit_address_overlay.js
+++ b/chrome/browser/resources/options/autofill_edit_address_overlay.js
@@ -50,7 +50,7 @@ cr.define('options', function() {
/**
* Creates, decorates and initializes the multi-value lists for full name,
- * phone, fax, and email.
+ * phone, and email.
* @private
*/
createMultiValueLists_: function() {
@@ -62,10 +62,6 @@ cr.define('options', function() {
options.autofillOptions.AutofillPhoneValuesList.decorate(list);
list.autoExpands = true;
- list = $('fax-list');
- options.autofillOptions.AutofillFaxValuesList.decorate(list);
- list.autoExpands = true;
-
list = $('email-list');
options.autofillOptions.AutofillValuesList.decorate(list);
list.autoExpands = true;
@@ -123,10 +119,8 @@ cr.define('options', function() {
address[8] = $('country').value;
list = $('phone-list');
address[9] = list.dataModel.slice(0, list.dataModel.length - 1);
- list = $('fax-list');
- address[10] = list.dataModel.slice(0, list.dataModel.length - 1);
list = $('email-list');
- address[11] = list.dataModel.slice(0, list.dataModel.length - 1);
+ address[10] = list.dataModel.slice(0, list.dataModel.length - 1);
chrome.send('setAddress', address);
},
@@ -164,7 +158,7 @@ cr.define('options', function() {
!$('addr-line-1').value && !$('addr-line-2').value &&
!$('city').value && !$('state').value && !$('postal-code').value &&
!$('country').value && $('phone-list').items.length <= 1 &&
- $('fax-list').items.length <= 1 && $('email-list').items.length <= 1;
+ $('email-list').items.length <= 1;
$('autofill-edit-address-apply-button').disabled = disabled;
},
@@ -248,7 +242,6 @@ cr.define('options', function() {
$('postal-code').value = '';
$('country').value = '';
this.setMultiValueList_('phone-list', []);
- this.setMultiValueList_('fax-list', []);
this.setMultiValueList_('email-list', []);
this.countryChanged_();
@@ -279,7 +272,6 @@ cr.define('options', function() {
$('postal-code').value = address['postalCode'];
$('country').value = address['country'];
this.setMultiValueList_('phone-list', address['phone']);
- this.setMultiValueList_('fax-list', address['fax']);
this.setMultiValueList_('email-list', address['email']);
this.countryChanged_();
@@ -303,11 +295,6 @@ cr.define('options', function() {
numbers);
};
- AutofillEditAddressOverlay.setValidatedFaxNumbers = function(numbers) {
- AutofillEditAddressOverlay.getInstance().setMultiValueList_('fax-list',
- numbers);
- };
-
// Export
return {
AutofillEditAddressOverlay: AutofillEditAddressOverlay

Powered by Google App Engine
This is Rietveld 408576698