Skip ARC initial screen when everything is set up by policy

chrome/browser/chromeos/arc/optin/arc_terms_of_service_default_negotiator.cc

Index: chrome/browser/chromeos/arc/optin/arc_terms_of_service_default_negotiator.cc
diff --git a/chrome/browser/chromeos/arc/optin/arc_terms_of_service_default_negotiator.cc b/chrome/browser/chromeos/arc/optin/arc_terms_of_service_default_negotiator.cc
index ca0fe339fb59128b7a44c4f9c2dadb84d2084f20..02cb48cb6335c2a04b4e1af2a6080df120f88d8f 100644
--- a/chrome/browser/chromeos/arc/optin/arc_terms_of_service_default_negotiator.cc
+++ b/chrome/browser/chromeos/arc/optin/arc_terms_of_service_default_negotiator.cc
@@ -81,13 +81,17 @@ void ArcTermsOfServiceDefaultNegotiator::OnMetricsModeChanged(bool enabled,
 void ArcTermsOfServiceDefaultNegotiator::OnBackupAndRestoreModeChanged(
     bool enabled,
     bool managed) {
-  support_host_->SetBackupAndRestorePreferenceCheckbox(enabled, managed);
+  const bool checkbox_checked = enabled || !managed;

[hidehiko, 2017/02/14 11:51:36]

For unmanaged users, even if backup-and-restore is set to false, ArcSupport will
receive "true", which sounds unexpected behavior.

Could you kindly explain what you'd like to do here?

Ditto for below.

[emaxx, 2017/02/15 03:40:59]

That's exactly what I was trying to achieve: the pref has its default ("false")
value
until the ToS screen completes, while the checkbox displayed to the user in the
unmanaged case should be "on" by default.
Added a comment on that.

+  support_host_->SetBackupAndRestorePreferenceCheckbox(checkbox_checked,
+                                                       managed);
 }
 
 void ArcTermsOfServiceDefaultNegotiator::OnLocationServicesModeChanged(
     bool enabled,
     bool managed) {
-  support_host_->SetLocationServicesPreferenceCheckbox(enabled, managed);
+  const bool checkbox_checked = enabled || !managed;
+  support_host_->SetLocationServicesPreferenceCheckbox(checkbox_checked,
+                                                       managed);
 }
 
 }  // namespace arc