Commit 507c96cf authored by Lei Zhang's avatar Lei Zhang Committed by Commit Bot

Remove various deprecated base::DeleteFile() calls.

Clean up some stragglers that called DeleteFile() without the base::
prefix, but still got the right function with argument-dependent lookup.

Bug: 1009837
Change-Id: I6a11c92cc9f928830ea772f7a998b4c23bf939fd
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2313043Reviewed-by: default avatarScott Violet <sky@chromium.org>
Commit-Queue: Lei Zhang <thestig@chromium.org>
Cr-Commit-Position: refs/heads/master@{#790998}
parent c7b22ce2
...@@ -165,7 +165,7 @@ bool ServiceProcessState::DeleteServiceProcessDataRegion() { ...@@ -165,7 +165,7 @@ bool ServiceProcessState::DeleteServiceProcessDataRegion() {
return false; return false;
if (PathExists(path)) if (PathExists(path))
return DeleteFile(path, false); return base::DeleteFile(path);
// Doesn't exist, so success. // Doesn't exist, so success.
return true; return true;
......
...@@ -15,14 +15,14 @@ ...@@ -15,14 +15,14 @@
namespace password_manager { namespace password_manager {
namespace { namespace {
// Synchronously deletes passwords directoy. // Synchronously deletes passwords directory.
void DeletePasswordsDirectorySync() { void DeletePasswordsDirectorySync() {
base::ScopedBlockingCall scoped_blocking_call(FROM_HERE, base::ScopedBlockingCall scoped_blocking_call(FROM_HERE,
base::BlockingType::MAY_BLOCK); base::BlockingType::MAY_BLOCK);
base::FilePath downloads_directory; base::FilePath downloads_directory;
if (GetPasswordsDirectory(&downloads_directory)) { if (GetPasswordsDirectory(&downloads_directory)) {
// It is assumed that deleting the directory always succeeds. // It is assumed that deleting the directory always succeeds.
DeleteFile(downloads_directory, /*recursive=*/true); base::DeletePathRecursively(downloads_directory);
} }
} }
} // namespace } // namespace
......
...@@ -67,7 +67,7 @@ bool MakeTestFile(const FilePath& from_path, FilePath* to_path) { ...@@ -67,7 +67,7 @@ bool MakeTestFile(const FilePath& from_path, FilePath* to_path) {
result = CopyFile(from_path, temp_file); result = CopyFile(from_path, temp_file);
if (!result) { if (!result) {
DeleteFile(temp_file, false); base::DeleteFile(temp_file);
return false; return false;
} }
......
...@@ -63,7 +63,7 @@ bool DevToolsStreamFile::InitOnFileSequenceIfNeeded() { ...@@ -63,7 +63,7 @@ bool DevToolsStreamFile::InitOnFileSequenceIfNeeded() {
LOG(ERROR) << "Failed to open temporary file: " << temp_path.value() << ", " LOG(ERROR) << "Failed to open temporary file: " << temp_path.value() << ", "
<< base::File::ErrorToString(file_.error_details()); << base::File::ErrorToString(file_.error_details());
had_errors_ = true; had_errors_ = true;
DeleteFile(temp_path, false); base::DeleteFile(temp_path);
return false; return false;
} }
return true; return true;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment