[DevTools] Hide empty toolbars

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 337 matching lines @@
      * @this {UI.Toolbar}
      */
     function appendItemsInOrder(items) {
       for (var i = 0; i < items.length; ++i) {
         var item = items[i];
         if (item)
           this.appendToolbarItem(item);
       }
     }
   }
+
+  /**
+   * @param {boolean} visible
+   */
+  setVisible(visible) {

[dgozman, 2017/01/24 17:32:14]

Don't introduce a method for this, since toolbar doesn't do anything special.
Instead, toggle the class directly in the client.

[eostroukhov, 2017/01/24 17:40:28]

Done.

+    this.element.classList.toggle('hidden', !visible);
+  }
 };
 
 /**
  * @unrestricted
  */
 UI.ToolbarItem = class extends Common.Object {
   /**
    * @param {!Element} element
    */
   constructor(element) {
@@ skipping 562 matching lines @@
 
   /**
    * @override
    * @param {boolean} enabled
    */
   _applyEnabledState(enabled) {
     super._applyEnabledState(enabled);
     this.inputElement.disabled = !enabled;
   }
 };