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

Unified Diff: webkit/fileapi/obfuscated_file_util.cc

Issue 14075016: Change some snapshot- or temporary-file related changes to use ScopedFile (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebased Created 7 years, 8 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « webkit/fileapi/obfuscated_file_util.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webkit/fileapi/obfuscated_file_util.cc
diff --git a/webkit/fileapi/obfuscated_file_util.cc b/webkit/fileapi/obfuscated_file_util.cc
index 84dd1f6cf335785e6b41fc132f9d0b2802064337..8103e62c432775b4ef697ba74a1a95912cd980ba 100644
--- a/webkit/fileapi/obfuscated_file_util.cc
+++ b/webkit/fileapi/obfuscated_file_util.cc
@@ -857,22 +857,19 @@ PlatformFileError ObfuscatedFileUtil::DeleteDirectory(
return base::PLATFORM_FILE_OK;
}
-base::PlatformFileError ObfuscatedFileUtil::CreateSnapshotFile(
+webkit_blob::ScopedFile ObfuscatedFileUtil::CreateSnapshotFile(
FileSystemOperationContext* context,
const FileSystemURL& url,
+ base::PlatformFileError* error,
base::PlatformFileInfo* file_info,
- base::FilePath* platform_path,
- SnapshotFilePolicy* policy) {
- DCHECK(policy);
+ base::FilePath* platform_path) {
// We're just returning the local file information.
- *policy = kSnapshotFileLocal;
- base::PlatformFileError error = GetFileInfo(
- context, url, file_info, platform_path);
- if (error == base::PLATFORM_FILE_OK && file_info->is_directory) {
+ *error = GetFileInfo(context, url, file_info, platform_path);
+ if (*error == base::PLATFORM_FILE_OK && file_info->is_directory) {
*file_info = base::PlatformFileInfo();
- return base::PLATFORM_FILE_ERROR_NOT_A_FILE;
+ *error = base::PLATFORM_FILE_ERROR_NOT_A_FILE;
}
- return error;
+ return webkit_blob::ScopedFile();
}
scoped_ptr<FileSystemFileUtil::AbstractFileEnumerator>
« no previous file with comments | « webkit/fileapi/obfuscated_file_util.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698