DevTools: properly update close icon in search field

third_party/WebKit/Source/devtools/front_end/sources/AdvancedSearchView.js

Index: third_party/WebKit/Source/devtools/front_end/sources/AdvancedSearchView.js
diff --git a/third_party/WebKit/Source/devtools/front_end/sources/AdvancedSearchView.js b/third_party/WebKit/Source/devtools/front_end/sources/AdvancedSearchView.js
index 6cc960fd292accdc9179b1ac35a2e3254565868e..96ca16938fa80b362127a4f896b14385997b92e4 100644
--- a/third_party/WebKit/Source/devtools/front_end/sources/AdvancedSearchView.js
+++ b/third_party/WebKit/Source/devtools/front_end/sources/AdvancedSearchView.js
@@ -18,7 +18,8 @@ WebInspector.AdvancedSearchView = function()
 
     this._searchPanelElement = this.contentElement.createChild("div", "search-drawer-header");
     this._searchPanelElement.addEventListener("keydown", this._onKeyDown.bind(this), false);
-    this._searchPanelElement.addEventListener("keyup", this._onKeyUp.bind(this), false);
+    this._searchPanelElement.addEventListener("keyup", this._textChanged.bind(this), false);

[lushnikov, 2016/05/02 18:43:41]

why would you need a keyup listener here? input should cover all the usecases,
shouldn't it?

[luoe, 2016/05/02 19:18:52]

You're right!

+    this._searchPanelElement.addEventListener("input", this._textChanged.bind(this), false);
 
     this._searchResultsElement = this.contentElement.createChild("div");
     this._searchResultsElement.className = "search-results";
@@ -296,7 +297,7 @@ WebInspector.AdvancedSearchView.prototype = {
         }
     },
 
-    _onKeyUp: function()
+    _textChanged: function()
     {
         if (this._search.value && this._search.value.length)
             this._searchInputClearElement.hidden = false;