[MD History] Use openForeignSession message to open foreign session tabs.

chrome/browser/resources/md_history/synced_device_card.js

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 Polymer({
   is: 'history-synced-device-card',
 
   properties: {
     // Name of the synced device.
     device: {type: String, value: ''},
@@ skipping 16 matching lines @@
      * separate array here is necessary for window separators to appear
      * correctly in search. See http://crrev.com/2022003002 for more details.
      * @type {!Array<number>}
      */
     separatorIndexes: Array,
 
     // Whether the card is open.
     cardOpen_: {type: Boolean, value: true},
 
     searchTerm: String,
+
+    windowId: Number,

[tsergeant, 2016/07/22 03:21:17]

Nit: Newline between these

[calamity, 2016/07/27 05:22:07]

Done.

+    sessionTag: String,
   },
 
   /**
    * Opens all the tabs displayed on the device in separate tabs.
+   * @param {Event} e
    * @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');
+  openAllTabs_: function(e) {

[tsergeant, 2016/07/22 03:21:17]

e is unused, you can get rid of it

[calamity, 2016/07/27 05:22:06]

Done.

+    md_history.BrowserService.getInstance().openForeignSessionAllTabs(
+        this.sessionTag);
   },
 
   /**
+   * @param {Event} e
+   * @private
+   */
+  openTab_: function(e) {
+    var currentTarget = e.currentTarget;

[tsergeant, 2016/07/22 03:21:17]

You should be able to use `e.model` to avoid sticking extra properties on the
<a>.

[calamity, 2016/07/27 05:22:07]

Done.

+    md_history.BrowserService.getInstance().openForeignSessionTab(
+        this.sessionTag, currentTarget.windowId, currentTarget.sessionId, e);
+    e.preventDefault();
+  },
+
+  /**
    * Toggles the dropdown display of synced tabs for each device card.
    */
   toggleTabCard: function() {
     this.$.collapse.toggle();
     this.$['dropdown-indicator'].icon =
         this.$.collapse.opened ? 'cr:expand-less' : 'cr:expand-more';
   },
 
   /**
    * When the synced tab information is set, the icon associated with the tab
    * website is also set.
    * @private
    */
   updateIcons_: function() {
     this.async(function() {
       var icons = Polymer.dom(this.root).querySelectorAll('.website-icon');
 
       for (var i = 0; i < this.tabs.length; i++) {
         icons[i].style.backgroundImage =
             cr.icon.getFaviconImageSet(this.tabs[i].url);
       }
     });
   },
 
   /** @private */
   isWindowSeparatorIndex_: function(index, separatorIndexes) {
     return this.separatorIndexes.indexOf(index) != -1;
   }
 });