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

Unified Diff: chrome/installer/setup/uninstall.cc

Issue 8729009: Implement an AutoLaunch experiment for Chrome for certain brand codes. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: '' Created 9 years 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 side-by-side diff with in-line comments
Download patch
Index: chrome/installer/setup/uninstall.cc
===================================================================
--- chrome/installer/setup/uninstall.cc (revision 111708)
+++ chrome/installer/setup/uninstall.cc (working copy)
@@ -22,6 +22,7 @@
#include "chrome/installer/setup/install_worker.h"
#include "chrome/installer/setup/setup_constants.h"
#include "chrome/installer/setup/setup_util.h"
+#include "chrome/installer/util/auto_launch_util.h"
#include "chrome/installer/util/browser_distribution.h"
#include "chrome/installer/util/channel_info.h"
#include "chrome/installer/util/delete_after_reboot_helper.h"
@@ -734,8 +735,11 @@
// Chrome is not in use so lets uninstall Chrome by deleting various files
// and registry entries. Here we will just make best effort and keep going
// in case of errors.
- if (is_chrome)
+ if (is_chrome) {
ClearRlzProductState();
+ if (auto_launch_util::WillLaunchAtLogin())
grt (UTC plus 2) 2011/12/03 04:43:38 this will always return false since the current ex
Finnur 2011/12/13 15:53:24 Good catch. Fixed. On 2011/12/03 04:43:38, grt wr
+ auto_launch_util::SetWillLaunchAtLogin(false, FilePath());
+ }
// First delete shortcuts from Start->Programs, Desktop & Quick Launch.
DeleteChromeShortcuts(installer_state, product);

Powered by Google App Engine
This is Rietveld 408576698