Re-implement IsolateFileUtil tests on top of new cross-fs operation architecture

Re-implement IsolateFileUtil tests on top of new cross-fs operation architecture

Deprecate FileUtilHelper::{Copy,Move} and recursive file enumerator
in favor of AsyncFileTestHelper::{Copy,Move,ReadDirectory}

BUG=146215, 174576
TEST=IsolatedFileUtilTest.CopyOut*

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

[kinuko, 2013-02-13 12:07:24]

Can you review?  Another cleanup patch for the cross-filesystem operation
change.

[tzik, 2013-02-13 13:18:48]

lgtm

https://codereview.chromium.org/12212030/diff/1/webkit/fileapi/isolated_file_...
File webkit/fileapi/isolated_file_util_unittest.cc (right):

https://codereview.chromium.org/12212030/diff/1/webkit/fileapi/isolated_file_...
webkit/fileapi/isolated_file_util_unittest.cc:78:
root.virtual_path().AppendRelativePath(url.virtual_path(), &relative);
could you check the return value of this?

[nhiroki, 2013-02-14 02:18:41]

lgtm

[kinuko, 2013-02-14 09:12:51]

Thanks, updated.

https://codereview.chromium.org/12212030/diff/1/webkit/fileapi/isolated_file_...
File webkit/fileapi/isolated_file_util_unittest.cc (right):

https://codereview.chromium.org/12212030/diff/1/webkit/fileapi/isolated_file_...
webkit/fileapi/isolated_file_util_unittest.cc:78:
root.virtual_path().AppendRelativePath(url.virtual_path(), &relative);
On 2013/02/13 13:18:48, tzik wrote:
> could you check the return value of this?

Done.

[commit-bot: I haz the power, 2013-02-14 09:25:52]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/kinuko@chromium.org/12212030/19001