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

Side by Side Diff: chrome/browser/resources/md_bookmarks/list.js

Issue 2614703003: [MD Bookmarks] Add search. (Closed)
Patch Set: Merge and fix closure. 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 unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 Polymer({ 5 Polymer({
6 is: 'bookmarks-list', 6 is: 'bookmarks-list',
7 7
8 properties: { 8 properties: {
9 /** @type {BookmarkTreeNode} */ 9 /** @type {BookmarkTreeNode} */
10 selectedNode: Object, 10 menuItem_: Object,
11 11
12 /** @type {BookmarkTreeNode} */ 12 displayedList: Array,
tsergeant 2017/01/12 06:45:05 Closure annotation here
angelayang 2017/01/12 07:51:24 Done.
13 menuItem_: Object,
14 }, 13 },
15 14
16 listeners: { 15 listeners: {
17 'open-item-menu': 'onOpenItemMenu_', 16 'open-item-menu': 'onOpenItemMenu_',
18 }, 17 },
19 18
20 /** 19 /**
21 * @param {Event} e 20 * @param {Event} e
22 * @private 21 * @private
23 */ 22 */
(...skipping 51 matching lines...) Expand 10 before | Expand all | Expand 10 after
75 74
76 /** @private */ 75 /** @private */
77 closeDropdownMenu_: function() { 76 closeDropdownMenu_: function() {
78 var menu = /** @type {!CrActionMenuElement} */ ( 77 var menu = /** @type {!CrActionMenuElement} */ (
79 this.$.dropdown); 78 this.$.dropdown);
80 menu.close(); 79 menu.close();
81 }, 80 },
82 81
83 /** @private */ 82 /** @private */
84 isListEmpty_: function() { 83 isListEmpty_: function() {
85 return this.selectedNode.children.length == 0; 84 return this.displayedList.length == 0;
86 } 85 }
87 }); 86 });
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698