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

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

Issue 2613683002: [MD Bookmarks] Add Delete and Copy URL for Material Bookmarks. (Closed)
Patch Set: fix a merge problem 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 | « chrome/browser/resources/md_bookmarks/item.js ('k') | 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 426cbb33be433e5dafe803e9ac440ab97314edc7..ea6cb45539e6daf35c7259a9195f658457b086b7 100644
--- a/chrome/browser/resources/md_bookmarks/list.js
+++ b/chrome/browser/resources/md_bookmarks/list.js
@@ -8,17 +8,21 @@ Polymer({
properties: {
/** @type {BookmarkTreeNode} */
selectedNode: Object,
+
+ /** @type {BookmarkTreeNode} */
+ menuItem_: Object,
},
listeners: {
- 'toggle-menu': 'onToggleMenu_'
+ 'open-item-menu': 'onOpenItemMenu_',
},
/**
* @param {Event} e
* @private
*/
- onToggleMenu_: function(e) {
+ onOpenItemMenu_: function(e) {
+ this.menuItem_ = e.detail.item;
var menu = /** @type {!CrActionMenuElement} */ (
this.$.dropdown);
menu.showAt(/** @type {!Element} */ (e.detail.target));
@@ -32,11 +36,24 @@ Polymer({
/** @private */
onCopyURLTap_: function() {
+ var idList = [this.menuItem_.id];
+ chrome.bookmarkManagerPrivate.copy(idList, function() {
+ // TODO(jiaxi): Add toast later.
+ });
this.closeDropdownMenu_();
},
/** @private */
onDeleteTap_: function() {
+ if (this.menuItem_.children) {
+ chrome.bookmarks.removeTree(this.menuItem_.id, function() {
+ // TODO(jiaxi): Add toast later.
+ }.bind(this));
+ } else {
+ chrome.bookmarks.remove(this.menuItem_.id, function() {
+ // TODO(jiaxi): Add toast later.
+ }.bind(this));
+ }
this.closeDropdownMenu_();
},
« no previous file with comments | « chrome/browser/resources/md_bookmarks/item.js ('k') | chrome/browser/resources/md_bookmarks/store.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698