Add truncation to StreamCopyOrMoveImpl and test case.

Add truncation to StreamCopyOrMoveImpl and test case.

BUG=410126
TEST=content_unittests

Committed: https://crrev.com/e5ce274558eaf7231b9294318cc1db2bd9c553ba
Cr-Commit-Position: refs/heads/master@{#293082}

[iseki, 2014-09-03 01:59:27]

iseki@chromium.org changed reviewers:
+ tzik@chromium.org

[iseki, 2014-09-03 01:59:27]

Please take a look.

[tzik, 2014-09-03 02:07:43]

https://codereview.chromium.org/536453007/diff/1/content/browser/fileapi/file...
File content/browser/fileapi/file_system_operation_impl_unittest.cc (right):

https://codereview.chromium.org/536453007/diff/1/content/browser/fileapi/file...
content/browser/fileapi/file_system_operation_impl_unittest.cc:1302:
base::RunLoop().RunUntilIdle();
This doesn't run the modified part of the code untile we enable the stream copy
for sandboxed filesystem...

Can we use Run() instead of RunUntilIdle()?

https://codereview.chromium.org/536453007/diff/1/webkit/browser/fileapi/copy_...
File webkit/browser/fileapi/copy_or_move_operation_delegate.cc (right):

https://codereview.chromium.org/536453007/diff/1/webkit/browser/fileapi/copy_...
webkit/browser/fileapi/copy_or_move_operation_delegate.cc:456: }
error == FILE_OK case handling?

[iseki, 2014-09-03 03:28:21]

Thank you for your review:)

https://codereview.chromium.org/536453007/diff/1/content/browser/fileapi/file...
File content/browser/fileapi/file_system_operation_impl_unittest.cc (right):

https://codereview.chromium.org/536453007/diff/1/content/browser/fileapi/file...
content/browser/fileapi/file_system_operation_impl_unittest.cc:1302:
base::RunLoop().RunUntilIdle();
On 2014/09/03 02:07:43, tzik wrote:
> This doesn't run the modified part of the code untile we enable the stream
copy
> for sandboxed filesystem...
> 
> Can we use Run() instead of RunUntilIdle()?

Done.

https://codereview.chromium.org/536453007/diff/1/webkit/browser/fileapi/copy_...
File webkit/browser/fileapi/copy_or_move_operation_delegate.cc (right):

https://codereview.chromium.org/536453007/diff/1/webkit/browser/fileapi/copy_...
webkit/browser/fileapi/copy_or_move_operation_delegate.cc:456: }
On 2014/09/03 02:07:43, tzik wrote:
> error == FILE_OK case handling?

Done.

[tzik, 2014-09-03 04:26:11]

lgtm

[iseki, 2014-09-03 05:08:38]

The CQ bit was checked by iseki@chromium.org

[commit-bot: I haz the power, 2014-09-03 05:09:46]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/iseki@chromium.org/536453007/20001

[commit-bot: I haz the power, 2014-09-03 06:12:54]

Committed patchset #2 (id:20001) as 2bdf4486ff836859b682b5cde420be5cbf8f2bbb

[commit-bot: I haz the power, 2014-09-10 03:23:54]

Patchset 2 (id:??) landed as
https://crrev.com/e5ce274558eaf7231b9294318cc1db2bd9c553ba
Cr-Commit-Position: refs/heads/master@{#293082}