[MD History] Add search for synced devices.

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: {
@@ skipping 10 matching lines @@
     /**
      * The list of tabs open for this device.
      * @type {!Array<!ForeignSessionTab>}
      */
     tabs: {
       type: Array,
       value: function() { return []; },
       observer: 'updateIcons_'
     },
 
+    /**
+     * The indexes where a window separator should be shown.

[tsergeant, 2016/06/02 07:12:53]

Nit: It would be nice to try and explain why this method is used, since it's a
little unintuitive and subtle.

[calamity, 2016/06/02 08:20:44]

Done.

For those wondering, this was done to remove the old method of window separator
insertion which relied on modifying the sessionList elements. This would require
resetting and notifying each item. ensuring stale values are handled correctly.
Binding this array to the computed function provides a simpler notification
model since we don't have any stale value problems and are guaranteed to just
recompute everything when this array changes.

+     * @type {!Array<number>}
+     */
+    separatorIndexes: Array,
+
     // Whether the card is open.
     cardOpen_: {
       type: Boolean,
       value: true
     },
   },
 
   /**
    * Opens all the tabs displayed on the device in separate tabs.
    * @private
@@ skipping 20 matching lines @@
    */
   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);
       }
     });
+  },
+
+  isWindowSeparatorIndex: function(index, separatorIndexes) {

[tsergeant, 2016/06/02 07:12:53]

Nit: Make this a private function.

[calamity, 2016/06/02 08:20:44]

Done.

+    return this.separatorIndexes.indexOf(index) != -1;
   }
 });