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

Unified Diff: chrome/common/extensions/extension_file_util.cc

Issue 16950028: Move file_util::Delete to the base namespace (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 6 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
Index: chrome/common/extensions/extension_file_util.cc
diff --git a/chrome/common/extensions/extension_file_util.cc b/chrome/common/extensions/extension_file_util.cc
index ce529b5a13107a83765257963f45a4b3d4a12ef9..f6f4a142ac10bc29d23d9e21f7c608caf11ebb6d 100644
--- a/chrome/common/extensions/extension_file_util.cc
+++ b/chrome/common/extensions/extension_file_util.cc
@@ -125,7 +125,7 @@ void UninstallExtension(const base::FilePath& extensions_dir,
// We don't care about the return value. If this fails (and it can, due to
// plugins that aren't unloaded yet), it will get cleaned up by
// ExtensionService::GarbageCollectExtensions.
- file_util::Delete(extensions_dir.AppendASCII(id), true); // recursive.
+ base::Delete(extensions_dir.AppendASCII(id), true); // recursive.
}
scoped_refptr<Extension> LoadExtension(const base::FilePath& extension_path,
@@ -345,7 +345,7 @@ void GarbageCollectExtensions(
// Clean up temporary files left if Chrome crashed or quit in the middle
// of an extension install.
if (basename.value() == kTempDirectoryName) {
- file_util::Delete(extension_path, true); // Recursive
+ base::Delete(extension_path, true); // Recursive
continue;
}
@@ -362,7 +362,7 @@ void GarbageCollectExtensions(
"directory: " << basename.value();
DVLOG(1) << "Deleting invalid extension directory "
<< extension_path.value() << ".";
- file_util::Delete(extension_path, true); // Recursive.
+ base::Delete(extension_path, true); // Recursive.
continue;
}
@@ -375,7 +375,7 @@ void GarbageCollectExtensions(
if (iter_pair.first == iter_pair.second) {
DVLOG(1) << "Deleting unreferenced install for directory "
<< extension_path.LossyDisplayName() << ".";
- file_util::Delete(extension_path, true); // Recursive.
+ base::Delete(extension_path, true); // Recursive.
continue;
}
@@ -396,7 +396,7 @@ void GarbageCollectExtensions(
if (!knownVersion) {
DVLOG(1) << "Deleting old version for directory "
<< version_dir.LossyDisplayName() << ".";
- file_util::Delete(version_dir, true); // Recursive.
+ base::Delete(version_dir, true); // Recursive.
}
}
}
@@ -570,7 +570,7 @@ base::FilePath GetInstallTempDir(const base::FilePath& extensions_dir) {
}
void DeleteFile(const base::FilePath& path, bool recursive) {
- file_util::Delete(path, recursive);
+ base::Delete(path, recursive);
}
} // namespace extension_file_util
« no previous file with comments | « chrome/common/extensions/extension_file_util.h ('k') | chrome/common/extensions/extension_file_util_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698