MD Settings: hide people section when signin is disabled (e.g. by policy)

MD Settings: hide people section when signin is disabled (e.g. by policy)

R=tommycli@chromium.org
BUG=629604
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

[Dan Beam, 2016-09-29 04:38:45]

Description was changed from

==========
MD Settings: hide people section when signin is disabled (e.g. by policy)

R=tommycli@chromium.org
BUG=629604
==========

to

==========
MD Settings: hide people section when signin is disabled (e.g. by policy)

R=tommycli@chromium.org
BUG=629604
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[Dan Beam, 2016-09-29 04:38:54]

The CQ bit was checked by dbeam@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-29 04:39:02]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-29 05:24:30]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-29 05:24:31]

Dry run: This issue passed the CQ dry run.

[tommycli, 2016-09-29 19:27:29]

The code is lgtm. But only if we are indeed sure we want to hide the whole
section if signin is disallowed.

But as discussed, don't we still want the user to be able to edit their local
profile's name and icon?

https://codereview.chromium.org/2381783002/diff/1/chrome/browser/resources/se...
File chrome/browser/resources/settings/basic_page/basic_page.js (right):

https://codereview.chromium.org/2381783002/diff/1/chrome/browser/resources/se...
chrome/browser/resources/settings/basic_page/basic_page.js:32: /** @private
{Object} */
nit: Add comment that this is populated by the child settings-people-page.

[Dan Beam, 2016-09-29 21:59:21]

Description was changed from

==========
MD Settings: hide people section when signin is disabled (e.g. by policy)

R=tommycli@chromium.org
BUG=629604
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
MD Settings: hide people section when signin is disabled (e.g. by policy)

R=tommycli@chromium.org
BUG=629604
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[Dan Beam, 2016-09-29 21:59:31]

closing in favor of https://codereview.chromium.org/2375013004/