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

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

Issue 2850123002: M58: arc: Set migration success notification pref. (Closed)
Patch Set: Fix some comment for M58 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.h
diff --git a/chrome/browser/chromeos/arc/arc_util.h b/chrome/browser/chromeos/arc/arc_util.h
index bde8bbcccb9790e32ca1b3d2d55dd37595f61e2b..9d4342f978532f33cbb75c0fe6230c009ac34680 100644
--- a/chrome/browser/chromeos/arc/arc_util.h
+++ b/chrome/browser/chromeos/arc/arc_util.h
@@ -5,16 +5,40 @@
#ifndef CHROME_BROWSER_CHROMEOS_ARC_ARC_UTIL_H_
#define CHROME_BROWSER_CHROMEOS_ARC_ARC_UTIL_H_
+#include <stdint.h>
+
+#include "base/callback_forward.h"
+
// Most utility should be put in components/arc/arc_util.{h,cc}, rather than
// here. However, some utility implementation requires other modules defined in
// chrome/, so this file contains such utilities.
// Note that it is not allowed to have dependency from components/ to chrome/
// by DEPS.
+class AccountId;
class Profile;
+namespace base {
+class FilePath;
+}
+
namespace arc {
+// Values to be stored in the local state preference to keep track of the
+// filesystem encryption migration status.
+enum FileSystemCompatibilityState : int32_t {
+ // No migiration has happend, user keeps using the old file system.
+ kFileSystemIncompatible = 0,
+ // Migration has happend. New filesystem is in use.
+ kFileSystemCompatible = 1,
+ // Migration has happend, and a notification about the fact was already shown.
+ kFileSystemCompatibleAndNotified = 2,
+
+ // Existing code assumes that kFileSystemIncompatible is the only state
+ // representing incompatibility and other values are all variants of
+ // "compatible" state. Be careful in the case adding a new enum value.
+};
+
// Returns true if ARC is allowed to run for the given profile.
// Otherwise, returns false, e.g. if the Profile is not for the primary user,
// ARC is not available on the device, it is in the flow to set up managed
@@ -57,6 +81,14 @@ bool IsArcPlayStoreEnabledPreferenceManagedForProfile(const Profile* profile);
// Google Play Store, then ARC can run without opt-in.
void SetArcPlayStoreEnabledForProfile(Profile* profile, bool enabled);
+// Checks and updates the preference value whether the underlying filesystem
+// for the profile is compatible with ARC, when necessary. After it's done (or
+// skipped), |callback| is run either synchronously or asynchronously.
+void UpdateArcFileSystemCompatibilityPrefIfNeeded(
+ const AccountId& account_id,
+ const base::FilePath& profile_path,
+ const base::Closure& callback);
+
} // namespace arc
#endif // CHROME_BROWSER_CHROMEOS_ARC_ARC_UTIL_H_
« no previous file with comments | « chrome/browser/chromeos/arc/arc_migration_guide_notification.cc ('k') | chrome/browser/chromeos/arc/arc_util.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698