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

webkit/browser/fileapi/copy_or_move_file_validator_unittest.cc

Index: webkit/browser/fileapi/copy_or_move_file_validator_unittest.cc
diff --git a/webkit/browser/fileapi/copy_or_move_file_validator_unittest.cc b/webkit/browser/fileapi/copy_or_move_file_validator_unittest.cc
index b971fd57c1a9f0f72b517fc1a3e186d78b76e71c..6160a59f5c21247519c2b2ad3e850b3a88b7e28d 100644
--- a/webkit/browser/fileapi/copy_or_move_file_validator_unittest.cc
+++ b/webkit/browser/fileapi/copy_or_move_file_validator_unittest.cc
@@ -196,7 +196,9 @@ class TestCopyOrMoveFileValidatorFactory
    public:
     explicit TestCopyOrMoveFileValidator(bool all_valid)

[vandebo (ex-Chrome), 2013/07/02 20:15:02]

Looks like you need another parameter here and several more test cases.

[Greg Billock, 2013/07/03 16:08:27]

Yes. I haven't added all that yet. Making sure this approach is what we want and
I'll definitely add tests.

         : result_(all_valid ? base::PLATFORM_FILE_OK
-                            : base::PLATFORM_FILE_ERROR_SECURITY) {
+                            : base::PLATFORM_FILE_ERROR_SECURITY),
+          write_result_(all_valid ? base::PLATFORM_FILE_OK
+                                  : base::PLATFORM_FILE_ERROR_SECURITY) {
     }
     virtual ~TestCopyOrMoveFileValidator() {}
 
@@ -207,8 +209,17 @@ class TestCopyOrMoveFileValidatorFactory
           FROM_HERE, base::Bind(result_callback, result_));
     }
 
+    virtual void StartPostWriteValidation(
+        const base::FilePath& dest_platform_path,
+        const ResultCallback& result_callback) OVERRIDE {
+      // Post the result since a real validator must do work asynchronously.
+      base::MessageLoop::current()->PostTask(
+          FROM_HERE, base::Bind(result_callback, write_result_));
+    }
+
    private:
     base::PlatformFileError result_;
+    base::PlatformFileError write_result_;
 
     DISALLOW_COPY_AND_ASSIGN(TestCopyOrMoveFileValidator);
   };