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

Unified Diff: chrome/browser/chromeos/login/wallpaper_manager.cc

Issue 18584011: Rename base::Delete to base::DeleteFile (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/chromeos/login/wallpaper_manager.cc
diff --git a/chrome/browser/chromeos/login/wallpaper_manager.cc b/chrome/browser/chromeos/login/wallpaper_manager.cc
index d5a7bbf75cfb3b37bc082d0f75f8b0c203430f0e..8957ecc8b4769d638eb4998455fc7a1529734264 100644
--- a/chrome/browser/chromeos/login/wallpaper_manager.cc
+++ b/chrome/browser/chromeos/login/wallpaper_manager.cc
@@ -394,7 +394,7 @@ void WallpaperManager::ResizeAndSaveWallpaper(const UserImage& wallpaper,
if (layout == ash::WALLPAPER_LAYOUT_CENTER) {
// TODO(bshe): Generates cropped custom wallpaper for CENTER layout.
if (base::PathExists(path))
- base::Delete(path, false);
+ base::DeleteFile(path, false);
return;
}
scoped_refptr<base::RefCountedBytes> data;
@@ -676,7 +676,7 @@ void WallpaperManager::DeleteAllExcept(const base::FilePath& path) {
for (base::FilePath current = files.Next(); !current.empty();
current = files.Next()) {
if (current != path)
- base::Delete(current, false);
+ base::DeleteFile(current, false);
}
}
}
@@ -688,8 +688,8 @@ void WallpaperManager::DeleteWallpaperInList(
base::FilePath path = *it;
// Some users may still have legacy wallpapers with png extension. We need
// to delete these wallpapers too.
- if (!base::Delete(path, true) &&
- !base::Delete(path.AddExtension(".png"), false)) {
+ if (!base::DeleteFile(path, true) &&
+ !base::DeleteFile(path.AddExtension(".png"), false)) {
LOG(ERROR) << "Failed to remove user wallpaper at " << path.value();
}
}
« no previous file with comments | « chrome/browser/chromeos/login/user_image_manager_impl.cc ('k') | chrome/browser/chromeos/policy/app_pack_updater.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698