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

Issues Search
    My Issues | Starred     Open | Closed | All

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

Issue 2675503002: [MD Bookmarks] Remove Multiple Selected Items. (Closed)
Patch Set: add test Created 3 years, 11 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
« no previous file with comments | « no previous file | chrome/browser/resources/md_bookmarks/store.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 018b9c6d6114c0a618318ee5477fce73d9a6634b..999778591d9fcfbf8072a7b1a3f0831bb5ae6b6a 100644
--- a/chrome/browser/resources/md_bookmarks/list.js
+++ b/chrome/browser/resources/md_bookmarks/list.js
@@ -49,15 +49,7 @@ Polymer({
/** @private */
onDeleteTap_: function() {
- if (this.menuItem_.url) {
- chrome.bookmarks.remove(this.menuItem_.id, function() {
- // TODO(jiaxi): Add toast later.
- }.bind(this));
- } else {
- chrome.bookmarks.removeTree(this.menuItem_.id, function() {
- // TODO(jiaxi): Add toast later.
- }.bind(this));
- }
+ this.fire('remove-item', this.menuItem_);
this.closeDropdownMenu_();
},
« no previous file with comments | « no previous file | chrome/browser/resources/md_bookmarks/store.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698