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

Unified Diff: chrome/browser/resources/settings/clear_browsing_data_dialog/clear_browsing_data_browser_proxy.js

Issue 2954863003: MD Settings: Convert all browser proxies to use ES6 class syntax. (Closed)
Patch Set: Remove @constructor annotations. 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/clear_browsing_data_dialog/clear_browsing_data_browser_proxy.js
diff --git a/chrome/browser/resources/settings/clear_browsing_data_dialog/clear_browsing_data_browser_proxy.js b/chrome/browser/resources/settings/clear_browsing_data_dialog/clear_browsing_data_browser_proxy.js
index 20e2ffe83a9c5016cfbc9ba854ffdd147451a671..24bdd3640095c68c4cf70ebf188b05cf23772ea8 100644
--- a/chrome/browser/resources/settings/clear_browsing_data_dialog/clear_browsing_data_browser_proxy.js
+++ b/chrome/browser/resources/settings/clear_browsing_data_dialog/clear_browsing_data_browser_proxy.js
@@ -26,52 +26,48 @@ var ImportantSite;
cr.define('settings', function() {
/** @interface */
- function ClearBrowsingDataBrowserProxy() {}
-
- ClearBrowsingDataBrowserProxy.prototype = {
+ class ClearBrowsingDataBrowserProxy {
/**
* @param {!Array<!ImportantSite>} importantSites
* @return {!Promise<void>}
* A promise resolved when data clearing has completed.
*/
- clearBrowsingData: function(importantSites) {},
+ clearBrowsingData(importantSites) {}
/**
* @return {!Promise<!Array<!ImportantSite>>}
* A promise resolved when imporant sites are retrieved.
*/
- getImportantSites: function() {},
+ getImportantSites() {}
/**
* Kick off counter updates and return initial state.
* @return {!Promise<void>} Signal when the setup is complete.
*/
- initialize: function() {},
- };
+ initialize() {}
+ }
/**
- * @constructor
* @implements {settings.ClearBrowsingDataBrowserProxy}
*/
- function ClearBrowsingDataBrowserProxyImpl() {}
- cr.addSingletonGetter(ClearBrowsingDataBrowserProxyImpl);
-
- ClearBrowsingDataBrowserProxyImpl.prototype = {
+ class ClearBrowsingDataBrowserProxyImpl {
/** @override */
- clearBrowsingData: function(importantSites) {
+ clearBrowsingData(importantSites) {
return cr.sendWithPromise('clearBrowsingData', importantSites);
- },
+ }
/** @override */
- getImportantSites: function() {
+ getImportantSites() {
return cr.sendWithPromise('getImportantSites');
- },
+ }
/** @override */
- initialize: function() {
+ initialize() {
return cr.sendWithPromise('initializeClearBrowsingData');
- },
- };
+ }
+ }
+
+ cr.addSingletonGetter(ClearBrowsingDataBrowserProxyImpl);
return {
ClearBrowsingDataBrowserProxy: ClearBrowsingDataBrowserProxy,

Powered by Google App Engine
This is Rietveld 408576698