Fix unable to opt-in to crash reports & usage statistics from web-app

Fix unable to opt-in to crash reports & usage statistics from web-app

Cause:
The crash opt-in setting is sometimes controlled by domain policy, which will
prevent the user from enabling it.  However, this is not communicated to the
user properly.

Fix:
- Shows a tooltip "This setting is controlled by your domain policy" if
  necessary.
- Hides the check box if the domain doesn't allow usage stats collection
  as an empty disabled check box offers no value to the user.

BUG=388864
Committed: https://crrev.com/2d1da95cd331870aca3f598c6a64d0d00ec0064d
Cr-Commit-Position: refs/heads/master@{#315105}

[kelvinp, 2015-02-05 23:48:58]

kelvinp@chromium.org changed reviewers:
+ jamiewalch@chromium.org

[kelvinp, 2015-02-05 23:48:59]

PTAL

[kelvinp, 2015-02-05 23:49:09]

Patchset #1 (id:1) has been deleted

[Jamie, 2015-02-06 02:22:02]

LGTM as long as my concern is unfounded.

https://codereview.chromium.org/898163002/diff/20001/remoting/webapp/crd/js/h...
File remoting/webapp/crd/js/host_setup_dialog.js (right):

https://codereview.chromium.org/898163002/diff/20001/remoting/webapp/crd/js/h...
remoting/webapp/crd/js/host_setup_dialog.js:200: that.usageStatsCheckbox_.title
= l10n.getTranslationOrError(
Do the disables and title attributes apply to both the checkbox and its label? I
thought we decided that the original behaviour was confusing in part because the
label wasn't greyed out.

[kelvinp, 2015-02-06 19:49:50]

New patchsets have been uploaded after l-g-t-m from jamiewalch@chromium.org

[kelvinp, 2015-02-06 19:50:21]

FYI

https://codereview.chromium.org/898163002/diff/20001/remoting/webapp/crd/js/h...
File remoting/webapp/crd/js/host_setup_dialog.js (right):

https://codereview.chromium.org/898163002/diff/20001/remoting/webapp/crd/js/h...
remoting/webapp/crd/js/host_setup_dialog.js:200: that.usageStatsCheckbox_.title
= l10n.getTranslationOrError(
On 2015/02/06 02:22:02, Jamie wrote:
> Do the disables and title attributes apply to both the checkbox and its label?
I
> thought we decided that the original behaviour was confusing in part because
the
> label wasn't greyed out.

Done.

[kelvinp, 2015-02-06 19:50:35]

The CQ bit was checked by kelvinp@chromium.org

[commit-bot: I haz the power, 2015-02-06 19:51:58]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/898163002/40001

[commit-bot: I haz the power, 2015-02-06 20:57:19]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2015-02-06 20:57:49]

Patchset 2 (id:??) landed as
https://crrev.com/2d1da95cd331870aca3f598c6a64d0d00ec0064d
Cr-Commit-Position: refs/heads/master@{#315105}