MD History: add shortcuts 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.
 
 Polymer({
   is: 'history-app',
 
   properties: {
     // The id of the currently selected page.
     selectedPage_: {type: String, value: 'history', observer: 'unselectAll'},
@@ skipping 29 matching lines @@
     'cr-menu-tap': 'onMenuTap_',
     'history-checkbox-select': 'checkboxSelected',
     'unselect-all': 'unselectAll',
     'delete-selected': 'deleteSelected',
     'search-domain': 'searchDomain_',
   },
 
   /** @override */
   ready: function() {
     this.grouped_ = loadTimeData.getBoolean('groupByDomain');
+
+    cr.ui.decorate('command', cr.ui.Command);
+    document.addEventListener('canExecute', this.onCanExecute_.bind(this));
+    document.addEventListener('command', this.onCommand_.bind(this));
   },
 
   /** @private */
   onMenuTap_: function() { this.$['side-bar'].toggle(); },
 
   /**
    * Listens for history-item being selected or deselected (through checkbox)
    * and changes the view of the top toolbar.
    * @param {{detail: {countAddition: number}}} e
    */
@@ skipping 41 matching lines @@
     listContainer.historyResult(info, results);
   },
 
   /**
    * Fired when the user presses 'More from this site'.
    * @param {{detail: {domain: string}}} e
    */
   searchDomain_: function(e) { this.$.toolbar.setSearchTerm(e.detail.domain); },
 
   /**
+   * @param {Event} e

[tsergeant, 2016/07/05 07:47:22]

Does the closure compile work successfully if you make this
{cr.ui.CanExecuteEvent} and remove the type cast below? I know this is what
Downloads does, but it seems like it's not the easiest way.

[lshang, 2016/07/06 05:03:08]

Seems closure compiler can't recognize cr.ui.CanExecuteEvent as Event, as we
see.

+   * @private
+   */
+  onCanExecute_: function(e) {
+    e = /** @type {cr.ui.CanExecuteEvent} */(e);
+    switch (e.command.id) {

[tsergeant, 2016/07/05 07:47:22]

I think there's no need to have the switch here, you can just do

e.canExecute = true;

for every event.

[lshang, 2016/07/06 05:03:08]

Done.

+      case 'find-command':
+        e.canExecute = true;
+        break;
+    }
+  },
+
+  /**
+   * @param {Event} e
+   * @private
+   */
+  onCommand_: function(e) {
+    e = /** @type {cr.ui.CanExecuteEvent} */(e);
+    if (e.command.id == 'find-command')
+      this.$.toolbar.onFindCommand();
+  },
+
+  /**
    * @param {!Array<!ForeignSession>} sessionList Array of objects describing
    *     the sessions from other devices.
    * @param {boolean} isTabSyncEnabled Is tab sync enabled for this profile?
    */
   setForeignSessions: function(sessionList, isTabSyncEnabled) {
     if (!isTabSyncEnabled)
       return;
 
     this.sessionList = sessionList;
   },
@@ skipping 23 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 != '';
   },
 });