MD Settings: Manage search engines, polish to match latest mocks.

chrome/browser/resources/settings/search_engines_page/search_engine_entry.css

Index: chrome/browser/resources/settings/search_engines_page/search_engine_entry.css
diff --git a/chrome/browser/resources/settings/search_engines_page/search_engine_entry.css b/chrome/browser/resources/settings/search_engines_page/search_engine_entry.css
index 0a643fef3ff5e58192fafe4a81bb4aceab2a239b..9d8e9b95175e978bf0a29dffda2ec16d9a6c4315 100644
--- a/chrome/browser/resources/settings/search_engines_page/search_engine_entry.css
+++ b/chrome/browser/resources/settings/search_engines_page/search_engine_entry.css
@@ -2,6 +2,10 @@
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file. */
 
+:host([is-default]) {
+  font-weight: 500;
+}
+
 .name-column,
 .keyword-column {
   flex: 3;
@@ -12,27 +16,27 @@
 }
 
 .url-column {
-  flex: 3.5;
+  flex: 4;
   overflow: hidden;
   text-overflow: ellipsis;
   white-space: nowrap;
 }
 
 .icon-container {
-  flex: 1;
-  margin: auto;
-  text-align: center;
+  -webkit-margin-end: 8px;
+  flex: 0.5;

[Dan Beam, 2016/02/23 22:19:47]

why is the icon flexing at all?  that doesn't make much sense

[dpapad, 2016/02/23 22:40:24]

Removed. I was using display:flex in name-column, and then I was assigning a
flex value in .icon-container and .name, but you are right, this was not needed.

+  text-align: left;

[Dan Beam, 2016/02/23 02:45:12]

RTL?

[dpapad, 2016/02/23 21:27:04]

Removed, this ended up being not needed. There are other things need to be
addressed for RTL though (some of them could be in Polymer), so I suggest doing
the remaining stuff in separate CL as needed.

 }
 
 .name {
-  flex: 3;
+  flex: 3.5;

[Dan Beam, 2016/02/23 02:45:12]

why are you using .5 here and previously?

[dpapad, 2016/02/23 21:27:04]

For two reasons,
1) It seems to work in the dev console.
2) It is much more convenient when adjusting visually the widths, to experiment
with a decimal value compared to keeping all values as integers and having to
adjust all children of the flexbox parent for every experimentation.

Are decimal values not allowed? I thought when using flexbox everything is
interpreted as a percentage of the maximum assigned, (for example 5 and 2.5, is
effectively same as 2 and 1).

[Dan Beam, 2016/02/23 22:08:38]

right, but where are these widths coming from?  bettes?

   margin: auto;
 }
 
 #container {
   border-top: 1px solid lightgray;
   display: flex;
-  padding: 10px 0;
+  padding: 2px 0;
 }
 
 .dropdown-content {
@@ -43,3 +47,7 @@
 paper-item:hover {
   background-color: #f0f0f0;
 }
+
+paper-icon-button {
+  -webkit-padding-end: 0;
+}