Check for space before storing files to the cache.

chrome/browser/chromeos/gdata/gdata_cache.cc

Index: chrome/browser/chromeos/gdata/gdata_cache.cc
diff --git a/chrome/browser/chromeos/gdata/gdata_cache.cc b/chrome/browser/chromeos/gdata/gdata_cache.cc
index 059e4f0d29950d36a42058dc4062269ac301ec97..f728f76d6dccb42b881b8515b64bec3fba74e550 100644
--- a/chrome/browser/chromeos/gdata/gdata_cache.cc
+++ b/chrome/browser/chromeos/gdata/gdata_cache.cc
@@ -832,6 +832,20 @@ void GDataCache::Store(const std::string& resource_id,
   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;