arc: kArcCompatibleFilesystemChosen pref to local state and integer.

chrome/browser/chromeos/arc/arc_util.h

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 7765832e9abad9327bef3b6adbac810e0718fc6c..03108e69d09bfcaf7cd20456d3f7a9afeae5920f 100644
--- a/chrome/browser/chromeos/arc/arc_util.h
+++ b/chrome/browser/chromeos/arc/arc_util.h
@@ -19,6 +19,17 @@ class FilePath;
 
 namespace arc {
 
+// Values to be stored in the local state preference to keep track of the
+// filesystem encryption migration status.
+enum FileSystemCompatibilityState {

[Luis Héctor Chávez, 2017/04/13 21:11:47]

nit: enum FileSystemCompatibilityState : int32_t

[kinaba, 2017/04/14 04:33:55]

Done.

+  // 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,

[hidehiko, 2017/04/13 12:38:27]

This is not yet used. So how about;
- Move this definition to the CL which actually needs to use this, or
- add comment (TODO) that this will be used just in case.

[kinaba, 2017/04/14 04:33:55]

Added a TODO comment.

+};
+
 // 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