MD Settings: adjust dialog style and scroll border visual

ui/webui/resources/cr_elements/cr_dialog/cr_dialog.html

Index: ui/webui/resources/cr_elements/cr_dialog/cr_dialog.html
diff --git a/ui/webui/resources/cr_elements/cr_dialog/cr_dialog.html b/ui/webui/resources/cr_elements/cr_dialog/cr_dialog.html
index 14a5a866e2d4e464289772fb38d5256ff0eed063..d10f3d286c43360a90841d5cf0dd0938a0b0dd71 100644
--- a/ui/webui/resources/cr_elements/cr_dialog/cr_dialog.html
+++ b/ui/webui/resources/cr_elements/cr_dialog/cr_dialog.html
@@ -30,9 +30,8 @@
 
       .top-container {
         align-items: center;
-        border-bottom: 1px solid var(--divider-color);
         display: flex;
-        min-height: 52px;
+        min-height: 47px;
       }
 
       .title-container {
@@ -42,15 +41,23 @@
       }
 
       :host ::content .title {
-        font-size: 123.07%; /* (16px / 13px) * 100 */
+        font-size: 115.38%; /* (15px / 13px) * 100 */

[dschuyler, 2017/04/06 21:53:22]

Two spaces between ; and comment on inline comments.
font-size: 115.38%;  /* (15px / 13px) * 100 */

[scottchen, 2017/04/06 22:15:57]

Done.

+        line-height: 1;
+        margin-bottom: 16px;
+        margin-top: 16px;
       }
 
       #close {
         --paper-icon-button: {
-          height: 40px;
-          width: 40px;
+          height: 36px;
+          width: 36px;
         };
-        -webkit-margin-end: 6px;
+
+        --layout-inline: {
+          display: flex;
+        };
+
+        -webkit-margin-end: 4px;
         /* <paper-icon-button> overrides --iron-icon-{height,width}, so this
          * padding essentially reduces 40x40 to 20x20. */
         padding: 10px;
@@ -66,12 +73,17 @@
       :host([show-scroll-borders]) .body-container {
         /* Prevent layout moving when border does appear. */
         border-bottom: 1px solid transparent;
+        border-top: 1px solid transparent;
       }
 
       :host([show-scroll-borders]) .body-container.bottom-scrollable {
         border-bottom: 1px solid var(--paper-grey-300);
       }
 
+      :host([show-scroll-borders]) .body-container.top-scrollable {
+        border-top: 1px solid var(--paper-grey-300);
+      }
+
       :host ::content .body {
         padding-bottom: 12px;
         padding-top: 12px;
@@ -79,8 +91,8 @@
 
       :host ::content .body,
       :host ::content .title {
-        -webkit-padding-end: 24px;
-        -webkit-padding-start: 24px;
+        -webkit-padding-end: 16px;
+        -webkit-padding-start: 16px;

[dschuyler, 2017/04/06 21:53:22]

How about 
padding: 0 16px;
(There's nothing special needed for -end and -start if they are the same).

[scottchen, 2017/04/06 22:15:57]

Done.

         flex: 1;
       }
 
@@ -89,8 +101,8 @@
         -webkit-padding-start: 16px;
         display: flex;
         justify-content: flex-end;
-        margin-bottom: 12px;
-        margin-top: 12px;
+        margin-bottom: 16px;
+        margin-top: 16px;
       }
 
       :host ::content .button-container .cancel-button {
@@ -118,6 +130,7 @@
       </paper-icon-button>
     </div>
     <div class="body-container">
+      <span id="bodyTopMarker"></span>
       <content select=".body"></content>
       <span id="bodyBottomMarker"></span>
     </div>