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

Unified Diff: chrome/browser/resources/md_history/synced_device_card.js

Issue 2204833003: MD History: Add menu to cards on Synced Tabs page (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@history_improve_collapse_button
Patch Set: Fix tests Created 4 years, 4 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_history/synced_device_card.js
diff --git a/chrome/browser/resources/md_history/synced_device_card.js b/chrome/browser/resources/md_history/synced_device_card.js
index 4e807781cab51ef016ee62d0b10d4a818dae5302..71760a79c882e4ff05090b0a614106fb27aff594 100644
--- a/chrome/browser/resources/md_history/synced_device_card.js
+++ b/chrome/browser/resources/md_history/synced_device_card.js
@@ -7,10 +7,10 @@ Polymer({
properties: {
// Name of the synced device.
- device: {type: String, value: ''},
+ device: String,
// When the device information was last updated.
- lastUpdateTime: {type: String, value: ''},
+ lastUpdateTime: String,
/**
* The list of tabs open for this device.
@@ -35,19 +35,11 @@ Polymer({
searchTerm: String,
+ // Internal identifier for the device.
sessionTag: String,
},
/**
- * Opens all the tabs displayed on the device in separate tabs.
- * @private
- */
- openAllTabs_: function() {
- md_history.BrowserService.getInstance().openForeignSessionAllTabs(
- this.sessionTag);
- },
-
- /**
* @param {TapEvent} e
* @private
*/
@@ -97,4 +89,16 @@ Polymer({
return cardOpen ? loadTimeData.getString('collapseSessionButton') :
loadTimeData.getString('expandSessionButton');
},
+
+ /**
+ * @param {CustomEvent} e
+ * @private
+ */
+ onMenuButtonTap_: function(e) {
+ this.fire('toggle-menu', {
+ target: Polymer.dom(e).localTarget,
+ tag: this.sessionTag
+ });
+ e.stopPropagation(); // Prevent iron-collapse.
+ },
});

Powered by Google App Engine
This is Rietveld 408576698