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

Side by Side Diff: chrome/browser/chromeos/arc/optin/arc_optin_preference_handler.cc

Issue 2702893002: Revert of Skip ARC initial screen when everything is set up by policy (Closed)
Patch Set: 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 unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/chromeos/arc/optin/arc_optin_preference_handler.h" 5 #include "chrome/browser/chromeos/arc/optin/arc_optin_preference_handler.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "chrome/browser/browser_process.h" 8 #include "chrome/browser/browser_process.h"
9 #include "chrome/browser/chromeos/arc/optin/arc_optin_preference_handler_observe r.h" 9 #include "chrome/browser/chromeos/arc/optin/arc_optin_preference_handler_observe r.h"
10 #include "chrome/browser/metrics/chrome_metrics_service_accessor.h" 10 #include "chrome/browser/metrics/chrome_metrics_service_accessor.h"
(...skipping 54 matching lines...) Expand 10 before | Expand all | Expand 10 after
65 65
66 void ArcOptInPreferenceHandler::SendMetricsMode() { 66 void ArcOptInPreferenceHandler::SendMetricsMode() {
67 if (g_browser_process->local_state()) { 67 if (g_browser_process->local_state()) {
68 observer_->OnMetricsModeChanged( 68 observer_->OnMetricsModeChanged(
69 ChromeMetricsServiceAccessor::IsMetricsAndCrashReportingEnabled(), 69 ChromeMetricsServiceAccessor::IsMetricsAndCrashReportingEnabled(),
70 IsMetricsReportingPolicyManaged()); 70 IsMetricsReportingPolicyManaged());
71 } 71 }
72 } 72 }
73 73
74 void ArcOptInPreferenceHandler::SendBackupAndRestoreMode() { 74 void ArcOptInPreferenceHandler::SendBackupAndRestoreMode() {
75 // Override the pref default to the true value, in order to encourage users to
76 // consent with it during OptIn flow.
77 const bool enabled =
78 pref_service_->HasPrefPath(prefs::kArcBackupRestoreEnabled)
79 ? pref_service_->GetBoolean(prefs::kArcBackupRestoreEnabled)
80 : true;
81 observer_->OnBackupAndRestoreModeChanged( 75 observer_->OnBackupAndRestoreModeChanged(
82 enabled, 76 pref_service_->GetBoolean(prefs::kArcBackupRestoreEnabled),
83 pref_service_->IsManagedPreference(prefs::kArcBackupRestoreEnabled)); 77 pref_service_->IsManagedPreference(prefs::kArcBackupRestoreEnabled));
84 } 78 }
85 79
86 void ArcOptInPreferenceHandler::SendLocationServicesMode() { 80 void ArcOptInPreferenceHandler::SendLocationServicesMode() {
87 // Override the pref default to the true value, in order to encourage users to
88 // consent with it during OptIn flow.
89 const bool enabled =
90 pref_service_->HasPrefPath(prefs::kArcLocationServiceEnabled)
91 ? pref_service_->GetBoolean(prefs::kArcLocationServiceEnabled)
92 : true;
93 observer_->OnLocationServicesModeChanged( 81 observer_->OnLocationServicesModeChanged(
94 enabled, 82 pref_service_->GetBoolean(prefs::kArcLocationServiceEnabled),
95 pref_service_->IsManagedPreference(prefs::kArcLocationServiceEnabled)); 83 pref_service_->IsManagedPreference(prefs::kArcLocationServiceEnabled));
96 } 84 }
97 85
98 void ArcOptInPreferenceHandler::EnableMetrics(bool is_enabled) { 86 void ArcOptInPreferenceHandler::EnableMetrics(bool is_enabled) {
99 if (g_browser_process->local_state()) 87 if (g_browser_process->local_state())
100 ChangeMetricsReportingState(is_enabled); 88 ChangeMetricsReportingState(is_enabled);
101 } 89 }
102 90
103 void ArcOptInPreferenceHandler::EnableBackupRestore(bool is_enabled) { 91 void ArcOptInPreferenceHandler::EnableBackupRestore(bool is_enabled) {
104 pref_service_->SetBoolean(prefs::kArcBackupRestoreEnabled, is_enabled); 92 pref_service_->SetBoolean(prefs::kArcBackupRestoreEnabled, is_enabled);
105 } 93 }
106 94
107 void ArcOptInPreferenceHandler::EnableLocationService(bool is_enabled) { 95 void ArcOptInPreferenceHandler::EnableLocationService(bool is_enabled) {
108 pref_service_->SetBoolean(prefs::kArcLocationServiceEnabled, is_enabled); 96 pref_service_->SetBoolean(prefs::kArcLocationServiceEnabled, is_enabled);
109 } 97 }
110 98
111 } // namespace arc 99 } // namespace arc
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698