WebUI: cr-search-field: Remove delegation pattern, use simple event instead.

chrome/browser/resources/md_extensions/manager.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.
 
 /**
  * The different pages that can be shown at a time.
  * Note: This must remain in sync with the order in manager.html!
  * @enum {string}
  */
 var Page = {
@@ skipping 73 matching lines @@
     created: function() {
       this.readyPromiseResolver = new PromiseResolver();
     },
 
     ready: function() {
       /** @type {extensions.Sidebar} */
       this.sidebar =
           /** @type {extensions.Sidebar} */(this.$$('extensions-sidebar'));
       this.listHelper_ = new ListHelper(this);
       this.sidebar.setListDelegate(this.listHelper_);
-      this.$.toolbar.setSearchDelegate(new SearchHelper(this));
       this.readyPromiseResolver.resolve();
     },
 
     /**
+     * @param {!CustomEvent} event
+     * @private
+     */
+    onFilterChanged_: function(event) {
+      this.filter = /** @type {string} */ (event.detail);

[Devlin, 2016/06/14 20:07:44]

nit: I think no space between */ and (event.detail), but I'll let Dan confirm.

[Dan Beam, 2016/06/14 22:11:20]

i think i originally discouraged this, but clang format might disagree with me

never cared enough to find out

+    },
+
+    /**
      * @param {chrome.developerPrivate.ExtensionType} type The type of item.
      * @return {string} The ID of the list that the item belongs in.
      * @private
      */
     getListId_: function(type) {
       var listId;
       var ExtensionType = chrome.developerPrivate.ExtensionType;
       switch (type) {
         case ExtensionType.HOSTED_APP:
         case ExtensionType.LEGACY_PACKAGED_APP:
@@ skipping 161 matching lines @@
 
       this.manager_.$['items-list'].set('items', assert(items));
       this.manager_.changePage(Page.ITEM_LIST);
     },
 
     showKeyboardShortcuts: function() {
       this.manager_.changePage(Page.KEYBOARD_SHORTCUTS);
     },
   };
 
-  /**
-   * @param {extensions.Manager} manager
-   * @constructor
-   * @implements {SearchFieldDelegate}
-   */
-  function SearchHelper(manager) {
-    this.manager_ = manager;
-  }
-
-  SearchHelper.prototype = {
-    /** @override */
-    onSearchTermSearch: function(searchTerm) {
-      this.manager_.filter = searchTerm;
-    },
-  };
-
   return {Manager: Manager};
 });