[MD settings] using h2 for sub-headers consistently.

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

Index: chrome/browser/resources/settings/search_engines_page/search_engines_page.html
diff --git a/chrome/browser/resources/settings/search_engines_page/search_engines_page.html b/chrome/browser/resources/settings/search_engines_page/search_engines_page.html
index 0933f290ad13dabb909d43997aab9d1078efc428..4b9809bbe879aeb1800085279938f4de60701b20 100644
--- a/chrome/browser/resources/settings/search_engines_page/search_engines_page.html
+++ b/chrome/browser/resources/settings/search_engines_page/search_engines_page.html
@@ -16,25 +16,23 @@
     <style include="settings-shared">
       :host {
         --search-engines-list-item-vertical-space: 8px;
-        --search-engines-list-margin: 0 40px;
-      }
-      .label {
-        margin: 20px;
+        --search-engines-list-margin: 0 36px;
       }
+
       .extension-engines {
         margin: var(--search-engines-list-margin);
       }
     </style>
-    <div>
-      <div class="label">$i18n{searchEnginesDefault}</div>
+    <div class="settings-box block">

[hcarmona, 2016/08/24 23:41:21]

This is adding a dividing line between the header and this list.
Is this expected?

[dschuyler, 2016/08/25 02:00:53]

Ah, I changed this in the next CL, but it should be in this CL

+      <h2>$i18n{searchEnginesDefault}</h2>
       <settings-search-engines-list engines="[[defaultEngines]]">
       </settings-search-engines-list>
     </div>
     <template is="dom-if" if="[[showAddSearchEngineDialog_]]" restamp>
       <settings-search-engine-dialog></settings-search-engine-dialog>
     </template>
-    <div>
-      <div class="label">$i18n{searchEnginesOther}</div>
+    <div class="settings-box block">

[hcarmona, 2016/08/24 23:41:21]

Same here for a dividing line between the lists.

[dschuyler, 2016/08/25 02:00:53]

I think a line makes sense here. I haven't found mocks for this though.

+      <h2>$i18n{searchEnginesOther}</h2>
       <!-- TODO(dbeam): why does on-click work with keyboard but on-tap
            doesn't? -->
       <settings-search-engines-list engines="[[otherEngines]]">
@@ -43,7 +41,7 @@
       </settings-search-engines-list>
 
       <template is="dom-if" if="[[showExtensionsList_]]">
-        <div class="label">$i18n{searchEnginesExtension}</div>
+        <h2>$i18n{searchEnginesExtension}</h2>
         <iron-list items="[[extensions]]" class="extension-engines" selectable>
           <template>
             <settings-omnibox-extension-entry engine="[[item]]"