[DevTools] Merge filter bar with the main toolbar

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

 /*
  * Copyright (C) 2009 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 534 matching lines @@
   Click: Symbol('Click'),
   MouseDown: Symbol('MouseDown'),
   MouseUp: Symbol('MouseUp')
 };
 
 /**
  * @unrestricted
  */
 UI.ToolbarInput = class extends UI.ToolbarItem {
   /**
-   * @param {string=} placeholder
+   * @param {string} placeholder
    * @param {number=} growFactor
    * @param {number=} shrinkFactor
+   * @param {boolean=} isSearchField
    */
-  constructor(placeholder, growFactor, shrinkFactor) {
-    super(createElementWithClass('input', 'toolbar-item'));
-    this.element.addEventListener('input', this._onChangeCallback.bind(this), false);
+  constructor(placeholder, growFactor, shrinkFactor, isSearchField) {
+    super(createElementWithClass('div', 'toolbar-input'));
+
+    this._input = this.element.createChild('input');
+    this._input.addEventListener('blur', () => this.element.classList.toggle('focused', false));
+    this._input.addEventListener('focus', () => this.element.classList.toggle('focused', true));
+    this._input.addEventListener('input', () => this._onChangeCallback(), false);
+    this._isSearchField = !!isSearchField;
     if (growFactor)
       this.element.style.flexGrow = growFactor;
     if (shrinkFactor)
       this.element.style.flexShrink = shrinkFactor;
     if (placeholder)
-      this.element.setAttribute('placeholder', placeholder);
-    this._value = '';
+      this._input.setAttribute('placeholder', placeholder);
+
+    if (isSearchField)
+      this._setupSearchControls();
+  }
+
+  _setupSearchControls() {
+    var clearButton = this.element.createChild('div', 'toolbar-input-clear-button');
+    clearButton.appendChild(UI.Icon.create('smallicon-clear-input', 'search-cancel-button'));
+    clearButton.addEventListener('click', () => this._internalSetValue('', true));
+    this._input.addEventListener('keydown', event => this._onKeydownCallback(event));
   }
 
   /**
    * @param {string} value
    */
   setValue(value) {
-    this._value = value;
-    this.element.value = value;
+    this._internalSetValue(value, false);
+  }
+
+  /**
+   * @param {string} value
+   * @param {boolean} notify
+   */
+  _internalSetValue(value, notify) {
+    this._input.value = value;
+    if (notify)
+      this._onChangeCallback();
   }
 
   /**
    * @return {string}
    */
   value() {
-    return this.element.value;
+    return this._input.value;
+  }
+
+  /**
+   * @param {!Event} event
+   */
+  _onKeydownCallback(event) {
+    this.dispatchEventToListeners(UI.ToolbarInput.Event.KeyDown, event);
+    if (this.isSearchField || event.code !== 'Escape' || !this._input.value)

[pfeldman, 2017/02/06 19:37:42]

isEscKey(event)

[eostroukhov, 2017/02/07 00:24:18]

Done.

+      return;
+    this._internalSetValue('', true);
+    event.consume(true);
   }
 
   _onChangeCallback() {
-    this.dispatchEventToListeners(UI.ToolbarInput.Event.TextChanged, this.element.value);
+    this.dispatchEventToListeners(UI.ToolbarInput.Event.TextChanged, this._input.value);
   }
 };
 
 UI.ToolbarInput.Event = {
-  TextChanged: Symbol('TextChanged')
+  TextChanged: Symbol('TextChanged'),
+  KeyDown: Symbol('KeyDown')

[pfeldman, 2017/02/06 19:37:42]

Don't expose it - it needs to be a part of traversal policy.

[eostroukhov, 2017/02/07 00:24:18]

Done.
(I've exposed the input element so the pane can set a listener directly)

 };
 
 /**
  * @unrestricted
  */
 UI.ToolbarToggle = class extends UI.ToolbarButton {
   /**
    * @param {string} title
    * @param {string=} glyph
    * @param {string=} toggledGlyph
@@ skipping 395 matching lines @@
 
   /**
    * @override
    * @param {boolean} enabled
    */
   _applyEnabledState(enabled) {
     super._applyEnabledState(enabled);
     this.inputElement.disabled = !enabled;
   }
 };