drive: Move temporary file removal responsiblity to FileCache

chrome/browser/chromeos/drive/file_cache.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_CHROMEOS_DRIVE_FILE_CACHE_H_
 #define CHROME_BROWSER_CHROMEOS_DRIVE_FILE_CACHE_H_
 
 #include <string>
 #include <vector>
 
@@ skipping 190 matching lines @@
   // Stores |source_path| to the cache and mark it as dirty, i.e., needs to be
   // uploaded to the remove server for syncing.
   // |callback| must not be null.
   // Must be called on the UI thread.
   void StoreLocallyModifiedOnUIThread(const std::string& resource_id,
                                       const std::string& md5,
                                       const base::FilePath& source_path,
                                       FileOperationType file_operation_type,
                                       const FileOperationCallback& callback);
 
-  // Pins the specified entry.
+  // Runs Pin() on |blocking_task_runner_|, and calls |callback| with the result
+  // asynchronously.
   // |callback| must not be null.
   // Must be called on the UI thread.
   void PinOnUIThread(const std::string& resource_id,
                      const std::string& md5,
                      const FileOperationCallback& callback);
 
+  // Pins the specified entry.
+  FileError Pin(const std::string& resource_id,
+                const std::string& md5);
+
   // Runs Unpin() on |blocking_task_runner_|, and calls |callback| with the
   // result asynchronously.
   // |callback| must not be null.
   // Must be called on the UI thread.
   void UnpinOnUIThread(const std::string& resource_id,
                        const std::string& md5,
                        const FileOperationCallback& callback);
 
   // Unpins the specified entry.
   FileError Unpin(const std::string& resource_id, const std::string& md5);
@@ skipping 68 matching lines @@
   // in linux box and unittest. (http://crosbug.com/27577)
   static bool CreateCacheDirectories(
       const std::vector<base::FilePath>& paths_to_create);
 
   // Returns the type of the sub directory where the cache file is stored.
   static CacheSubDirectoryType GetSubDirectoryType(
       const FileCacheEntry& cache_entry);
 
  private:
   friend class FileCacheTest;
+  friend class FileCacheTestOnUIThread;
 
   // Enum defining origin of a cached file.
   enum CachedFileOrigin {
     CACHED_FILE_FROM_SERVER = 0,
     CACHED_FILE_LOCALLY_MODIFIED,
     CACHED_FILE_MOUNTED,
   };
 
   ~FileCache();
 
@@ skipping 18 matching lines @@
 
   // Used to implement Store and StoreLocallyModifiedOnUIThread.
   // TODO(hidehiko): Merge this method with Store(), after
   // StoreLocallyModifiedOnUIThread is removed.
   FileError StoreInternal(const std::string& resource_id,
                           const std::string& md5,
                           const base::FilePath& source_path,
                           FileOperationType file_operation_type,
                           CachedFileOrigin origin);
 
-  // Used to implement PinOnUIThread.
-  FileError Pin(const std::string& resource_id,
-                const std::string& md5);
-
   // Used to implement MarkAsMountedOnUIThread.
   FileError MarkAsMounted(const std::string& resource_id,
                           const std::string& md5,
                           base::FilePath* cache_file_path);
 
   // Used to implement MarkAsUnmountedOnUIThread.
   FileError MarkAsUnmounted(const base::FilePath& file_path);
 
   // Used to implement MarkDirtyOnUIThread.
   FileError MarkDirty(const std::string& resource_id,
@@ skipping 49 matching lines @@
 // this value.
 //
 // Copied from cryptohome/homedirs.h.
 // TODO(satorux): Share the constant.
 const int64 kMinFreeSpace = 512 * 1LL << 20;
 
 }  // namespace internal
 }  // namespace drive
 
 #endif  // CHROME_BROWSER_CHROMEOS_DRIVE_FILE_CACHE_H_