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

Unified Diff: chrome/browser/resources/settings/on_startup_page/on_startup_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/on_startup_page/on_startup_browser_proxy.js
diff --git a/chrome/browser/resources/settings/on_startup_page/on_startup_browser_proxy.js b/chrome/browser/resources/settings/on_startup_page/on_startup_browser_proxy.js
index a99f6185b54e5ae5132999de1f2966ce4111d211..917029b832d712627e2be074132c0df9365514cd 100644
--- a/chrome/browser/resources/settings/on_startup_page/on_startup_browser_proxy.js
+++ b/chrome/browser/resources/settings/on_startup_page/on_startup_browser_proxy.js
@@ -7,26 +7,22 @@ var NtpExtension;
cr.define('settings', function() {
/** @interface */
- function OnStartupBrowserProxy() {}
-
- OnStartupBrowserProxy.prototype = {
+ class OnStartupBrowserProxy {
/** @return {!Promise<?NtpExtension>} */
- getNtpExtension: assertNotReached,
- };
+ getNtpExtension() {}
+ }
/**
- * @constructor
* @implements {settings.OnStartupBrowserProxy}
*/
- function OnStartupBrowserProxyImpl() {}
- cr.addSingletonGetter(OnStartupBrowserProxyImpl);
-
- OnStartupBrowserProxyImpl.prototype = {
+ class OnStartupBrowserProxyImpl {
/** @override */
- getNtpExtension: function() {
+ getNtpExtension() {
return cr.sendWithPromise('getNtpExtension');
- },
- };
+ }
+ }
+
+ cr.addSingletonGetter(OnStartupBrowserProxyImpl);
return {
OnStartupBrowserProxy: OnStartupBrowserProxy,

Powered by Google App Engine
This is Rietveld 408576698