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

Unified Diff: chrome/browser/resources/settings/passwords_and_forms_page/passwords_and_forms_page.js

Issue 2015463003: Add dialog to edit and save credit cards. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 6 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/settings/passwords_and_forms_page/passwords_and_forms_page.js
diff --git a/chrome/browser/resources/settings/passwords_and_forms_page/passwords_and_forms_page.js b/chrome/browser/resources/settings/passwords_and_forms_page/passwords_and_forms_page.js
index bc1045dcff8c8bb1538d4a412ed87c73e73ddb97..a932c48b5f2e4a97d54ac7f2f501f5f40e18565a 100644
--- a/chrome/browser/resources/settings/passwords_and_forms_page/passwords_and_forms_page.js
+++ b/chrome/browser/resources/settings/passwords_and_forms_page/passwords_and_forms_page.js
@@ -71,7 +71,7 @@ PasswordManager.prototype = {
/**
* Should remove the password exception and notify that the list has changed.
- * @param {!string} exception The exception that should be removed from the
+ * @param {string} exception The exception that should be removed from the
* list. No-op if |exception| is not in the list.
*/
removeException: assertNotReached,
@@ -142,6 +142,12 @@ AutofillManager.prototype = {
* remove.
*/
removeCreditCard: assertNotReached,
+
+ /**
+ * Saves the given credit card.
+ * @param {!AutofillManager.CreditCardEntry} creditCard
+ */
+ saveCreditCard: assertNotReached,
};
/**
@@ -264,6 +270,11 @@ AutofillManagerImpl.prototype = {
removeCreditCard: function(creditCard) {
chrome.autofillPrivate.removeEntry(/** @type {string} */(creditCard.guid));
},
+
+ /** @override */
+ saveCreditCard: function(creditCard) {
+ chrome.autofillPrivate.saveCreditCard(creditCard);
+ }
};
(function() {
@@ -293,33 +304,25 @@ Polymer({
* An array of passwords to display.
* @type {!Array<!PasswordManager.PasswordUiEntry>}
*/
- savedPasswords: {
- type: Array,
- },
+ savedPasswords: Array,
/**
* An array of sites to display.
* @type {!Array<!PasswordManager.ExceptionPair>}
*/
- passwordExceptions: {
- type: Array,
- },
+ passwordExceptions: Array,
/**
* An array of saved addresses.
* @type {!Array<!AutofillManager.AddressEntry>}
*/
- addresses: {
- type: Array,
- },
+ addresses: Array,
/**
* An array of saved addresses.
* @type {!Array<!AutofillManager.CreditCardEntry>}
*/
- creditCards: {
- type: Array,
- },
+ creditCards: Array,
},
listeners: {
@@ -327,6 +330,7 @@ Polymer({
'remove-credit-card': 'removeCreditCard_',
'remove-password-exception': 'removePasswordException_',
'remove-saved-password': 'removeSavedPassword_',
+ 'save-credit-card': 'saveCreditCard_',
'show-password': 'showPassword_',
},
@@ -458,6 +462,15 @@ Polymer({
},
/**
+ * Listens for the save-credit-card event, and calls the private API.
+ * @param {!Event} event
+ * @private
+ */
+ saveCreditCard_: function(event) {
+ this.autofillManager_.saveCreditCard(event.detail);
+ },
+
+ /**
* Listens for the show-password event, and calls the private API.
* @param {!Event} event
* @private

Powered by Google App Engine
This is Rietveld 408576698