Mark removable Drive caches for cryptohome.

components/drive/chromeos/file_cache.h

Index: components/drive/chromeos/file_cache.h
diff --git a/components/drive/chromeos/file_cache.h b/components/drive/chromeos/file_cache.h
index 23b104c9630a9b609e4957501ac8f8fe8823e414..45b59618a0938e1a8f4c7e136ad5860f36a23fc0 100644
--- a/components/drive/chromeos/file_cache.h
+++ b/components/drive/chromeos/file_cache.h
@@ -53,6 +53,9 @@ class FreeDiskSpaceGetterInterface {
 // GetCacheFilePath() for example), should be run with |blocking_task_runner|.
 class FileCache {
  public:
+  // The file extended attribute assigned to Drive cache directory.
+  static const std::string kGCacheFilesAttribute;

[hashimoto, 2016/05/10 10:33:49]

Variables of class type with static storage duration are forbidden by our C++
style guide.
https://google.github.io/styleguide/cppguide.html#Static_and_Global_Variables

Please make this char[].

[oka, 2016/05/11 04:24:10]

Done.

+
   // Enum defining type of file operation e.g. copy or move, etc.
   enum FileOperationType {
     FILE_OPERATION_MOVE = 0,
@@ -158,14 +161,6 @@ class FileCache {
       const ResourceMetadataStorage::RecoveredCacheInfoMap&
           recovered_cache_info);
 
-  // Migrates cache files from |from| directory to |to_files| directory with
-  // creating links in |to_links| directory if necessary. Returns true for
-  // success.
-  static bool MigrateCacheFiles(const base::FilePath& from,
-                                const base::FilePath& to_files,
-                                const base::FilePath& to_links,
-                                ResourceMetadataStorage* metadata_storage);
-
  private:
   friend class FileCacheTest;
 
@@ -191,6 +186,12 @@ class FileCache {
   // TODO(hashimoto): Remove this method at some point.
   bool RenameCacheFilesToNewFormat();
 
+  // Adds appropriate file attributes to the Drive cache directory and files in
+  // it for crbug.com/533750. Returns true on success.
+  // This also resolves inconsistency between cache files and metadata which can
+  // be produced when cryptohome removed cache files or on abrupt shutdown.
+  bool MigrateCacheFilesResolvingInconsistency();
+
   // This method must be called after writing to a cache file.
   // Used to implement OpenForWrite().
   void CloseForWrite(const std::string& id);