DevTools: cleanup button styles

third_party/WebKit/Source/devtools/front_end/ui/searchableView.css

 /*
  * Copyright (c) 2014 The Chromium Authors. All rights reserved.
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 .search-bar {
     flex: 0 0 23px;
     background-color: #eee;
     border-top: 1px solid #ccc;
@@ skipping 60 matching lines @@
 .toolbar-search-navigation.enabled {
     opacity: 1.0;
 }
 
 .toolbar-search button.search-action-button {
     border: 1px solid rgb(163, 163, 163);
     border-radius: 8px;
     margin: 1px 3px 0 3px;
     background-image: linear-gradient(rgb(241, 241, 241), rgb(220, 220, 220));
     width: 75px;
-    height: 20px;
+    min-height: 20px !important;

[pfeldman, 2017/05/30 21:18:17]

What else is setting min-height? Can we resolve the conflict?

[luoe, 2017/05/30 22:39:45]

Done.  We can resolve this by defining `height` for all buttons in
textButton.css.

Before, we put the same !important min-height in textButton.css to override the
universal selector in inspectorCommon.css that gives everything min-height: 0.
https://codereview.chromium.org/1085603003

     white-space: nowrap;
+    padding: 0;

[pfeldman, 2017/05/30 21:18:17]

ditto

[luoe, 2017/05/30 22:39:45]

Done.  We can avoid this by letting these buttons use the default button
padding.  The width will adjust, accordingly.

 }
 
 .toolbar-search button.search-action-button:active {
     background-image: linear-gradient(rgb(185, 185, 185), rgb(156, 156, 156));
 }
 
 .toolbar-search-control {
     display: -webkit-flex;
     position: relative;
     background-color: white;
@@ skipping 43 matching lines @@
     background-image: url(Images/searchNext.png), linear-gradient(rgb(168, 168, 168), rgb(116, 116, 116));
 }
 
 .search-results-matches {
     display: inline-block;
     text-align: right;
     padding: 0 4px;
     color: rgb(165, 165, 165);
     align-self: center;
 }