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

Unified Diff: chrome/browser/chromeos/arc/optin/arc_optin_preference_handler.cc

Issue 2682833003: Skip ARC initial screen when everything is set up by policy (Closed)
Patch Set: Rebase Created 3 years, 10 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/optin/arc_optin_preference_handler.cc
diff --git a/chrome/browser/chromeos/arc/optin/arc_optin_preference_handler.cc b/chrome/browser/chromeos/arc/optin/arc_optin_preference_handler.cc
index 11972d688fd39a34511fb8db97483b2f1af2b97e..8ae3b0f1a8ff713b9a54d3115ccddfadf6eeee1c 100644
--- a/chrome/browser/chromeos/arc/optin/arc_optin_preference_handler.cc
+++ b/chrome/browser/chromeos/arc/optin/arc_optin_preference_handler.cc
@@ -72,14 +72,26 @@ void ArcOptInPreferenceHandler::SendMetricsMode() {
}
void ArcOptInPreferenceHandler::SendBackupAndRestoreMode() {
+ // Override the pref default to the true value, in order to encourage users to
+ // consent with it during OptIn flow.
+ const bool enabled =
+ pref_service_->HasPrefPath(prefs::kArcBackupRestoreEnabled)
+ ? pref_service_->GetBoolean(prefs::kArcBackupRestoreEnabled)
+ : true;
observer_->OnBackupAndRestoreModeChanged(
- pref_service_->GetBoolean(prefs::kArcBackupRestoreEnabled),
+ enabled,
pref_service_->IsManagedPreference(prefs::kArcBackupRestoreEnabled));
}
void ArcOptInPreferenceHandler::SendLocationServicesMode() {
+ // Override the pref default to the true value, in order to encourage users to
+ // consent with it during OptIn flow.
+ const bool enabled =
+ pref_service_->HasPrefPath(prefs::kArcLocationServiceEnabled)
+ ? pref_service_->GetBoolean(prefs::kArcLocationServiceEnabled)
+ : true;
observer_->OnLocationServicesModeChanged(
- pref_service_->GetBoolean(prefs::kArcLocationServiceEnabled),
+ enabled,
pref_service_->IsManagedPreference(prefs::kArcLocationServiceEnabled));
}

Powered by Google App Engine
This is Rietveld 408576698