DevTools: make toolbar button click targets square

Source/devtools/front_end/main/module.json

 {
     "extensions": [
         {
             "type": "@WebInspector.AppProvider",
             "condition": "can_dock",
             "className": "WebInspector.AdvancedAppProvider",
             "order": 0
         },
         {
             "type": "@WebInspector.AppProvider",
@@ skipping 140 matching lines @@
         },
         {
             "type": "@WebInspector.ToolbarItem.Provider",
             "className": "WebInspector.InspectElementModeController.ToggleButtonProvider",
             "location": "main-toolbar-left",
             "order": 0,
             "actionId": "main.toggle-element-search"
         },
         {
             "type": "@WebInspector.ToolbarItem.Provider",
-            "className": "WebInspector.Main.WarningErrorCounter",
+            "separator": true,
+            "location": "main-toolbar-left",
+            "order": 100,

[dgozman, 2015/06/11 16:14:12]

100 is for close button. 99?

[pfeldman, 2015/06/11 16:38:06]

These are different toolbars.

+            "actionId": "main.toggle-element-search"

[dgozman, 2015/06/11 16:14:12]

Remove actionId.

[pfeldman, 2015/06/11 16:38:06]

Done.

+        },
+        {
+            "type": "@WebInspector.ToolbarItem.Provider",
+            "className": "WebInspector.InspectorView.ToggleDrawerButtonProvider",
             "order": 1,
             "location": "main-toolbar-right"
         },
         {
             "type": "@WebInspector.ToolbarItem.Provider",
-            "className": "WebInspector.InspectorView.ToggleDrawerButtonProvider",
+            "className": "WebInspector.Main.WarningErrorCounter",

[dgozman, 2015/06/11 16:14:11]

Why swapped with toggle drawer?

[pfeldman, 2015/06/11 16:38:06]

Done.

             "order": 2,
             "location": "main-toolbar-right"
         },
         {
             "type": "@WebInspector.ToolbarItem.Provider",
-            "className": "WebInspector.DockController.ButtonProvider",
-            "order": 4,
+            "separator": true,
+            "order": 3,
             "location": "main-toolbar-right"
         },
         {
+            "type": "@WebInspector.ToolbarItem.Provider",
+            "className": "WebInspector.DockController.ButtonProvider",
+            "order": 5,
+            "location": "main-toolbar-right"
+        },
+        {
+            "type": "@WebInspector.ToolbarItem.Provider",
+            "separator": true,
+            "order": 90,
+            "location": "main-toolbar-right"
+        },
+        {
             "type": "@WebInspector.ToolbarItem.Provider",
             "className": "WebInspector.DockController.CloseButtonProvider",
             "order": 100,
             "location": "main-toolbar-right"
         },
         {
             "type": "setting",
             "category": "",
             "title": "Disable cache (while DevTools is open)",
             "settingName": "cacheDisabled",
@@ skipping 101 matching lines @@
         "Tests.js",
         "OverlayController.js",
         "Main.js"
     ],
     "skip_compilation": [
         "Tests.js"
     ],
     "resources": [
     ]
 }