MD Settings: Bring <settings-dialog> closer to dialog spec.

chrome/browser/resources/settings/settings_dialog.html

 <link rel="import" href="chrome://resources/html/polymer.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-dialog-behavior/paper-dialog-behavior.html">
 <link rel="import" href="chrome://resources/polymer/v1_0/paper-dialog-behavior/paper-dialog-shared-styles.html">
 <link rel="import" href="chrome://resources/polymer/v1_0/paper-icon-button/paper-icon-button.html">
 
 <dom-module id="settings-dialog">
   <template>
     <style include="paper-dialog-shared-styles"></style>
     <style>
       .body-content {
+        display: flex;
+        flex-direction: column;
         font-size: 1em;
-        margin: 20px 0;
+        min-height: 120px;
       }
 
       .dialog-content {
         -webkit-padding-end: 0;
         -webkit-padding-start: 0;
         margin-bottom: 0;
         margin-top: 0;
       }
 
       .footer-container {
         margin: 0;
         padding: 0;
       }
 
       .top-row {
         align-items: center;
-        border-bottom: 1px solid var(--paper-grey-300);
+        border-bottom: 1px solid rgba(0, 0, 0, 0.14);
         display: flex;
-        padding-bottom: 5px;
-        padding-top: 5px;
+        min-height: 52px;
+      }
+
+      paper-icon-button {
+        height: 20px;
+        margin: 16px;
+        padding: 0;
+        width: 20px;
       }
 
       :host {
         max-width: 800px;
-        min-width: 500px;
+        min-width: 512px;
         overflow: auto;
       }
 
       :host ::content .body {
-        margin-bottom: 35px;
+        flex: 1;
+        margin: 12px 0 24px 0;
       }
 
       :host ::content .button-container {
         display: flex;
         justify-content: flex-end;
+        margin-bottom: 12px;
       }
 
       :host ::content .button-container .cancel-button {
+        -webkit-margin-end: 8px;
         color: var(--paper-grey-600);
         font-weight: 500;
       }
 
       :host ::content .footer {
         background-color: var(--paper-grey-200);
         margin: 0;
         padding: 20px;
       }
 
       :host ::content .row {
         align-items: center;
         display: flex;
         min-height: 40px;
       }
 
       :host ::content .row .start {
         flex: 1;
       }
 
       :host ::content .title {
         flex: 1;
-        font-size: 1.13em;
+        font-size: 114.28%; /* (16px / 14px) * 100 */

[dpapad, 2016/04/18 17:24:26]

Per our last week's conversation, I figured out why 100% => 14px instead of 13px
here. Because <settings-dialog> is using the shared CSS style module
"paper-dialog-shared-styles" (see [1]), which in turn is using
@apply(--paper-font-body1), see [2], which is defined at [3].

[1]
https://code.google.com/p/chromium/codesearch#chromium/src/chrome/browser/res...
[2]
https://code.google.com/p/chromium/codesearch#chromium/src/third_party/polyme...
[3]
https://code.google.com/p/chromium/codesearch#chromium/src/third_party/polyme...

So basically using "paper-dialog-shared-styles" causes the 13px setting to be
overriden.

[dschuyler, 2016/04/18 18:52:52]

That sounds right :) So there are a couple options.
We could set the font size on the body text or we
can use the --paper-dialog mixin to inherit the body
font size.  I'd vote for the --paper-dialog mixin.

This can be done in this CL or another CL.

       }
 
       :host ::content .title,
       .body-content {
         -webkit-padding-end: 24px;
         -webkit-padding-start: 24px;
       }
     </style>
     <div class="dialog-content">
       <div class="top-row">
         <content select=".title"></content>
         <paper-icon-button icon="clear" on-tap="cancel" id="close">
         </paper-icon-button>
       </div>
       <div class="body-content">
         <content select=".body"></content>
         <content select=".button-container"></content>
       </div>
     </div>
     <div class="footer-container">
       <content select=".footer"></content>
     </div>
   </template>
   <script src="settings_dialog.js"></script>
 </dom-module>