Launch system-level Active Setup immediately following self-destruction of a user-level Chrome.

chrome/browser/chrome_browser_main.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/chrome_browser_main.h"
 
 #if defined(TOOLKIT_GTK)
 #include <gtk/gtk.h>
 #endif
 
@@ skipping 1179 matching lines @@
   PostProfileInit();
 
 #if !defined(OS_ANDROID)
   // Show the First Run UI if this is the first time Chrome has been run on
   // this computer, or we're being compelled to do so by a command line flag.
   // Note that this be done _after_ the PrefService is initialized and all
   // preferences are registered, since some of the code that the importer
   // touches reads preferences.
   if (is_first_run_) {
 #if defined(OS_WIN)
-    // On Windows, trigger the Active Setup command in some scenarios to finish
-    // configuring this user's install (e.g. per-user shortcuts on system-level
-    // installs).
+    // On Windows, trigger the Active Setup command for system-level Chromes to
+    // finish configuring this user's install (e.g. per-user shortcuts).
     // Delay the task slightly to give Chrome launch I/O priority while also
     // making sure shortcuts are created promptly to avoid annoying the user by
     // re-creating shortcuts he previously deleted.
     // TODO(gab): Add a first run section to ChromeBrowserMainParts and remove
     // OS specific sections below.

[Nico, 2012/11/20 18:07:12]

Sounds like a good opportunity to work on this TODO :-)

[gab, 2012/11/20 20:42:11]

Actually, this needs to merged in M24 so I want to keep the change minimal.

This piece of code will need to move anyways to solve http://crbug.com/161914
and thus will likely get out of chrome_browser_main.cc very soon.

     static const int64 kTiggerActiveSetupDelaySeconds = 5;
-    BrowserThread::GetBlockingPool()->PostDelayedTask(
-        FROM_HERE,
-        base::Bind(&InstallUtil::TriggerActiveSetupCommandIfNeeded),
-        base::TimeDelta::FromSeconds(kTiggerActiveSetupDelaySeconds));
+    FilePath chrome_exe;
+    if (!PathService::Get(base::FILE_EXE, &chrome_exe)) {
+      NOTREACHED();
+    } else if (!InstallUtil::IsPerUserInstall(chrome_exe.value().c_str())) {
+      BrowserThread::GetBlockingPool()->PostDelayedTask(
+          FROM_HERE,
+          base::Bind(&InstallUtil::TriggerActiveSetupCommand),
+          base::TimeDelta::FromSeconds(kTiggerActiveSetupDelaySeconds));
+    }
 #endif  // OS_WIN
     if (!first_run_ui_bypass_) {
       first_run::AutoImport(profile_,
                             master_prefs_->homepage_defined,
                             master_prefs_->do_import_items,
                             master_prefs_->dont_import_items,
                             master_prefs_->make_chrome_default,
                             process_singleton_.get());
 #if defined(OS_POSIX) && !defined(OS_CHROMEOS)
       // On Windows, the download is tagged with enable/disable stats so there
@@ skipping 493 matching lines @@
     if (base::win::GetVersion() <= base::win::VERSION_XP)
       uma_name += "_XP";
 
     uma_name += "_PreRead_";
     uma_name += pre_read_percentage;
     AddPreReadHistogramTime(uma_name.c_str(), time);
   }
 #endif
 #endif
 }