[MD History] Add URL parameter for search.

chrome/browser/resources/md_history/app.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.
 
 /**
  * @typedef {{querying: boolean,
  *            searchTerm: string,
  *            results: ?Array<!HistoryEntry>,
  *            info: ?HistoryQuery,
  *            incremental: boolean,
@@ skipping 46 matching lines @@
 
     // Route data for the current page.
     routeData: Object,
   },
 
   observers: [
     // routeData.page <=> selectedPage
     'routeDataChanged_(routeData.page)',
     'selectedPageChanged_(selectedPage_)',
 
+    // route.__queryParams.q <=> queryState.searchTerm
     'searchTermChanged_(queryState_.searchTerm)',
+    'searchQueryParamChanged_(route.__queryParams.q)',

[tsergeant, 2016/06/16 02:00:15]

app-route exposes a `queryParams` property. You should use that rather than bind
on something that's private.

[calamity, 2016/06/16 06:49:14]

Ugh! So _that's_ where that was. I was so confused about why I had to access a
private thing inside the route/tail. Why do we even _have_ that lever?

+
     'groupedRangeChanged_(queryState_.range)',
   ],
 
   // TODO(calamity): Replace these event listeners with data bound properties.
   listeners: {
     'history-checkbox-select': 'checkboxSelected',
     'unselect-all': 'unselectAll',
     'delete-selected': 'deleteSelected',
     'search-domain': 'searchDomain_',
     'load-more-history': 'loadMoreHistory_',
   },
 
   /** @override */
   ready: function() {
     this.grouped_ = loadTimeData.getBoolean('groupByDomain');
+    if (window.location.hash) {
+      window.location.href = window.location.href.split('#')[0] + '?' +
+          window.location.hash.substr(1);
+    }
   },
 
   /**
    * Listens for history-item being selected or deselected (through checkbox)
    * and changes the view of the top toolbar.
    * @param {{detail: {countAddition: number}}} e
    */
   checkboxSelected: function(e) {
     var toolbar = /** @type {HistoryToolbarElement} */(this.$.toolbar);
     toolbar.count += e.detail.countAddition;
@@ skipping 77 matching lines @@
   routeDataChanged_: function(page) {
     this.selectedPage_ = page;
   },
 
   selectedPageChanged_: function(selectedPage) {
     this.set('routeData.page', selectedPage);
   },
 
   searchTermChanged_: function(searchTerm) {
     this.queryHistory(false);
+    this.set('route.__queryParams.q', searchTerm || null);
+  },
+
+  searchQueryParamChanged_: function(searchTerm) {
+    this.$.toolbar.setSearchTerm(searchTerm || '');
   },
 
   groupedRangeChanged_: function(range) {
     this.set('routeData.page', range == HistoryRange.ALL_TIME ? '' : 'grouped');
     this.queryHistory(false);
   },
 
   loadMoreHistory_: function() {
     this.queryHistory(true);
   },
@@ skipping 61 matching lines @@
    * @param {boolean} incremental
    * @param {string} searchTerm
    * @return {boolean} Whether a loading spinner should be shown (implies the
    *     backend is querying a new search term).
    * @private
    */
   shouldShowSpinner_: function(querying, incremental, searchTerm) {
     return querying && !incremental && searchTerm != '';
   }
 });