Commit dab51eaa authored by Sam McNally's avatar Sam McNally Committed by Commit Bot

Document that FileSystemFileUtil::CopyOrMoveFile can move directories.

One of the two implementations now supports moving directories so update
the documentation to mention that moving directories may be supported,
depending on the implementation.

Bug: 172187
Change-Id: Ie5fde919681fef8a80528d25fd7b9e5a48225013
Reviewed-on: https://chromium-review.googlesource.com/1186931Reviewed-by: default avatarHiroki Nakagawa <nhiroki@chromium.org>
Commit-Queue: Sam McNally <sammc@chromium.org>
Cr-Commit-Position: refs/heads/master@{#585714}
parent 158f9449
...@@ -125,14 +125,17 @@ class STORAGE_EXPORT FileSystemFileUtil { ...@@ -125,14 +125,17 @@ class STORAGE_EXPORT FileSystemFileUtil {
const FileSystemURL& url, const FileSystemURL& url,
int64_t length) = 0; int64_t length) = 0;
// Copies or moves a single file from |src_url| to |dest_url|. // Copies a single file or moves a single file or directory from |src_url| to
// The filesystem type of |src_url| and |dest_url| MUST be same. // |dest_url|. Whether moving a directory is supported is
// For |option|, please see file_system_operation.h // implementation-defined. The filesystem type of |src_url| and |dest_url|
// MUST be same. For |option|, please see file_system_operation.h
// //
// This returns: // This returns:
// - File::FILE_ERROR_NOT_FOUND if |src_url| // - File::FILE_ERROR_NOT_FOUND if |src_url|
// or the parent directory of |dest_url| does not exist. // or the parent directory of |dest_url| does not exist.
// - File::FILE_ERROR_NOT_A_FILE if |src_url| exists but is not a file. // - File::FILE_ERROR_NOT_A_FILE if |src_url| exists but is not a file and the
// operation is copy or the implementation does not support moving
// directories.
// - File::FILE_ERROR_INVALID_OPERATION if |dest_url| exists and // - File::FILE_ERROR_INVALID_OPERATION if |dest_url| exists and
// is not a file. // is not a file.
// - File::FILE_ERROR_FAILED if |dest_url| does not exist and // - File::FILE_ERROR_FAILED if |dest_url| does not exist and
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment