MD History: Implement search functionality.

chrome/browser/resources/md_history/history_toolbar.js

 // Copyright 2015 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-toolbar',
   properties: {
     // Number of history items currently selected.
     count: {
       type: Number,
       value: 0,
       observer: 'changeToolbarView_'
     },
+
     // True if 1 or more history items are selected. When this value changes
     // the background colour changes.
     itemsSelected_: {
       type: Boolean,
       value: false,
       reflectToAttribute: true
+    },
+
+    searchTerm: {

[calamity, 2016/02/05 02:30:10]

Comment.

[hsampson, 2016/02/08 04:40:23]

Done.

+      type: String,
+      value: ''
     }
   },
 
   /**
    * Changes the toolbar background color depending on whether any history items
    * are currently selected.
    * @private
    */
   changeToolbarView_: function() {
     this.itemsSelected_ = this.count != 0;
   },
 
+  /**
+   * If the search term has changed reload for the new search.
+   */
+  onSearchTermSearch: function(searchTerm) {
+    if (searchTerm != this.searchTerm)
+      this.fire('refresh-results', {search: searchTerm});
+    this.searchTerm = searchTerm;
+  },
+
+  attached: function() {
+    this.async(function() {

[calamity, 2016/02/05 02:30:10]

Why async?

[hsampson, 2016/02/08 04:40:23]

Because otherwise this function was being called before the search bar had been
created and this is what was breaking the refresh when devtools was open.

[tsergeant, 2016/02/08 22:59:01]

This -should- be fixed by removing the dom-ifs from the toolbar. Have you tried
removing the async and seeing if it works?

[hsampson, 2016/02/09 02:17:51]

If the async is removed and you refresh the page with devtools open the search
bar no longer works. Refreshing the page normally works, but I think this is the
same bug as last time. 

+      this.searchFieldDelegate_ = new ToolbarSearchFieldDelegate(this);
+      this.$$('#search-input').setDelegate(this.searchFieldDelegate_);

[calamity, 2016/02/05 02:30:10]

I believe this can also become this.$['search-input'].

Actually, I'm super surprised that the this keyword has bound correctly here.
*shrug*

[hsampson, 2016/02/08 04:40:23]

Done.

+    });
+  },
+
   onClearSelectionTap_: function() {
     this.fire('unselect-all');
   }
 });
+
+/**
+ * @constructor
+ * @implements {SearchFieldDelegate}
+ * @param {!Object} toolbar This history-toolbar.
+ */
+function ToolbarSearchFieldDelegate(toolbar) {
+  this.toolbar_ = toolbar;
+}
+
+ToolbarSearchFieldDelegate.prototype = {
+  /** @override */
+  onSearchTermSearch: function(searchTerm) {
+    this.toolbar_.onSearchTermSearch(searchTerm);
+  }
+};