Force migrate all clients from multi-install back to single-install.

chrome/installer/setup/installer_state.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/installer_state.h"
 
 #include <stddef.h>
 
 #include <algorithm>
 #include <functional>
@@ skipping 55 matching lines @@
 
 InstallerState::InstallerState()
     : operation_(UNINITIALIZED),
       state_type_(BrowserDistribution::CHROME_BROWSER),
       multi_package_distribution_(NULL),
       level_(UNKNOWN_LEVEL),
       package_type_(UNKNOWN_PACKAGE_TYPE),
       root_key_(NULL),
       msi_(false),
       background_mode_(false),
-      verbose_logging_(false) {}
+      verbose_logging_(false),
+      is_migrating_to_single_(false) {}
 
 InstallerState::InstallerState(Level level)
     : operation_(UNINITIALIZED),
       state_type_(BrowserDistribution::CHROME_BROWSER),
       multi_package_distribution_(NULL),
       level_(UNKNOWN_LEVEL),
       package_type_(UNKNOWN_PACKAGE_TYPE),
       root_key_(NULL),
       msi_(false),
       background_mode_(false),
-      verbose_logging_(false) {
+      verbose_logging_(false),
+      is_migrating_to_single_(false) {
   // Use set_level() so that root_key_ is updated properly.
   set_level(level);
 }
 
 InstallerState::~InstallerState() {
 }
 
 void InstallerState::Initialize(const base::CommandLine& command_line,
                                 const MasterPreferences& prefs,
                                 const InstallationState& machine_state) {
@@ skipping 33 matching lines @@
   }
 
   BrowserDistribution* operand = NULL;
 
   if (is_uninstall) {
     operation_ = UNINSTALL;
   } else if (!prefs.is_multi_install()) {
     // For a single-install, the current browser dist is the operand.
     operand = BrowserDistribution::GetDistribution();
     operation_ = SINGLE_INSTALL_OR_UPDATE;
+    // Is this a migration from multi-install to single-install?
+    const ProductState* state =
+        machine_state.GetProductState(system_install(), operand->GetType());
+    is_migrating_to_single_ = state && state->is_multi_install();
   } else if (IsMultiInstallUpdate(prefs, machine_state)) {
     // Updates driven by Google Update take place under the multi-installer's
     // app guid.
     operand = multi_package_distribution_;
     operation_ = MULTI_UPDATE;
   } else {
     operation_ = MULTI_INSTALL;
   }
 
   // Initial, over, and un-installs will take place under Chrome or Binaries
@@ skipping 315 matching lines @@
   state_key_.clear();
   state_type_ = BrowserDistribution::CHROME_BROWSER;
   products_.clear();
   multi_package_distribution_ = NULL;
   critical_update_version_ = base::Version();
   level_ = UNKNOWN_LEVEL;
   package_type_ = UNKNOWN_PACKAGE_TYPE;
   root_key_ = NULL;
   msi_ = false;
   verbose_logging_ = false;
+  is_migrating_to_single_ = false;
 }
 
 bool InstallerState::AnyExistsAndIsInUse(const InstallationState& machine_state,
                                          uint32_t file_bits) const {
   static const wchar_t* const kBinaryFileNames[] = {
     kChromeDll,
     kChromeFrameDll,
     kChromeFrameHelperDll,
     kChromeFrameHelperExe,
   };
@@ skipping 132 matching lines @@
        ++scan) {
     InstallUtil::AddInstallerResultItems(
         system_install, (*scan)->distribution()->GetStateKey(), status,
         string_resource_id, launch_cmd, install_list.get());
   }
   // And for the binaries if this is a multi-install.
   if (is_multi_install()) {
     InstallUtil::AddInstallerResultItems(
         system_install, multi_package_binaries_distribution()->GetStateKey(),
         status, string_resource_id, launch_cmd, install_list.get());
+  } else if (is_migrating_to_single() &&
+             InstallUtil::GetInstallReturnCode(status)) {
+#if defined(GOOGLE_CHROME_BUILD)
+    // Also write to the binaries on error if this is a migration back to
+    // single-install for Google Chrome builds. Skip this for Chromium builds
+    // because they lump the "ClientState" and "Clients" keys into a single
+    // key. As a consequence, writing this value causes Software\Chromium to be
+    // re-created after it was deleted during the migration to single-install.
+    // Google Chrome builds don't suffer this since the two keys are distinct
+    // and have different lifetimes. The result is only written on failure since
+    // for success, the binaries have been uninstalled and therefore the result
+    // will not be read by Google Update.
+    InstallUtil::AddInstallerResultItems(
+        system_install, BrowserDistribution::GetSpecificDistribution(
+                            BrowserDistribution::CHROME_BINARIES)
+                            ->GetStateKey(),
+        status, string_resource_id, launch_cmd, install_list.get());
+#endif
   }
   install_list->Do();
 }
 
 bool InstallerState::RequiresActiveSetup() const {
   return system_install() && FindProduct(BrowserDistribution::CHROME_BROWSER);
 }
 
 }  // namespace installer