Skip ARC initial screen when everything is set up by policy

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

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/chromeos/arc/optin/arc_terms_of_service_default_negotiator.h"
 
 #include <string>
 
 #include "base/memory/ptr_util.h"
 #include "chrome/browser/chromeos/arc/optin/arc_optin_preference_handler.h"
@@ skipping 63 matching lines @@
 }
 
 void ArcTermsOfServiceDefaultNegotiator::OnMetricsModeChanged(bool enabled,
                                                               bool managed) {
   support_host_->SetMetricsPreferenceCheckbox(enabled, managed);
 }
 
 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