diff options
author | erikkay@chromium.org <erikkay@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-08-18 16:07:55 +0000 |
---|---|---|
committer | erikkay@chromium.org <erikkay@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-08-18 16:07:55 +0000 |
commit | 33edeab42b851a5617a77caa1d65a3782a01ee7b (patch) | |
tree | d1baa44ff796c9f0c4881e9819e7e5b359489b1d /chrome/installer/setup/uninstall.cc | |
parent | f348b92e95dd67fef3a9a6109bc0465fb4078d6b (diff) | |
download | chromium_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.cc | 23 |
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]); |