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

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

Issue 2614703003: [MD Bookmarks] Add search. (Closed)
Patch Set: Add search result refresh for bookmark editing. 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
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 f1c3b6b639770686a39ac795432f9dc7d1dd7b80..3c1ad92146c8a4b4abc6d344bc40831231ec8cd4 100644
--- a/chrome/browser/resources/md_bookmarks/list.js
+++ b/chrome/browser/resources/md_bookmarks/list.js
@@ -7,10 +7,10 @@ Polymer({
properties: {
/** @type {BookmarkTreeNode} */
- selectedNode: Object,
-
- /** @type {BookmarkTreeNode} */
menuItem_: Object,
+
+ /** @type {Array<BookmarkTreeNode>} */
+ displayedList: Array,
},
listeners: {
@@ -47,12 +47,12 @@ Polymer({
/** @private */
onDeleteTap_: function() {
- if (this.menuItem_.children) {
- chrome.bookmarks.removeTree(this.menuItem_.id, function() {
+ if (this.menuItem_.url) {
+ chrome.bookmarks.remove(this.menuItem_.id, function() {
// TODO(jiaxi): Add toast later.
}.bind(this));
} else {
- chrome.bookmarks.remove(this.menuItem_.id, function() {
+ chrome.bookmarks.removeTree(this.menuItem_.id, function() {
// TODO(jiaxi): Add toast later.
}.bind(this));
}
@@ -82,6 +82,6 @@ Polymer({
/** @private */
isListEmpty_: function() {
- return this.selectedNode.children.length == 0;
+ return this.displayedList.length == 0;
}
});

Powered by Google App Engine
This is Rietveld 408576698