Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(87)

Side by Side Diff: chrome/installer/setup/uninstall.cc

Issue 9972012: Resolve the conflict that auto-launch has with the background mode feature (part 1). (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 8 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 // 4 //
5 // This file defines the methods useful for uninstalling Chrome. 5 // This file defines the methods useful for uninstalling Chrome.
6 6
7 #include "chrome/installer/setup/uninstall.h" 7 #include "chrome/installer/setup/uninstall.h"
8 8
9 #include <vector> 9 #include <vector>
10 10
(...skipping 745 matching lines...) Expand 10 before | Expand all | Expand 10 after
756 InstallUtil::ExecuteExeAsAdmin(new_cmd, &exit_code); 756 InstallUtil::ExecuteExeAsAdmin(new_cmd, &exit_code);
757 } 757 }
758 } 758 }
759 759
760 // Chrome is not in use so lets uninstall Chrome by deleting various files 760 // Chrome is not in use so lets uninstall Chrome by deleting various files
761 // and registry entries. Here we will just make best effort and keep going 761 // and registry entries. Here we will just make best effort and keep going
762 // in case of errors. 762 // in case of errors.
763 if (is_chrome) { 763 if (is_chrome) {
764 ClearRlzProductState(); 764 ClearRlzProductState();
765 765
766 if (auto_launch_util::WillLaunchAtLogin( 766 if (auto_launch_util::WillLaunchAtLoginWithSwitch(
767 installer_state.target_path(), 767 installer_state.target_path(),
768 ASCIIToUTF16(chrome::kInitialProfile))) { 768 ASCIIToUTF16(chrome::kInitialProfile),
769 "")) {
769 auto_launch_util::SetWillLaunchAtLogin( 770 auto_launch_util::SetWillLaunchAtLogin(
770 false, FilePath(), ASCIIToUTF16(chrome::kInitialProfile)); 771 FilePath(),
772 ASCIIToUTF16(chrome::kInitialProfile),
773 auto_launch_util::FLAG_DISABLE,
774 auto_launch_util::FLAG_DISABLE);
771 } 775 }
772 } 776 }
773 777
774 // First delete shortcuts from Start->Programs, Desktop & Quick Launch. 778 // First delete shortcuts from Start->Programs, Desktop & Quick Launch.
775 DeleteChromeShortcuts(installer_state, product); 779 DeleteChromeShortcuts(installer_state, product);
776 780
777 // Delete the registry keys (Uninstall key and Version key). 781 // Delete the registry keys (Uninstall key and Version key).
778 HKEY reg_root = installer_state.root_key(); 782 HKEY reg_root = installer_state.root_key();
779 783
780 // Note that we must retrieve the distribution-specific data before deleting 784 // Note that we must retrieve the distribution-specific data before deleting
(...skipping 147 matching lines...) Expand 10 before | Expand all | Expand 10 after
928 932
929 // Try and delete the preserved local state once the post-install 933 // Try and delete the preserved local state once the post-install
930 // operations are complete. 934 // operations are complete.
931 if (!backup_state_file.empty()) 935 if (!backup_state_file.empty())
932 file_util::Delete(backup_state_file, false); 936 file_util::Delete(backup_state_file, false);
933 937
934 return ret; 938 return ret;
935 } 939 }
936 940
937 } // namespace installer 941 } // namespace installer
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698