diff options
author | agl@chromium.org <agl@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-03-17 15:58:27 +0000 |
---|---|---|
committer | agl@chromium.org <agl@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-03-17 15:58:27 +0000 |
commit | 8c30a48a97f132e99a1b07df90a231eabda4c9ed (patch) | |
tree | e67c563c3db831d98f4f4436ed7f0cb62b6dcfff /chrome/installer/setup/install.cc | |
parent | 8f578533569695a7458ecd2d5cfe7dc104a58485 (diff) | |
download | chromium_src-8c30a48a97f132e99a1b07df90a231eabda4c9ed.zip chromium_src-8c30a48a97f132e99a1b07df90a231eabda4c9ed.tar.gz chromium_src-8c30a48a97f132e99a1b07df90a231eabda4c9ed.tar.bz2 |
Revert "Change mini_installer's chrome.release specification..."
This reverts r11864 - it broke the tree.
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@11867 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/installer/setup/install.cc')
-rw-r--r-- | chrome/installer/setup/install.cc | 35 |
1 files changed, 3 insertions, 32 deletions
diff --git a/chrome/installer/setup/install.cc b/chrome/installer/setup/install.cc index d872335..e6c51dd 100644 --- a/chrome/installer/setup/install.cc +++ b/chrome/installer/setup/install.cc @@ -12,14 +12,10 @@ #include "chrome/installer/setup/setup_constants.h" #include "chrome/installer/util/browser_distribution.h" #include "chrome/installer/util/google_update_constants.h" -#include "chrome/installer/util/install_util.h" #include "chrome/installer/util/set_reg_value_work_item.h" #include "chrome/installer/util/shell_util.h" #include "chrome/installer/util/work_item.h" -// Build-time generated include file. -#include "registered_dlls.h" - namespace { std::wstring AppendPath(const std::wstring parent_path, const std::wstring path) { @@ -115,6 +111,7 @@ void AddInstallerCopyTasks(const std::wstring& exe_path, install_list->AddMoveTreeWorkItem(archive_path, archive_dst, temp_path); } + // This method tells if we are running on 64 bit platform so that we can copy // one extra exe. If the API call to determine 64 bit fails, we play it safe // and return true anyway so that the executable can be copied. @@ -133,7 +130,7 @@ bool Is64bit() { return false; } -} // namespace +} bool installer::InstallNewVersion(const std::wstring& exe_path, const std::wstring& archive_path, @@ -142,6 +139,7 @@ bool installer::InstallNewVersion(const std::wstring& exe_path, const std::wstring& temp_dir, const HKEY reg_root, const Version& new_version) { + if (reg_root != HKEY_LOCAL_MACHINE && reg_root != HKEY_CURRENT_USER) return false; @@ -270,33 +268,6 @@ bool installer::InstallNewVersion(const std::wstring& exe_path, } } - // Now we need to register any self registering components and unregister - // any that were left from the old version that is being upgraded: - if (!current_version.empty()) { - std::wstring old_dll_path(install_path); - file_util::AppendToPath(&old_dll_path, current_version); - scoped_ptr<WorkItemList> old_dll_list(WorkItem::CreateWorkItemList()); - if (InstallUtil::BuildDLLRegistrationList(old_dll_path, kDllsToRegister, - kNumDllsToRegister, false, - old_dll_list.get())) { - // Don't abort the install as a result of a failure to unregister old - // DLLs. - old_dll_list->Do(); - } - } - - std::wstring dll_path(install_path); - file_util::AppendToPath(&dll_path, new_version.GetString()); - scoped_ptr<WorkItemList> dll_list(WorkItem::CreateWorkItemList()); - if (InstallUtil::BuildDLLRegistrationList(dll_path, kDllsToRegister, - kNumDllsToRegister, true, - dll_list.get())) { - success = dll_list->Do(); - if (!success) { - dll_list->Rollback(); - } - } - if (!success) { LOG(ERROR) << "Install failed, rolling back... "; install_list->Rollback(); |