[DevTools] Rework some focus code.

third_party/WebKit/Source/devtools/front_end/ui/FilterBar.js

Index: third_party/WebKit/Source/devtools/front_end/ui/FilterBar.js
diff --git a/third_party/WebKit/Source/devtools/front_end/ui/FilterBar.js b/third_party/WebKit/Source/devtools/front_end/ui/FilterBar.js
index abb3f0f1b186aec031b37a319bb7f9c37e57f6cb..cc62f9c5362c2a9d2379accf3794a0e5c27249d8 100644
--- a/third_party/WebKit/Source/devtools/front_end/ui/FilterBar.js
+++ b/third_party/WebKit/Source/devtools/front_end/ui/FilterBar.js
@@ -224,7 +224,6 @@ WebInspector.TextFilterUI = function(supportRegex)
     this._filterInputElement = /** @type {!HTMLInputElement} */ (this._filterElement.createChild("input", "filter-input-field"));
     this._filterInputElement.placeholder = WebInspector.UIString("Filter");
     this._filterInputElement.id = "filter-input-field";
-    this._filterInputElement.addEventListener("mousedown", this._onFilterFieldManualFocus.bind(this), false); // when the search field is manually selected

[einbinder, 2016/09/30 19:37:52]

Why can this be removed? Or more precisely why was it here in the first place?

[dgozman, 2016/09/30 20:27:19]

This is just a strange piece of code. Things work manually.

     this._filterInputElement.addEventListener("input", this._onInput.bind(this), false);
     this._filterInputElement.addEventListener("change", this._onChange.bind(this), false);
     this._filterInputElement.addEventListener("keydown", this._onInputKeyDown.bind(this), true);
@@ -311,14 +310,6 @@ WebInspector.TextFilterUI.prototype = {
     /**
      * @param {!Event} event
      */
-    _onFilterFieldManualFocus: function(event)
-    {
-        WebInspector.setCurrentFocusElement(/** @type {?Node} */ (event.target));
-    },
-
-    /**
-     * @param {!Event} event
-     */
     _onBlur: function(event)
     {
         this._cancelSuggestion();