Make use of the new UpdateActiveSetupVersionWorkItem.

chrome/installer/setup/install.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/installer/setup/install.h"
 
 #include <windows.h>
 #include <shlobj.h>
 #include <time.h>
 
@@ skipping 611 matching lines @@
     // At least attempt to update potentially existing all-users shortcuts.
     InstallShortcutLevel level = installer_state.system_install() ?
         ALL_USERS : CURRENT_USER;
     base::FilePath chrome_exe(installer_state.target_path().Append(kChromeExe));
     CreateOrUpdateShortcuts(
         chrome_exe, chrome, prefs, level, INSTALL_SHORTCUT_REPLACE_EXISTING);
     RegisterChromeOnMachine(installer_state, chrome, false);
 
     UpdateOsUpgradeBeacon(installer_state.system_install(),
                           BrowserDistribution::GetDistribution());
-    if (!installer_state.system_install())
+
+    // Update the per-user default-browser-beacon. For user-level installs this

[grt (UTC plus 2), 2015/07/09 14:24:22]

nit: "default browser beacon" for consistency with comments in shell_util.h.

[gab, 2015/07/09 20:41:00]

Done.

+    // can be done directly; whereas it requires triggering Active Setup for
+    // each users' subsequent login on system-level installs.

[grt (UTC plus 2), 2015/07/09 14:24:22]

english nit: users' -> user's ("each user" is singular)

[gab, 2015/07/09 20:41:00]

Done.

+    if (!installer_state.system_install()) {
       UpdateDefaultBrowserBeaconForPath(chrome_exe);
+    } else {
+      UpdateActiveSetupVersionWorkItem active_setup_work_item(
+          InstallUtil::GetActiveSetupPath(chrome.distribution()),
+          UpdateActiveSetupVersionWorkItem::
+              UPDATE_AND_BUMP_OS_UPGRADES_COMPONENT);
+      if (active_setup_work_item.Do())
+        VLOG(1) << "Bumped Active Setup Version on-os-upgrade.";
+      else
+        LOG(ERROR) << "Bumping Active Setup Version on-os-upgrade failed.";

[grt (UTC plus 2), 2015/07/09 14:24:22]

nit: it looks like more than half of the error messages like this in the
installer begin with "Failed to...". how do you feel about changing it to be
consistent with them?

[gab, 2015/07/09 20:41:00]

Sure, kind of redundant with "Error: (...)" but my message ended with "failed"
anyways so not much better :-).

+    }
   }
 }
 
 // NOTE: Should the work done here, on Active Setup, change: kActiveSetupVersion
 // in install_worker.cc needs to be increased for Active Setup to invoke this
 // again for all users of this install.
 void HandleActiveSetupForBrowser(const base::FilePath& installation_root,
                                  const installer::Product& chrome,
                                  bool force) {
   DCHECK(chrome.is_chrome());
@@ skipping 12 matching lines @@
   // Read master_preferences copied beside chrome.exe at install.
   MasterPreferences prefs(installation_root.AppendASCII(kDefaultMasterPrefs));
   base::FilePath chrome_exe(installation_root.Append(kChromeExe));
   CreateOrUpdateShortcuts(
       chrome_exe, chrome, prefs, CURRENT_USER, install_operation);
 
   UpdateDefaultBrowserBeaconForPath(chrome_exe);
 }
 
 }  // namespace installer