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

Unified Diff: chrome/browser/resources/md_extensions/manager.js

Issue 2960293002: MD Extensions: Convert classes to use ES6 class syntax. (Closed)
Patch Set: 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/md_extensions/manager.js
diff --git a/chrome/browser/resources/md_extensions/manager.js b/chrome/browser/resources/md_extensions/manager.js
index f4070d37b73b4b8164fb36dfed76b601e67e19fe..4bac6d83e6b8bb722028adb6af38bd5418a394b3 100644
--- a/chrome/browser/resources/md_extensions/manager.js
+++ b/chrome/browser/resources/md_extensions/manager.js
@@ -422,18 +422,15 @@ cr.define('extensions', function() {
}
});
- /**
- * @param {extensions.Manager} manager
- * @constructor
- * @implements {extensions.SidebarListDelegate}
- */
- function ListHelper(manager) {
- this.manager_ = manager;
- }
+ /** @implements {extensions.SidebarListDelegate} */
+ class ListHelper {
+ /** @param {extensions.Manager} manager */
+ constructor(manager) {
+ this.manager_ = manager;
+ }
- ListHelper.prototype = {
/** @override */
- showType: function(type) {
+ showType(type) {
var items;
switch (type) {
case extensions.ShowingType.EXTENSIONS:
@@ -446,13 +443,13 @@ cr.define('extensions', function() {
this.manager_.$ /* hack */['items-list'].set('items', assert(items));
this.manager_.changePage({page: Page.LIST});
- },
+ }
/** @override */
- showKeyboardShortcuts: function() {
+ showKeyboardShortcuts() {
this.manager_.changePage({page: Page.SHORTCUTS});
- },
- };
+ }
+ }
return {Manager: Manager};
});

Powered by Google App Engine
This is Rietveld 408576698