[MD settings] i18n-content to $i18n switch in the settings menu

[MD settings] i18n-content to $i18n switch in the settings menu

This is a mechanical change from using i18n-content to using $i18n
replacements in the settings menu (the side nav).

BUG=611188
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/23ecf06e5064f55c9f0e93306b82fd1edb58ba2a
Cr-Commit-Position: refs/heads/master@{#393145}

[dschuyler, 2016-05-11 23:54:36]

Description was changed from

==========
[MD settings] i18n-content to $i18n switch in the settings menu

This is a mechanical change from using i18n-content to using $i18n
replacements in the settings menu (the side nav).

BUG=611188
==========

to

==========
[MD settings] i18n-content to $i18n switch in the settings menu

This is a mechanical change from using i18n-content to using $i18n
replacements in the settings menu (the side nav).

BUG=611188
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

[dschuyler, 2016-05-12 00:03:34]

Patchset #1 (id:1) has been deleted

[dschuyler, 2016-05-12 00:05:04]

dschuyler@chromium.org changed reviewers:
+ tommycli@chromium.org

[tommycli, 2016-05-12 00:06:30]

lgtm sans below

https://codereview.chromium.org/1975583002/diff/20001/chrome/browser/resource...
File chrome/browser/resources/settings/settings_menu/settings_menu.html (right):

https://codereview.chromium.org/1975583002/diff/20001/chrome/browser/resource...
chrome/browser/resources/settings/settings_menu/settings_menu.html:118:
<span>$i18n{advancedPageTitle}</span>
Is this span still needed?

[dschuyler, 2016-05-12 00:09:13]

https://codereview.chromium.org/1975583002/diff/20001/chrome/browser/resource...
File chrome/browser/resources/settings/settings_menu/settings_menu.html (right):

https://codereview.chromium.org/1975583002/diff/20001/chrome/browser/resource...
chrome/browser/resources/settings/settings_menu/settings_menu.html:118:
<span>$i18n{advancedPageTitle}</span>
On 2016/05/12 00:06:30, tommycli wrote:
> Is this span still needed?

I believe so. The span allows the title to 
flex expand which pushes the iron-icon to the
right edge. (It was not needed in the other
changes in this file, so I removed those, for
example).

[dschuyler, 2016-05-12 01:22:07]

The CQ bit was checked by dschuyler@chromium.org

[commit-bot: I haz the power, 2016-05-12 01:22:23]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1975583002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1975583002/20001

[commit-bot: I haz the power, 2016-05-12 02:02:51]

Committed patchset #1 (id:20001)

[commit-bot: I haz the power, 2016-05-12 02:06:27]

Description was changed from

==========
[MD settings] i18n-content to $i18n switch in the settings menu

This is a mechanical change from using i18n-content to using $i18n
replacements in the settings menu (the side nav).

BUG=611188
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

to

==========
[MD settings] i18n-content to $i18n switch in the settings menu

This is a mechanical change from using i18n-content to using $i18n
replacements in the settings menu (the side nav).

BUG=611188
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/23ecf06e5064f55c9f0e93306b82fd1edb58ba2a
Cr-Commit-Position: refs/heads/master@{#393145}
==========

[commit-bot: I haz the power, 2016-05-12 02:06:28]

Patchset 1 (id:??) landed as
https://crrev.com/23ecf06e5064f55c9f0e93306b82fd1edb58ba2a
Cr-Commit-Position: refs/heads/master@{#393145}