fileapi: Pass virtual path to copy validator.

webkit/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_FILEAPI_COPY_OR_MOVE_FILE_VALIDATOR_H_
 #define WEBKIT_FILEAPI_COPY_OR_MOVE_FILE_VALIDATOR_H_
 
 #include "base/callback.h"
 #include "base/files/file_path.h"
 #include "base/platform_file.h"
 
 namespace fileapi {
 
 class 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() {}
 
   virtual void StartValidation(const ResultCallback& result_callback) = 0;
 };
 
 class CopyOrMoveFileValidatorFactory {
  public:
   virtual ~CopyOrMoveFileValidatorFactory() {}
 
-  // This method must always return a non-NULL validator.
+  // This method must always return a non-NULL validator. |virtual_path| is
+  // the logical file name and |platform_path| is the real filename as stored
+  // on disk. Generally the two will match, except in the case of an obfuscated
+  // file system.
   virtual CopyOrMoveFileValidator* CreateCopyOrMoveFileValidator(
+      const base::FilePath& virtual_path,

[kinuko, 2013/04/18 03:32:00]

Does it make sense if we just pass the source FileSystemURL instead of
virtual_path here? (The term 'virtual_path' is a bit overloaded and may not be
always obvious what it really means)

       const base::FilePath& platform_path) = 0;
 };
 
 }  // namespace fileapi
 
 #endif  // WEBKIT_FILEAPI_COPY_OR_MOVE_FILE_VALIDATOR_H_