In DownloadOperation use locally modified file info if dirty Drive cache exists.

In DownloadOperation use locally modified file info if dirty Drive cache exists.

BUG=245693
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=203940

[kinaba, 2013-06-03 09:45:32]

ptal.

The patch duplicates some code with drive::FileSystem which is bad,
but I don't think we can cleanly merge them until we rewrite the
FileSystem operation into the blocking-poolish style. So for now
I'd like to go in this form and try to clean up later.

[hashimoto, 2013-06-03 10:33:27]

lgtm

https://codereview.chromium.org/16278006/diff/2018/chrome/browser/chromeos/dr...
File chrome/browser/chromeos/drive/file_system/download_operation.cc (right):

https://codereview.chromium.org/16278006/diff/2018/chrome/browser/chromeos/dr...
chrome/browser/chromeos/drive/file_system/download_operation.cc:77: // be stored
in |entry|.
nit: Could you make a note about the fact that this code is a dup of code in
file_system.cc? Having a TODO might be also nice.

[kinaba, 2013-06-04 03:30:16]

https://codereview.chromium.org/16278006/diff/2018/chrome/browser/chromeos/dr...
File chrome/browser/chromeos/drive/file_system/download_operation.cc (right):

https://codereview.chromium.org/16278006/diff/2018/chrome/browser/chromeos/dr...
chrome/browser/chromeos/drive/file_system/download_operation.cc:77: // be stored
in |entry|.
On 2013/06/03 10:33:27, hashimoto wrote:
> nit: Could you make a note about the fact that this code is a dup of code in
> file_system.cc? Having a TODO might be also nice.

Done.

[commit-bot: I haz the power, 2013-06-04 03:30:53]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/kinaba@chromium.org/16278006/7001

[commit-bot: I haz the power, 2013-06-04 12:46:50]

Change committed as 203940