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

chrome/browser/media_galleries/fileapi/media_file_validator_factory.cc

 // 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.
 
 #include "chrome/browser/media_galleries/fileapi/media_file_validator_factory.h"
 
 #include "base/files/file_path.h"
 #include "base/platform_file.h"
 #include "chrome/browser/media_galleries/fileapi/supported_image_type_validator.h"
 #include "webkit/browser/fileapi/copy_or_move_file_validator.h"
 #include "webkit/browser/fileapi/file_system_url.h"
+#include "webkit/common/blob/shareable_file_reference.h"
 
 namespace chrome {
 
 namespace {
 
 class InvalidFileValidator : public fileapi::CopyOrMoveFileValidator {
  public:
   virtual ~InvalidFileValidator() {}
   virtual void StartValidation(

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

Should we rename this to StartPreWriteValidation() ?

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

Not opposed.

[vandebo (ex-Chrome), 2013/07/12 15:08:15]

But not enthusiastic either?

[Greg Billock, 2013/07/12 18:02:19]

This stage seems more like validation. I can't think of a better name for
post-write operation, though. It's a validation from the perspective of the
fileapi code. I guess I'm hampered by thinking too much about what we anticipate
doing in these methods. Let's rename it.

       const fileapi::CopyOrMoveFileValidator::ResultCallback&
           result_callback) OVERRIDE {
     result_callback.Run(base::PLATFORM_FILE_ERROR_SECURITY);
   }
 
+  virtual void StartPostWriteValidation(
+      const base::FilePath& dest_platform_path,
+      scoped_refptr<webkit_blob::ShareableFileReference> file_ref,
+      const fileapi::CopyOrMoveFileValidator::ResultCallback&
+          result_callback) OVERRIDE {
+    result_callback.Run(base::PLATFORM_FILE_ERROR_SECURITY);
+  }
+
  private:
   friend class chrome::MediaFileValidatorFactory;
 
   InvalidFileValidator() {}
 
   DISALLOW_COPY_AND_ASSIGN(InvalidFileValidator);
 };
 
 }  // namespace
 
 MediaFileValidatorFactory::MediaFileValidatorFactory() {}
 MediaFileValidatorFactory::~MediaFileValidatorFactory() {}
 
 fileapi::CopyOrMoveFileValidator*
 MediaFileValidatorFactory::CreateCopyOrMoveFileValidator(
     const fileapi::FileSystemURL& src,
     const base::FilePath& platform_path) {
   base::FilePath src_path = src.virtual_path();
   if (SupportedImageTypeValidator::SupportsFileType(src_path))
     return new SupportedImageTypeValidator(platform_path);
   // TODO(vandebo): Support other file types.
 
   return new InvalidFileValidator();
 }
 
 }  // namespace chrome