summaryrefslogtreecommitdiffstats
path: root/chrome/installer
diff options
context:
space:
mode:
authortfarina@chromium.org <tfarina@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2010-05-24 03:49:01 +0000
committertfarina@chromium.org <tfarina@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2010-05-24 03:49:01 +0000
commit08729cc56a5177d795ada1723ea1bf70557df0ee (patch)
treea056473e78c206403b53489913559cf92b3736bc /chrome/installer
parent0556cfbcf7cb03445a628d6b71a9536f762cc3c3 (diff)
downloadchromium_src-08729cc56a5177d795ada1723ea1bf70557df0ee.zip
chromium_src-08729cc56a5177d795ada1723ea1bf70557df0ee.tar.gz
chromium_src-08729cc56a5177d795ada1723ea1bf70557df0ee.tar.bz2
Use FilePath::DirName instead of the deprecated file_util::GetDirectoryFromPath.
BUG=24672 TEST=compiles Review URL: http://codereview.chromium.org/2110021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@48014 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/installer')
-rw-r--r--chrome/installer/setup/uninstall.cc21
1 files changed, 11 insertions, 10 deletions
diff --git a/chrome/installer/setup/uninstall.cc b/chrome/installer/setup/uninstall.cc
index 79ccdbe..4338cb6 100644
--- a/chrome/installer/setup/uninstall.cc
+++ b/chrome/installer/setup/uninstall.cc
@@ -1,4 +1,4 @@
-// Copyright (c) 2009 The Chromium Authors. All rights reserved.
+// Copyright (c) 2010 The Chromium Authors. All rights reserved.
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
//
@@ -139,20 +139,21 @@ bool ScheduleParentAndGrandparentForDeletion(const FilePath& path) {
}
// Deletes empty parent & empty grandparent dir of given path.
-bool DeleteEmptyParentDir(const std::wstring& path) {
+bool DeleteEmptyParentDir(const FilePath& path) {
bool ret = true;
- std::wstring parent_dir = file_util::GetDirectoryFromPath(path);
- if (!parent_dir.empty() && file_util::IsDirectoryEmpty(parent_dir)) {
+ FilePath parent_dir = path.DirName();
+ if (!parent_dir.empty() && file_util::IsDirectoryEmpty(parent_dir.value())) {
if (!file_util::Delete(parent_dir, true)) {
ret = false;
- LOG(ERROR) << "Failed to delete folder: " << parent_dir;
+ LOG(ERROR) << "Failed to delete folder: " << parent_dir.value();
}
- parent_dir = file_util::GetDirectoryFromPath(parent_dir);
- if (!parent_dir.empty() && file_util::IsDirectoryEmpty(parent_dir)) {
+ parent_dir = parent_dir.DirName();
+ if (!parent_dir.empty() &&
+ file_util::IsDirectoryEmpty(parent_dir.value())) {
if (!file_util::Delete(parent_dir, true)) {
ret = false;
- LOG(ERROR) << "Failed to delete folder: " << parent_dir;
+ LOG(ERROR) << "Failed to delete folder: " << parent_dir.value();
}
}
}
@@ -257,7 +258,7 @@ DeleteResult DeleteFilesAndFolders(const std::wstring& exe_path,
if (result == DELETE_REQUIRES_REBOOT) {
ScheduleParentAndGrandparentForDeletion(user_local_state);
} else {
- DeleteEmptyParentDir(user_local_state.value());
+ DeleteEmptyParentDir(user_local_state);
}
}
@@ -269,7 +270,7 @@ DeleteResult DeleteFilesAndFolders(const std::wstring& exe_path,
} else {
// Now check and delete if the parent directories are empty
// For example Google\Chrome or Chromium
- DeleteEmptyParentDir(install_path);
+ DeleteEmptyParentDir(FilePath(install_path));
}
return result;
}