Add fileManagerPrivate.onCopyError event.

storage/browser/fileapi/file_system_operation.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef STORAGE_BROWSER_FILEAPI_FILE_SYSTEM_OPERATION_H_
 #define STORAGE_BROWSER_FILEAPI_FILE_SYSTEM_OPERATION_H_
 
 #include <vector>
 
 #include "base/callback.h"
@@ skipping 54 matching lines @@
 
   virtual ~FileSystemOperation() {}
 
   // Used for CreateFile(), etc. |result| is the return code of the operation.
   typedef base::Callback<void(base::File::Error result)> StatusCallback;
 
   // Called when an error had happened during operation.
   // |url| is the url of processed entry.
   // |error| is the error code of the failed operation.
   typedef base::Callback<void(const FileSystemURL& url,
                               base::File::Error error)> ErrorCallback;

[yawano, 2015/06/19 06:50:16]

@tzik: While I added this in http://crrev.com/1184343002, we are not using
ErrorCallback for now. Should we remove this?

If Yes, I'm going to add a patchset to remove it in this CL.
If No, I'm going to add a patchset to swap the |url| and |error| parameters of
ErrorCallback to keep consistent with CopyOrMoveErrorCallback.

I prefer to remove it since it's not used for now. WDYT?

 
+  // Called when an error had happened during copy operation.
+  // |error| is the error code of the failed copy operation.
+  // |source_url| is the url of source file.
+  // |destination_url| is the url of destination file.
+  //
+  // An error can be happened during a recursive operation. The urls of children
+  // files or directories are set to source_url and destination_url.
+  typedef base::Callback<void(base::File::Error error,
+                              const FileSystemURL& source_url,
+                              const FileSystemURL& destination_url)>
+      CopyOrMoveErrorCallback;
+
   // Used for GetMetadata(). |result| is the return code of the operation,
   // |file_info| is the obtained file info.
   typedef base::Callback<
       void(base::File::Error result,
            const base::File::Info& file_info)> GetMetadataCallback;
 
   // Used for OpenFile(). |on_close_callback| will be called after the file is
   // closed in the child process. It can be null, if no operation is needed on
   // closing a file.
   typedef base::Callback<
@@ skipping 157 matching lines @@
                                bool exclusive,
                                bool recursive,
                                const StatusCallback& callback) = 0;
 
   // Copies a file or directory from |src_path| to |dest_path|. If
   // |src_path| is a directory, the contents of |src_path| are copied to
   // |dest_path| recursively. A new file or directory is created at
   // |dest_path| as needed.
   // |option| specifies the minor behavior of Copy(). See CopyOrMoveOption's
   // comment for details.
+  // |continue_with_error| is whether it continues copy operation even when it
+  // fails to copy one of the files.
+  // |error_callback| is called when an error had happened during the operation.
   // |progress_callback| is periodically called to report the progress
   // update. See also the comment of CopyProgressCallback. This callback is
   // optional.
   //
   // For recursive case this internally creates new FileSystemOperations and
   // calls:
   // - ReadDirectory, CopyFileLocal and CreateDirectory
   //   for same-filesystem case, or
   // - ReadDirectory and CreateSnapshotFile on source filesystem and
   //   CopyInForeignFile and CreateDirectory on dest filesystem
   //   for cross-filesystem case.
   //
   virtual void Copy(const FileSystemURL& src_path,
                     const FileSystemURL& dest_path,
                     CopyOrMoveOption option,
+                    const bool continue_with_error,
+                    const CopyOrMoveErrorCallback& error_callback,
                     const CopyProgressCallback& progress_callback,
                     const StatusCallback& callback) = 0;
 
   // Moves a file or directory from |src_path| to |dest_path|. A new file
   // or directory is created at |dest_path| as needed.
   // |option| specifies the minor behavior of Copy(). See CopyOrMoveOption's
   // comment for details.
   //
   // For recursive case this internally creates new FileSystemOperations and
   // calls:
@@ skipping 203 matching lines @@
     kOperationOpenFile,
     kOperationCloseFile,
     kOperationGetLocalPath,
     kOperationCancel,
   };
 };
 
 }  // namespace storage
 
 #endif  // STORAGE_BROWSER_FILEAPI_FILE_SYSTEM_OPERATION_H_