Devtools: Introduce WI.ExtensibleToolbar

Source/devtools/front_end/components/InspectorView.js

Index: Source/devtools/front_end/components/InspectorView.js
diff --git a/Source/devtools/front_end/components/InspectorView.js b/Source/devtools/front_end/components/InspectorView.js
index 48fb5a248e674d7f4fc2bdd0b3071b2894bfdcca..392051bcde41700d5dcbfb6058056be45d003bac 100644
--- a/Source/devtools/front_end/components/InspectorView.js
+++ b/Source/devtools/front_end/components/InspectorView.js
@@ -53,20 +53,6 @@ WebInspector.InspectorView = function()
     this._drawerSplitView.setMainView(this._tabbedPane);
     this._drawer = new WebInspector.Drawer(this._drawerSplitView);
 
-    this._leftToolbar = new WebInspector.Toolbar();
-    this._leftToolbar.element.classList.add("inspector-view-toolbar", "inspector-view-toolbar-left");
-    this._leftToolbar.makeNarrow();
-    this._tabbedPane.insertBeforeTabStrip(this._leftToolbar.element);
-
-    this._rightToolbar = new WebInspector.Toolbar();
-    this._rightToolbar.element.classList.add("inspector-view-toolbar");
-    this._tabbedPane.appendAfterTabStrip(this._rightToolbar.element);
-
-    this._closeButtonToolbarItem = createElementWithClass("div", "inspector-view-close-button");
-    var closeButtonElement = this._closeButtonToolbarItem.createChild("div", "", "dt-close-button");
-    closeButtonElement.addEventListener("click", InspectorFrontendHost.closeWindow.bind(InspectorFrontendHost), true);
-    this._rightToolbar.element.appendChild(this._closeButtonToolbarItem);
-
     this._panels = {};
     // Used by tests.
     WebInspector["panels"] = this._panels;
@@ -127,20 +113,21 @@ WebInspector.InspectorView.prototype = {
         WebInspector.endBatchUpdate();
     },
 
-    /**
-     * @param {!WebInspector.ToolbarItem} item
-     */
-    appendToLeftToolbar: function(item)
+    createToolbars: function()
     {
-        this._leftToolbar.appendToolbarItem(item);
-    },
+        this._leftToolbar = new WebInspector.ExtensibleToolbar("toolbar-left");

[dgozman, 2015/04/23 14:09:36]

Let's rename to "main-toolbar-left".

[sergeyv, 2015/04/23 17:42:12]

Done.

+        this._leftToolbar.element.classList.add("inspector-view-toolbar", "inspector-view-toolbar-left");
+        this._leftToolbar.makeNarrow();
+        this._tabbedPane.insertBeforeTabStrip(this._leftToolbar.element);
 
-    /**
-     * @param {!WebInspector.ToolbarItem} item
-     */
-    appendToRightToolbar: function(item)
-    {
-        this._rightToolbar.appendToolbarItem(item);
+        this._rightToolbar = new WebInspector.ExtensibleToolbar("toolbar-right");
+        this._rightToolbar.element.classList.add("inspector-view-toolbar");
+        this._tabbedPane.appendAfterTabStrip(this._rightToolbar.element);
+
+        this._closeButtonToolbarItem = createElementWithClass("div", "inspector-view-close-button");

[dgozman, 2015/04/23 14:09:36]

Turn this into StatusBarItem.

[sergeyv, 2015/04/23 17:42:12]

Done.

+        var closeButtonElement = this._closeButtonToolbarItem.createChild("div", "", "dt-close-button");
+        closeButtonElement.addEventListener("click", InspectorFrontendHost.closeWindow.bind(InspectorFrontendHost), true);
+        this._rightToolbar.element.appendChild(this._closeButtonToolbarItem);
     },
 
     /**
@@ -204,8 +191,10 @@ WebInspector.InspectorView.prototype = {
     {
         this._currentPanelLocked = WebInspector.targetManager.allTargetsSuspended();
         this._tabbedPane.setCurrentTabLocked(this._currentPanelLocked);
-        this._leftToolbar.setEnabled(!this._currentPanelLocked);
-        this._rightToolbar.setEnabled(!this._currentPanelLocked);
+        if (this._leftToolbar)
+            this._leftToolbar.setEnabled(!this._currentPanelLocked);
+        if (this._rightToolbar)
+            this._rightToolbar.setEnabled(!this._currentPanelLocked);
     },
 
     /**