Check for space before storing files to the cache.

Check for space before storing files to the cache.


R=kinaba@chromium.org
BUG=126980
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=151238

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

http://codereview.chromium.org/10831279/diff/1/chrome/browser/chromeos/gdata/...
File chrome/browser/chromeos/gdata/gdata_cache.cc (right):

http://codereview.chromium.org/10831279/diff/1/chrome/browser/chromeos/gdata/...
chrome/browser/chromeos/gdata/gdata_cache.cc:834: 
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]

http://codereview.chromium.org/10831279/diff/1/chrome/browser/chromeos/gdata/...
File chrome/browser/chromeos/gdata/gdata_cache.cc (right):

http://codereview.chromium.org/10831279/diff/1/chrome/browser/chromeos/gdata/...
chrome/browser/chromeos/gdata/gdata_cache.cc:834: 
On 2012/08/13 05:06:30, kinaba wrote:
> I think we can wrap whole these checks with:
> if (file_operation_type == FILE_OPERATION_COPY) { ... }
> since _MOVE doesn't consume additional disk space.

Done.

[kinaba, 2012-08-13 05:21:21]

lgtm

[commit-bot: I haz the power, 2012-08-13 05:37:49]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/zork@chromium.org/10831279/5001

[commit-bot: I haz the power, 2012-08-13 08:35:24]

Change committed as 151238