Policy to control ARC Backup&Restore.

chrome/browser/chromeos/arc/arc_backup_restore_policy_handler.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/arc_backup_restore_policy_handler.h"
+
+#include "base/values.h"
+#include "chrome/common/pref_names.h"
+#include "components/policy/core/common/policy_map.h"
+#include "components/prefs/pref_value_map.h"
+#include "policy/policy_constants.h"
+
+namespace arc {
+
+ArcBackupRestorePolicyHandler::ArcBackupRestorePolicyHandler()
+    : policy::TypeCheckingPolicyHandler(policy::key::kArcBackupRestoreEnabled,
+                                        base::Value::TYPE_BOOLEAN) {}

[Yusuke Sato, 2016/07/11 03:01:06]

nit: s/{}/= default;/

[Sergey Poromov, 2016/07/11 14:35:14]

It's not possible for this constructor :(

+
+ArcBackupRestorePolicyHandler::~ArcBackupRestorePolicyHandler() {}

[Yusuke Sato, 2016/07/11 03:01:06]

same

[Sergey Poromov, 2016/07/11 14:35:14]

Done.

+
+void ArcBackupRestorePolicyHandler::ApplyPolicySettings(
+    const policy::PolicyMap& policies,
+    PrefValueMap* prefs) {
+  const base::Value* value = policies.GetValue(policy_name());
+  if (value) {
+    bool backup_restore_enabled;
+    value->GetAsBoolean(&backup_restore_enabled);

[Yusuke Sato, 2016/07/11 03:01:06]

Can you initialize backup_restore_enabled (with false) and/or check the return
value of GetAsBoolean()? Doing neither of them seems a bit scary to me.

[Sergey Poromov, 2016/07/11 14:35:14]

Done.

+    prefs->SetBoolean(prefs::kArcBackupRestoreEnabled, backup_restore_enabled);
+    prefs->SetBoolean(prefs::kArcBackupRestoreManaged, true);
+  } else {
+    prefs->SetBoolean(prefs::kArcBackupRestoreManaged, false);
+  }
+}
+
+}  // namespace arc