summaryrefslogtreecommitdiffstats
path: root/chrome/installer/setup/uninstall.cc
diff options
context:
space:
mode:
authorerikkay@chromium.org <erikkay@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2009-08-18 16:07:55 +0000
committererikkay@chromium.org <erikkay@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2009-08-18 16:07:55 +0000
commit33edeab42b851a5617a77caa1d65a3782a01ee7b (patch)
treed1baa44ff796c9f0c4881e9819e7e5b359489b1d /chrome/installer/setup/uninstall.cc
parentf348b92e95dd67fef3a9a6109bc0465fb4078d6b (diff)
downloadchromium_src-33edeab42b851a5617a77caa1d65a3782a01ee7b.zip
chromium_src-33edeab42b851a5617a77caa1d65a3782a01ee7b.tar.gz
chromium_src-33edeab42b851a5617a77caa1d65a3782a01ee7b.tar.bz2
Renames the function CreateTemporaryFilename to CreateTemporaryFile and track down all callers, also removes the
deprecated function that uses std::wstring. BUG=3078 (http://crbug.com/3078) TEST=run base_unittests, installer_util_unittests, net_unittests, setup_unittests, and unit_tests. Review URL: http://codereview.chromium.org/164537 Patch from Thiago Farina. git-svn-id: svn://svn.chromium.org/chrome/trunk/src@23631 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/installer/setup/uninstall.cc')
-rw-r--r--chrome/installer/setup/uninstall.cc23
1 files changed, 13 insertions, 10 deletions
diff --git a/chrome/installer/setup/uninstall.cc b/chrome/installer/setup/uninstall.cc
index 460c4a2..206cf72 100644
--- a/chrome/installer/setup/uninstall.cc
+++ b/chrome/installer/setup/uninstall.cc
@@ -158,22 +158,25 @@ bool DeleteFilesAndFolders(const std::wstring& exe_path, bool system_uninstall,
install_path, installed_version.GetString()));
file_util::AppendToPath(&setup_exe, file_util::GetFilenameFromPath(exe_path));
- std::wstring temp_file;
- if (!file_util::CreateTemporaryFileName(&temp_file))
+ FilePath temp_file;
+ if (!file_util::CreateTemporaryFile(&temp_file)) {
LOG(ERROR) << "Failed to create temporary file for setup.exe.";
- else
- file_util::Move(setup_exe, temp_file);
+ } else {
+ FilePath setup_exe_path = FilePath::FromWStringHack(setup_exe);
+ file_util::Move(setup_exe_path, temp_file);
+ }
// Move the browser's persisted local state
FilePath user_local_state;
if (chrome::GetDefaultUserDataDirectory(&user_local_state)) {
- std::wstring user_local_file(
- user_local_state.Append(chrome::kLocalStateFilename).value());
+ FilePath user_local_file(
+ user_local_state.Append(chrome::kLocalStateFilename));
- if (!file_util::CreateTemporaryFileName(local_state_path))
+ FilePath path = FilePath::FromWStringHack(*local_state_path);
+ if (!file_util::CreateTemporaryFile(&path))
LOG(ERROR) << "Failed to create temporary file for Local State.";
- else
- file_util::CopyFile(user_local_file, *local_state_path);
+ else
+ file_util::CopyFile(user_local_file, path);
}
LOG(INFO) << "Deleting install path " << install_path;
@@ -317,7 +320,7 @@ bool installer_setup::DeleteChromeRegistrationKeys(HKEY root,
file_util::AppendToPath(&app_path_key, installer_util::kChromeExe);
DeleteRegistryKey(key, app_path_key);
- //Cleanup OpenWithList
+ // Cleanup OpenWithList
for (int i = 0; ShellUtil::kFileAssociations[i] != NULL; i++) {
std::wstring open_with_key(ShellUtil::kRegClasses);
file_util::AppendToPath(&open_with_key, ShellUtil::kFileAssociations[i]);