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

Unified Diff: chrome/browser/chromeos/login/wallpaper_manager.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/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 4fdcecec427d00380c9cb3da5cf4e2f052d56680..875d61bbd39c5286374a27a04b18764114ce2b55 100644
--- a/chrome/browser/chromeos/login/wallpaper_manager.cc
+++ b/chrome/browser/chromeos/login/wallpaper_manager.cc
@@ -393,7 +393,7 @@ void WallpaperManager::ResizeAndSaveWallpaper(const UserImage& wallpaper,
if (layout == ash::WALLPAPER_LAYOUT_CENTER) {
// TODO(bshe): Generates cropped custom wallpaper for CENTER layout.
if (file_util::PathExists(path))
- file_util::Delete(path, false);
+ base::Delete(path, false);
return;
}
scoped_refptr<base::RefCountedBytes> data;
@@ -675,7 +675,7 @@ void WallpaperManager::DeleteAllExcept(const base::FilePath& path) {
for (base::FilePath current = files.Next(); !current.empty();
current = files.Next()) {
if (current != path)
- file_util::Delete(current, false);
+ base::Delete(current, false);
}
}
}
@@ -687,8 +687,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 (!file_util::Delete(path, true) &&
- !file_util::Delete(path.AddExtension(".png"), false)) {
+ if (!base::Delete(path, true) &&
+ !base::Delete(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