[FileSystem] Add another copy-or-move validation hook for post-write.

webkit/browser/fileapi/copy_or_move_file_validator.h

 // Copyright 2013 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 WEBKIT_BROWSER_FILEAPI_COPY_OR_MOVE_FILE_VALIDATOR_H_
 #define WEBKIT_BROWSER_FILEAPI_COPY_OR_MOVE_FILE_VALIDATOR_H_
 
 #include "base/callback.h"
 #include "base/platform_file.h"
 #include "webkit/browser/webkit_storage_browser_export.h"
 
 namespace base {
 class FilePath;
 }
 
+namespace webkit_blob {
+class ShareableFileReference;
+}
+
 namespace fileapi {
 
 class FileSystemURL;
 
 class WEBKIT_STORAGE_BROWSER_EXPORT CopyOrMoveFileValidator {
  public:
   // Callback that is invoked when validation completes. A result of
   // base::PLATFORM_FILE_OK means the file validated.
   typedef base::Callback<void(base::PlatformFileError result)> ResultCallback;
 
   virtual ~CopyOrMoveFileValidator() {}
 
+  // Called on a source file before copying or moving to the final
+  // destination.
   virtual void StartValidation(const ResultCallback& result_callback) = 0;
+
+  // Called on a destination file after copying or moving to the final
+  // destination. Suitable for running Anti-Virus checks. The |file_ref| is
+  // a refcounted keepalive token for the destination file which may or
+  // may not be present. Implementations should maintain the |file_ref| until

[vandebo (ex-Chrome), 2013/07/10 15:52:46]

Hmm, can the caller manage the lifetime of the file_ref so that the validators
don't have to worry about it?

[Greg Billock, 2013/07/11 22:54:35]

There was a good spot you found to manage this. Removing it.

+  // they finish and return in order to make sure the file is not prematurely
+  // deleted from under them.
+  virtual void StartPostWriteValidation(
+      const base::FilePath& dest_platform_path,
+      scoped_refptr<webkit_blob::ShareableFileReference> file_ref,
+      const ResultCallback& result_callback) = 0;
 };
 
 class CopyOrMoveFileValidatorFactory {
  public:
   virtual ~CopyOrMoveFileValidatorFactory() {}
 
   // This method must always return a non-NULL validator. |src_url| is needed
   // in addition to |platform_path| because in the obfuscated file system
   // case, |platform_path| will be an obfuscated filename and extension.
   virtual CopyOrMoveFileValidator* CreateCopyOrMoveFileValidator(
       const FileSystemURL& src_url,
       const base::FilePath& platform_path) = 0;
 };
 
 }  // namespace fileapi
 
 #endif  // WEBKIT_BROWSER_FILEAPI_COPY_OR_MOVE_FILE_VALIDATOR_H_