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

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

Issue 2965643004: MD Settings: Convert remaining classes to ES6 syntax. (Closed)
Patch Set: Fix Created 3 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/autofill_section.js
diff --git a/chrome/browser/resources/settings/passwords_and_forms_page/autofill_section.js b/chrome/browser/resources/settings/passwords_and_forms_page/autofill_section.js
index 0a7dff75c10fa0789dc77cf144e69e86a8d70264..76f7afaece43e0d2e75d8d2399477ab2e574dfe7 100644
--- a/chrome/browser/resources/settings/passwords_and_forms_page/autofill_section.js
+++ b/chrome/browser/resources/settings/passwords_and_forms_page/autofill_section.js
@@ -11,139 +11,133 @@
* Interface for all callbacks to the autofill API.
* @interface
*/
-function AutofillManager() {}
-
-/** @typedef {chrome.autofillPrivate.AddressEntry} */
-AutofillManager.AddressEntry;
-
-/** @typedef {chrome.autofillPrivate.CreditCardEntry} */
-AutofillManager.CreditCardEntry;
-
-AutofillManager.prototype = {
+class AutofillManager {
/**
* Add an observer to the list of addresses.
* @param {function(!Array<!AutofillManager.AddressEntry>):void} listener
*/
- addAddressListChangedListener: assertNotReached,
+ addAddressListChangedListener(listener) {}
/**
* Remove an observer from the list of addresses.
* @param {function(!Array<!AutofillManager.AddressEntry>):void} listener
*/
- removeAddressListChangedListener: assertNotReached,
+ removeAddressListChangedListener(listener) {}
/**
* Request the list of addresses.
* @param {function(!Array<!AutofillManager.AddressEntry>):void} callback
*/
- getAddressList: assertNotReached,
+ getAddressList(callback) {}
/**
* Saves the given address.
* @param {!AutofillManager.AddressEntry} address
*/
- saveAddress: assertNotReached,
+ saveAddress(address) {}
/** @param {string} guid The guid of the address to remove. */
- removeAddress: assertNotReached,
+ removeAddress(guid) {}
/**
* Add an observer to the list of credit cards.
* @param {function(!Array<!AutofillManager.CreditCardEntry>):void} listener
*/
- addCreditCardListChangedListener: assertNotReached,
+ addCreditCardListChangedListener(listener) {}
/**
* Remove an observer from the list of credit cards.
* @param {function(!Array<!AutofillManager.CreditCardEntry>):void} listener
*/
- removeCreditCardListChangedListener: assertNotReached,
+ removeCreditCardListChangedListener(listener) {}
/**
* Request the list of credit cards.
* @param {function(!Array<!AutofillManager.CreditCardEntry>):void} callback
*/
- getCreditCardList: assertNotReached,
+ getCreditCardList(callback) {}
/** @param {string} guid The GUID of the credit card to remove. */
- removeCreditCard: assertNotReached,
+ removeCreditCard(guid) {}
/** @param {string} guid The GUID to credit card to remove from the cache. */
- clearCachedCreditCard: assertNotReached,
+ clearCachedCreditCard(guid) {}
/**
* Saves the given credit card.
* @param {!AutofillManager.CreditCardEntry} creditCard
*/
- saveCreditCard: assertNotReached,
-};
+ saveCreditCard(creditCard) {}
+}
+
+/** @typedef {chrome.autofillPrivate.AddressEntry} */
+AutofillManager.AddressEntry;
+
+/** @typedef {chrome.autofillPrivate.CreditCardEntry} */
+AutofillManager.CreditCardEntry;
/**
* Implementation that accesses the private API.
* @implements {AutofillManager}
- * @constructor
*/
-function AutofillManagerImpl() {}
-cr.addSingletonGetter(AutofillManagerImpl);
-
-AutofillManagerImpl.prototype = {
- __proto__: AutofillManager,
-
+class AutofillManagerImpl {
/** @override */
- addAddressListChangedListener: function(listener) {
+ addAddressListChangedListener(listener) {
chrome.autofillPrivate.onAddressListChanged.addListener(listener);
- },
+ }
/** @override */
- removeAddressListChangedListener: function(listener) {
+ removeAddressListChangedListener(listener) {
chrome.autofillPrivate.onAddressListChanged.removeListener(listener);
- },
+ }
/** @override */
- getAddressList: function(callback) {
+ getAddressList(callback) {
chrome.autofillPrivate.getAddressList(callback);
- },
+ }
/** @override */
- saveAddress: function(address) {
+ saveAddress(address) {
chrome.autofillPrivate.saveAddress(address);
- },
+ }
/** @override */
- removeAddress: function(guid) {
+ removeAddress(guid) {
chrome.autofillPrivate.removeEntry(assert(guid));
- },
+ }
/** @override */
- addCreditCardListChangedListener: function(listener) {
+ addCreditCardListChangedListener(listener) {
chrome.autofillPrivate.onCreditCardListChanged.addListener(listener);
- },
+ }
/** @override */
- removeCreditCardListChangedListener: function(listener) {
+ removeCreditCardListChangedListener(listener) {
chrome.autofillPrivate.onCreditCardListChanged.removeListener(listener);
- },
+ }
/** @override */
- getCreditCardList: function(callback) {
+ getCreditCardList(callback) {
chrome.autofillPrivate.getCreditCardList(callback);
- },
+ }
/** @override */
- removeCreditCard: function(guid) {
+ removeCreditCard(guid) {
chrome.autofillPrivate.removeEntry(assert(guid));
- },
+ }
/** @override */
- clearCachedCreditCard: function(guid) {
+ clearCachedCreditCard(guid) {
chrome.autofillPrivate.maskCreditCard(assert(guid));
- },
+ }
/** @override */
- saveCreditCard: function(creditCard) {
+ saveCreditCard(creditCard) {
chrome.autofillPrivate.saveCreditCard(creditCard);
}
-};
+}
+
+cr.addSingletonGetter(AutofillManagerImpl);
(function() {
'use strict';

Powered by Google App Engine
This is Rietveld 408576698