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

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

 <link rel="import" href="chrome://resources/polymer/v1_0/polymer/polymer.html">
+<link rel="import" href="chrome://md-settings/search_engines_page/search_engine_dialog.html">
+<link rel="import" href="chrome://md-settings/search_engines_page/search_engines_browser_proxy.html">
 <link rel="import" href="chrome://resources/polymer/v1_0/iron-dropdown/iron-dropdown.html">
-<link rel="import" href="chrome://md-settings/search_engines_page/search_engines_browser_proxy.html">
+<link rel="import" href="chrome://resources/polymer/v1_0/iron-icon/iron-icon.html">
+<link rel="import" href="chrome://resources/polymer/v1_0/iron-icons/iron-icons.html">
 <link rel="import" href="chrome://resources/polymer/v1_0/paper-icon-button/paper-icon-button.html">
 <link rel="import" href="chrome://resources/polymer/v1_0/paper-item/paper-item.html">
-<link rel="import" href="chrome://md-settings/search_engines_page/search_engine_dialog.html">
 <link rel="import" href="chrome://md-settings/settings_shared_css.html">
 
 <dom-module id="settings-search-engine-entry">
   <link rel="import" type="css" href="chrome://md-settings/search_engines_page/search_engine_entry.css">
   <template>
     <style include="settings-shared"></style>
     <template is="dom-if" if="[[showEditSearchEngineDialog_]]" restamp>
       <settings-search-engine-dialog model="[[engine]]">
       </settings-search-engine-dialog>
     </template>
     <div id="container" class="list-item">
       <div class="name-column">
         <span class="icon-container">
-          <iron-icon src="[[engine.iconURL]]"></iron-icon>
+          <template is="dom-if" if="[[engine.iconURL]]">
+            <iron-icon src="[[engine.iconURL]]"></iron-icon>
+          </template>
+          <template is="dom-if" if="[[!engine.iconURL]]">
+            <iron-icon icon="icons:find-in-page"></iron-icon>
+          </template>
         </span>
-        <div class="name">[[engine.displayName]]</div>
+        <span class="name">[[engine.displayName]]</span>

[Dan Beam, 2016/02/23 23:11:15]

wait, does this change to a computed display: inline; if you move from <div> ->
<span>?  why do we want that and were is it overridden otherwise?  does it not
matter if it's flexing?

[dpapad, 2016/02/23 23:25:54]

The "name-column" div is not having display:flex anymore (per previous comment).
That means that its children ("icon-container" and "name") follow normal rules
of display block (div) vs inline (span). So in order to have the icon and the
name appear side by side I need to change to a span instead of a div.

Does this address your question? (I am not sure I fully understood it)

       </div>
       <div class="keyword-column">[[engine.keyword]]</div>
       <div class="url-column">[[engine.url]]</div>
-      <div class="icon-container">
         <paper-icon-button icon="more-vert" toggles active="{{editMenuOpened}}">
         </paper-icon-button>
         <iron-dropdown opened="{{editMenuOpened}}" horizontal-align="right"
             vertical-align="top">
           <div class="dropdown-content">
             <paper-item on-tap="onMakeDefaultTap_"
                 i18n-content="searchEnginesMakeDefault"
                 hidden="[[engine.default]]" id="makeDefault"></paper-item>
             <paper-item on-tap="onEditTap_"
                 i18n-content="searchEnginesEdit" id="edit"></paper-item>
             <paper-item on-tap="onDeleteTap_"
                 i18n-content="searchEnginesRemoveFromList"
                 hidden="[[engine.default]]" id="delete"></paper-item>
           <div>
         </iron-dropdown>
-      </div>
     </div>
   </template>
   <script src="search_engine_entry.js"></script>
 </dom-module>