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

Side by Side Diff: chrome/browser/resources/md_history/synced_device_card.js

Issue 2583353003: MD History: Replace last usage of cr-shared-menu with cr-action-menu. (Closed)
Patch Set: Test found real subtle bug (display flex overriding [hidden] attribute) Created 3 years, 12 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: 'history-synced-device-card', 6 is: 'history-synced-device-card',
7 7
8 properties: { 8 properties: {
9 /** 9 /**
10 * The list of tabs open for this device. 10 * The list of tabs open for this device.
(...skipping 132 matching lines...) Expand 10 before | Expand all | Expand 10 after
143 getCollapseTitle_: function(opened) { 143 getCollapseTitle_: function(opened) {
144 return opened ? loadTimeData.getString('collapseSessionButton') : 144 return opened ? loadTimeData.getString('collapseSessionButton') :
145 loadTimeData.getString('expandSessionButton'); 145 loadTimeData.getString('expandSessionButton');
146 }, 146 },
147 147
148 /** 148 /**
149 * @param {CustomEvent} e 149 * @param {CustomEvent} e
150 * @private 150 * @private
151 */ 151 */
152 onMenuButtonTap_: function(e) { 152 onMenuButtonTap_: function(e) {
153 this.fire('toggle-menu', { 153 this.fire('open-menu', {
154 target: Polymer.dom(e).localTarget, 154 target: Polymer.dom(e).localTarget,
155 tag: this.sessionTag, 155 tag: this.sessionTag,
156 }); 156 });
157 e.stopPropagation(); // Prevent iron-collapse. 157 e.stopPropagation(); // Prevent iron-collapse.
158 }, 158 },
159 159
160 onLinkRightClick_: function() { 160 onLinkRightClick_: function() {
161 md_history.BrowserService.getInstance().recordHistogram( 161 md_history.BrowserService.getInstance().recordHistogram(
162 SYNCED_TABS_HISTOGRAM_NAME, SyncedTabsHistogram.LINK_RIGHT_CLICKED, 162 SYNCED_TABS_HISTOGRAM_NAME, SyncedTabsHistogram.LINK_RIGHT_CLICKED,
163 SyncedTabsHistogram.LIMIT); 163 SyncedTabsHistogram.LIMIT);
164 }, 164 },
165 }); 165 });
OLDNEW
« no previous file with comments | « chrome/browser/resources/md_history/shared_style.html ('k') | chrome/browser/resources/md_history/synced_device_manager.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698