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

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

Issue 2172563002: [MD History] Use openForeignSession message to open foreign session tabs. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@move_menu_up
Patch Set: Created 4 years, 5 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 f58cec9b879a49fbe0ea21c0c047470168aa99c3..da1f0f38a833ee746625d3476bcfe96cc3d023b3 100644
--- a/chrome/browser/resources/md_history/synced_device_card.js
+++ b/chrome/browser/resources/md_history/synced_device_card.js
@@ -34,6 +34,10 @@ Polymer({
cardOpen_: {type: Boolean, value: true},
searchTerm: String,
+
+ windowId: Number,
+
+ sessionTag: String,
},
/**
@@ -41,9 +45,20 @@ Polymer({
* @private
*/
openAllTabs_: function() {
- // TODO(calamity): add a warning if an excessive number of tabs will open.
- for (var i = 0; i < this.tabs.length; i++)
- window.open(this.tabs[i].url, '_blank');
+ md_history.BrowserService.getInstance().openForeignSessionAllTabs(
+ this.sessionTag);
+ },
+
+ /**
+ * @param {TapEvent} e
+ * @private
+ */
+ openTab_: function(e) {
+ var model = /** @type {ForeignSessionTab} */(e.model);
+ var srcEvent = /** @type {Event} */(e.detail.sourceEvent);
+ md_history.BrowserService.getInstance().openForeignSessionTab(
+ this.sessionTag, model.windowId, model.sessionId, srcEvent);
+ e.preventDefault();
},
/**

Powered by Google App Engine
This is Rietveld 408576698