Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(5795)

Unified Diff: chrome/browser/chromeos/arc/policy/arc_policy_bridge.cc

Issue 2809773004: Wait for compliance report to start ARC Kiosk app. (Closed)
Patch Set: small comment update Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/chromeos/arc/policy/arc_policy_bridge.cc
diff --git a/chrome/browser/chromeos/arc/policy/arc_policy_bridge.cc b/chrome/browser/chromeos/arc/policy/arc_policy_bridge.cc
index f628eb5e8cdf48b9d57c5dab6ce8e5f5374a44b1..e837bbf5f63482636960c8a54b81634e201d92ae 100644
--- a/chrome/browser/chromeos/arc/policy/arc_policy_bridge.cc
+++ b/chrome/browser/chromeos/arc/policy/arc_policy_bridge.cc
@@ -311,6 +311,12 @@ ArcPolicyBridge::~ArcPolicyBridge() {
arc_bridge_service()->policy()->RemoveObserver(this);
}
+// static
+void ArcPolicyBridge::RegisterProfilePrefs(
+ user_prefs::PrefRegistrySyncable* registry) {
+ registry->RegisterBooleanPref(prefs::kArcPolicyComplianceReported, false);
+}
+
void ArcPolicyBridge::OverrideIsManagedForTesting(bool is_managed) {
is_managed_ = is_managed;
}
@@ -392,6 +398,8 @@ void ArcPolicyBridge::OnReportComplianceParseSuccess(
std::unique_ptr<base::Value> parsed_json) {
// Always returns "compliant".
callback.Run(kPolicyCompliantJson);
+ GetProfile()->GetPrefs()->SetBoolean(prefs::kArcPolicyComplianceReported,
+ true);
const base::DictionaryValue* dict = nullptr;
if (parsed_json->GetAsDictionary(&dict))

Powered by Google App Engine
This is Rietveld 408576698