Check for space before storing files to the cache.

chrome/browser/chromeos/gdata/gdata_cache.cc

 // 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.
 
 #include "chrome/browser/chromeos/gdata/gdata_cache.h"
 
 #include <vector>
 
 #include "base/chromeos/chromeos_version.h"
 #include "base/file_util.h"
@@ skipping 813 matching lines @@
   }
 }
 
 void GDataCache::Store(const std::string& resource_id,
                        const std::string& md5,
                        const FilePath& source_path,
                        FileOperationType file_operation_type,
                        GDataFileError* error) {
   AssertOnSequencedWorkerPool();
   DCHECK(error);
 

[kinaba, 2012/08/13 05:06:30]

I think we can wrap whole these checks with:
if (file_operation_type == FILE_OPERATION_COPY) { ... }
since _MOVE doesn't consume additional disk space.

[Zachary Kuznia, 2012/08/13 05:12:26]

Done.

+  int64 file_size;
+  if (!file_util::GetFileSize(source_path, &file_size)) {
+    LOG(WARNING) << "Couldn't get file size for: " << source_path.value();
+    *error = GDATA_FILE_ERROR_FAILED;
+    return;
+  }
+
+  bool enough_space = false;
+  FreeDiskSpaceIfNeededFor(file_size, &enough_space);
+  if (!enough_space) {
+    *error = GDATA_FILE_ERROR_NO_SPACE;
+    return;
+  }
+
   FilePath dest_path;
   FilePath symlink_path;
   CacheSubDirectoryType sub_dir_type = CACHE_TYPE_TMP;
 
   // If file was previously pinned, store it in persistent dir and create
   // symlink in pinned dir.
   GDataCacheEntry cache_entry;
   if (GetCacheEntry(resource_id, md5, &cache_entry)) {  // File exists in cache.
     // If file is dirty or mounted, return error.
     if (cache_entry.is_dirty() || cache_entry.is_mounted()) {
@@ skipping 734 matching lines @@
     const GDataCacheEntry& cache_entry) {
   return cache_entry.is_persistent() ? CACHE_TYPE_PERSISTENT : CACHE_TYPE_TMP;
 }
 
 void SetFreeDiskSpaceGetterForTesting(FreeDiskSpaceGetterInterface* getter) {
   delete global_free_disk_getter_for_testing;  // Safe to delete NULL;
   global_free_disk_getter_for_testing = getter;
 }
 
 }  // namespace gdata