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

Unified Diff: chrome/browser/chromeos/arc/arc_util.cc

Issue 2801173002: arc: Provide API to control Play Store state from autotests (Closed)
Patch Set: changed JS API from isEnabled/Managed to getState 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/arc_util.cc
diff --git a/chrome/browser/chromeos/arc/arc_util.cc b/chrome/browser/chromeos/arc/arc_util.cc
index 7723792c34c6f22bc5eecadd693adb496c8f206d..31ef3479987e69a410ad7f41c25aaffa1dbe7e78 100644
--- a/chrome/browser/chromeos/arc/arc_util.cc
+++ b/chrome/browser/chromeos/arc/arc_util.cc
@@ -163,6 +163,10 @@ bool IsArcPlayStoreEnabledPreferenceManagedForProfile(const Profile* profile) {
void SetArcPlayStoreEnabledForProfile(Profile* profile, bool enabled) {
DCHECK(IsArcAllowedForProfile(profile));
if (IsArcPlayStoreEnabledPreferenceManagedForProfile(profile)) {
+ if (enabled && !IsArcPlayStoreEnabledForProfile(profile)) {
+ LOG(WARNING) << "Attempt to enable disabled by policy ARC.";
+ return;
+ }
VLOG(1) << "Google-Play-Store-enabled pref is managed. Request to "
<< (enabled ? "enable" : "disable") << " Play Store is not stored";
// Need update ARC session manager manually for managed case in order to

Powered by Google App Engine
This is Rietveld 408576698