[MD settings] adding dom-if to hide homepage settings

chrome/browser/resources/settings/appearance_page/appearance_page.html

Index: chrome/browser/resources/settings/appearance_page/appearance_page.html
diff --git a/chrome/browser/resources/settings/appearance_page/appearance_page.html b/chrome/browser/resources/settings/appearance_page/appearance_page.html
index 92a704163dea9dabab2f4ff0e3ce6b28fd6e875d..736219727c18770ade73cb51072c54a57e1fbb32 100644
--- a/chrome/browser/resources/settings/appearance_page/appearance_page.html
+++ b/chrome/browser/resources/settings/appearance_page/appearance_page.html
@@ -25,16 +25,17 @@
     <cr-settings-checkbox pref="{{prefs.browser.show_home_button}}"
         i18n-values="label:showHomeButton">
     </cr-settings-checkbox>
+    <template is="dom-if" if="{{prefs.browser.show_home_button.value}}">

[Dan Beam, 2015/09/16 21:50:13]

indent inside the template

[dschuyler, 2015/09/16 22:06:44]

Done.

     <div class="radio-container">
       <paper-radio-group
-          hidden="[[!prefs.browser.show_home_button.value]]"
           selected="{{isNewTabPageString}}">
-        <paper-radio-button name="t" i18n-content="homePageNtp">
+        <paper-radio-button name="t"><span>[[i18n_.homePageNtp]]</span>

[Dan Beam, 2015/09/16 21:50:14]

why do you need the <span>?

<paper-radio-button name="t">[[i18n_.homePageNtp]]</paper-radio-button>

note: wrapping lines with <x-tag>[[...]]</x-tag> doesn't always work.  in this
case: just go over 80 cols, imo

[dschuyler, 2015/09/16 22:06:43]

Thanks, I didn't know it was ok to go over the 80 cols.  And yeah,
using <span> makes it work while wrapping the rest of the line.

Done.

         </paper-radio-button>
-        <paper-radio-button name="f" i18n-content="openThisPage">
+        <paper-radio-button name="f"><span>[[i18n_.openThisPage]]</span>
         </paper-radio-button>
       </paper-radio-group>
     </div>
+    </template>
     <cr-settings-checkbox pref="{{prefs.bookmark_bar.show_on_all_tabs}}"
         i18n-values="label:showBookmarksBar">
     </cr-settings-checkbox>