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

Unified Diff: chrome/browser/resources/md_bookmarks/list.js

Issue 2929053004: MD Bookmarks: Make shortcuts for select all/deselect all global (Closed)
Patch Set: Fix test flub 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_bookmarks/list.js
diff --git a/chrome/browser/resources/md_bookmarks/list.js b/chrome/browser/resources/md_bookmarks/list.js
index 0505d470682b99a8874e72f7f2e9c3b8b0c9f86a..c14fee3a875d948df19b170085da4ecbf1dce45f 100644
--- a/chrome/browser/resources/md_bookmarks/list.js
+++ b/chrome/browser/resources/md_bookmarks/list.js
@@ -123,12 +123,6 @@ Polymer({
this.dispatch(bookmarks.actions.deselectItems());
},
- /** @private */
- selectAllItems_: function() {
- this.dispatch(
- bookmarks.actions.selectAll(this.displayedIds_, this.getState()));
- },
-
/**
* @param{HTMLElement} el
* @private
@@ -168,10 +162,6 @@ Polymer({
range: false,
toggle: true,
}));
- } else if (e.key == 'a' && e.ctrlKey) {
- this.selectAllItems_();
- } else if (e.key == 'Escape') {
- this.deselectItems_();
} else {
handled = false;
}
« no previous file with comments | « chrome/browser/resources/md_bookmarks/constants.js ('k') | chrome/test/data/webui/md_bookmarks/md_bookmarks_focus_test.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698