diff options
author | tfarina@chromium.org <tfarina@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-05-24 03:49:01 +0000 |
---|---|---|
committer | tfarina@chromium.org <tfarina@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-05-24 03:49:01 +0000 |
commit | 08729cc56a5177d795ada1723ea1bf70557df0ee (patch) | |
tree | a056473e78c206403b53489913559cf92b3736bc /chrome/installer | |
parent | 0556cfbcf7cb03445a628d6b71a9536f762cc3c3 (diff) | |
download | chromium_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.cc | 21 |
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; } |