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

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

Issue 2613683002: [MD Bookmarks] Add Delete and Copy URL for Material Bookmarks. (Closed)
Patch Set: 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-item', 6 is: 'bookmarks-item',
7 7
8 properties: { 8 properties: {
9 /** @type {BookmarkTreeNode} */ 9 /** @type {BookmarkTreeNode} */
10 item: { 10 item: {
11 type: Object, 11 type: Object,
12 observer: 'onItemChanged_' 12 observer: 'onItemChanged_',
13 }, 13 },
14 14
15 isFolder_: Boolean, 15 isFolder_: Boolean,
16 }, 16 },
17 17
18 observers: [ 18 observers: [
19 'updateFavicon_(item.url)' 19 'updateFavicon_(item.url)',
20 ], 20 ],
21 21
22 /** 22 /**
23 * @param {Event} e 23 * @param {Event} e
24 * @private 24 * @private
25 */ 25 */
26 onMenuButtonOpenTap_: function(e) { 26 onMenuButtonOpenTap_: function(e) {
27 this.fire('toggle-menu', { 27 this.fire('open-item-menu', {
28 target: e.target 28 target: e.target,
29 item: this.item
29 }); 30 });
30 }, 31 },
31 32
32 /** @private */ 33 /** @private */
33 onItemChanged_: function() { 34 onItemChanged_: function() { this.isFolder_ = !(this.item.url); },
calamity 2017/01/04 06:57:44 This format is wrong. Try running git cl format, i
jiaxi 2017/01/04 22:44:00 Done.
34 this.isFolder_ = !(this.item.url);
35 },
36 35
37 /** @private */ 36 /** @private */
38 updateFavicon_: function(url) { 37 updateFavicon_: function(url) {
39 this.$.icon.style.backgroundImage = cr.icon.getFavicon(url); 38 this.$.icon.style.backgroundImage = cr.icon.getFavicon(url);
40 } 39 },
41 }); 40 });
OLDNEW
« no previous file with comments | « no previous file | chrome/browser/resources/md_bookmarks/list.js » ('j') | chrome/browser/resources/md_bookmarks/list.js » ('J')

Powered by Google App Engine
This is Rietveld 408576698