Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(276)

Side by Side Diff: webkit/browser/fileapi/copy_or_move_file_validator.h

Issue 18565002: [FileSystem] Add another copy-or-move validation hook for post-write. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Simplify dest validation callbacks and delete dest file on error Created 7 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef WEBKIT_BROWSER_FILEAPI_COPY_OR_MOVE_FILE_VALIDATOR_H_ 5 #ifndef WEBKIT_BROWSER_FILEAPI_COPY_OR_MOVE_FILE_VALIDATOR_H_
6 #define WEBKIT_BROWSER_FILEAPI_COPY_OR_MOVE_FILE_VALIDATOR_H_ 6 #define WEBKIT_BROWSER_FILEAPI_COPY_OR_MOVE_FILE_VALIDATOR_H_
7 7
8 #include "base/callback.h" 8 #include "base/callback.h"
9 #include "base/platform_file.h" 9 #include "base/platform_file.h"
10 #include "webkit/browser/webkit_storage_browser_export.h" 10 #include "webkit/browser/webkit_storage_browser_export.h"
11 11
12 namespace base { 12 namespace base {
13 class FilePath; 13 class FilePath;
14 } 14 }
15 15
16 namespace fileapi { 16 namespace fileapi {
17 17
18 class FileSystemURL; 18 class FileSystemURL;
19 19
20 class WEBKIT_STORAGE_BROWSER_EXPORT CopyOrMoveFileValidator { 20 class WEBKIT_STORAGE_BROWSER_EXPORT CopyOrMoveFileValidator {
21 public: 21 public:
22 // Callback that is invoked when validation completes. A result of 22 // Callback that is invoked when validation completes. A result of
23 // base::PLATFORM_FILE_OK means the file validated. 23 // base::PLATFORM_FILE_OK means the file validated.
24 typedef base::Callback<void(base::PlatformFileError result)> ResultCallback; 24 typedef base::Callback<void(base::PlatformFileError result)> ResultCallback;
25 25
26 virtual ~CopyOrMoveFileValidator() {} 26 virtual ~CopyOrMoveFileValidator() {}
27 27
28 // Called on a source file before copying or moving to the final
29 // destination.
28 virtual void StartValidation(const ResultCallback& result_callback) = 0; 30 virtual void StartValidation(const ResultCallback& result_callback) = 0;
31
32 // Called on a destination file after copying or moving to the final
33 // destination. Suitable for running AV checks.
kinuko 2013/07/03 04:53:08 sorry, what does this 'AV' refer?
Greg Billock 2013/07/03 16:08:27 Anti-Virus. Will expand.
34 virtual void StartPostWriteValidation(
35 const base::FilePath& dest_platform_path,
36 const ResultCallback& result_callback) = 0;
29 }; 37 };
30 38
31 class CopyOrMoveFileValidatorFactory { 39 class CopyOrMoveFileValidatorFactory {
32 public: 40 public:
33 virtual ~CopyOrMoveFileValidatorFactory() {} 41 virtual ~CopyOrMoveFileValidatorFactory() {}
34 42
35 // This method must always return a non-NULL validator. |src_url| is needed 43 // This method must always return a non-NULL validator. |src_url| is needed
36 // in addition to |platform_path| because in the obfuscated file system 44 // in addition to |platform_path| because in the obfuscated file system
37 // case, |platform_path| will be an obfuscated filename and extension. 45 // case, |platform_path| will be an obfuscated filename and extension.
38 virtual CopyOrMoveFileValidator* CreateCopyOrMoveFileValidator( 46 virtual CopyOrMoveFileValidator* CreateCopyOrMoveFileValidator(
39 const FileSystemURL& src_url, 47 const FileSystemURL& src_url,
40 const base::FilePath& platform_path) = 0; 48 const base::FilePath& platform_path) = 0;
41 }; 49 };
42 50
43 } // namespace fileapi 51 } // namespace fileapi
44 52
45 #endif // WEBKIT_BROWSER_FILEAPI_COPY_OR_MOVE_FILE_VALIDATOR_H_ 53 #endif // WEBKIT_BROWSER_FILEAPI_COPY_OR_MOVE_FILE_VALIDATOR_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698