Don't uninstall Google Update for Windows Installer-driven uninstalls.

chrome/installer/setup/setup_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/installer/setup/setup_main.h"
 
 #include <windows.h>
 #include <msi.h>
 #include <shellapi.h>
 #include <shlobj.h>
@@ skipping 610 matching lines @@
 
   // Trigger Active Setup if it was requested for the chrome product. This needs
   // to be done after the UninstallProduct calls as some of them might
   // otherwise terminate the process launched by TriggerActiveSetupCommand().
   if (cmd_line.HasSwitch(installer::switches::kTriggerActiveSetup))
     InstallUtil::TriggerActiveSetupCommand();
 
   if (!system_level_cmd.GetProgram().empty())
     base::LaunchProcess(system_level_cmd, base::LaunchOptions());
 
-  // Tell Google Update that an uninstall has taken place.
-  // Ignore the return value: success or failure of Google Update
-  // has no bearing on the success or failure of Chrome's uninstallation.
-  google_update::UninstallGoogleUpdate(installer_state.system_install());
+  // Tell Google Update that an uninstall has taken place if this install did
+  // not originate from the MSI. Gogle Update has its own logic relating to

[Georges Khalil, 2017/04/25 15:18:23]

nit: s/Gogle/Google

[grt (UTC plus 2), 2017/04/25 18:01:31]

<facepalm>

+  // MSI-driven uninstalls that conflicts with this. Ignore the return value:
+  // success or failure of Google Update has no bearing on the success or
+  // failure of Chrome's uninstallation.
+  if (!installer_state.is_msi())
+    google_update::UninstallGoogleUpdate(installer_state.system_install());
 
   return install_status;
 }
 
 installer::InstallStatus InstallProducts(
     const InstallationState& original_state,
     const base::FilePath& setup_exe,
     const base::CommandLine& cmd_line,
     const MasterPreferences& prefs,
     InstallerState* installer_state,
@@ skipping 855 matching lines @@
     // Note that we allow the status installer::UNINSTALL_REQUIRES_REBOOT
     // to pass through, since this is only returned on uninstall which is
     // never invoked directly by Google Update.
     return_code = InstallUtil::GetInstallReturnCode(install_status);
   }
 
   VLOG(1) << "Installation complete, returning: " << return_code;
 
   return return_code;
 }