Reset user data directory and disk cache directory after downgrade.

chrome/installer/util/install_util.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.
 //
 // See the corresponding header file for description of the functions in this
 // file.
 
 #include "chrome/installer/util/install_util.h"
 
 #include <shellapi.h>
@@ skipping 26 matching lines @@
 #include "chrome/installer/util/installation_state.h"
 #include "chrome/installer/util/l10n_string_util.h"
 #include "chrome/installer/util/util_constants.h"
 #include "chrome/installer/util/work_item_list.h"
 
 using base::win::RegKey;
 using installer::ProductState;
 
 namespace {
 
+const wchar_t kRegDowngradeVersion[] = L"DowngradeVersion";
 const wchar_t kStageBinaryPatching[] = L"binary_patching";
 const wchar_t kStageBuilding[] = L"building";
 const wchar_t kStageConfiguringAutoLaunch[] = L"configuring_auto_launch";
 const wchar_t kStageCopyingPreferencesFile[] = L"copying_prefs";
 const wchar_t kStageCreatingShortcuts[] = L"creating_shortcuts";
 const wchar_t kStageEnsemblePatching[] = L"ensemble_patching";
 const wchar_t kStageExecuting[] = L"executing";
 const wchar_t kStageFinishing[] = L"finishing";
 const wchar_t kStagePreconditions[] = L"preconditions";
 const wchar_t kStageRefreshingPolicy[] = L"refreshing_policy";
@@ skipping 563 matching lines @@
 }
 
 // Populate |info| for |file|, returning true on success.
 // static
 bool InstallUtil::ProgramCompare::GetInfo(const base::File& file,
                                           BY_HANDLE_FILE_INFORMATION* info) {
   DCHECK(file.IsValid());
   return GetFileInformationByHandle(file.GetPlatformFile(), info) != 0;
 }
 
+// static
+base::Version InstallUtil::GetDowngradeVersion(
+    bool system_install,
+    const BrowserDistribution* dist) {
+  DCHECK(dist);
+  base::win::RegKey key;
+  base::string16 downgrade_version;
+  if (key.Open(system_install ? HKEY_LOCAL_MACHINE : HKEY_CURRENT_USER,
+               dist->GetStateKey().c_str(),
+               KEY_QUERY_VALUE | KEY_WOW64_32KEY) != ERROR_SUCCESS ||
+      key.ReadValue(kRegDowngradeVersion, &downgrade_version) !=
+          ERROR_SUCCESS) {
+    return base::Version();
+  }
+  return base::Version(base::UTF16ToASCII(downgrade_version));
+}
+
+// static
+void InstallUtil::AddUpdateDowngradeVersionItem(
+    bool system_install,
+    const base::Version* current_version,
+    const base::Version& new_version,
+    const BrowserDistribution* dist,
+    WorkItemList* list) {
+  DCHECK(list);
+  DCHECK(dist);

[grt (UTC plus 2), 2016/05/20 18:17:05]

to enforce the "only call this for Chrome", add:
  DCHECK_EQ(BrowserDistribution::CHROME_BROWSER, dist->GetType());

[zmin, 2016/05/20 19:11:21]

Done.

+  base::Version downgrade_version = GetDowngradeVersion(system_install, dist);
+  HKEY root = system_install ? HKEY_LOCAL_MACHINE : HKEY_CURRENT_USER;
+  if (current_version == nullptr ||

[grt (UTC plus 2), 2016/05/20 18:17:05]

nit: if (!current_version

[zmin, 2016/05/20 19:11:21]

Done.

+      (*current_version < new_version &&

[grt (UTC plus 2), 2016/05/20 18:17:05]

should this be <= so that the delete is done for same-version-repair as well? is
there harm in this?

[zmin, 2016/05/20 19:11:21]

For same-version-repair, the |downgrade_version| should be taken care of by
previous install. However, because "strange things happen in the registry on a
regular basis", I think it worth to add '=' so that we can fix the
|downgrade_version| during repair if needed.

+       ((!downgrade_version.IsValid() || downgrade_version <= new_version)))) {
+    list->AddDeleteRegValueWorkItem(root, dist->GetStateKey(), KEY_WOW64_32KEY,
+                                    kRegDowngradeVersion);
+  } else if (*current_version > new_version && !downgrade_version.IsValid()) {
+    list->AddSetRegValueWorkItem(
+        root, dist->GetStateKey(), KEY_WOW64_32KEY, kRegDowngradeVersion,
+        base::ASCIIToUTF16(current_version->GetString()), true);
+  }
+}
+
 InstallUtil::ProgramCompare::ProgramCompare(const base::FilePath& path_to_match)
     : ProgramCompare(path_to_match, ComparisonType::FILE) {}
 
 InstallUtil::ProgramCompare::ProgramCompare(const base::FilePath& path_to_match,
                                             ComparisonType comparison_type)
     : path_to_match_(path_to_match),
       file_info_(),
       comparison_type_(comparison_type) {
   DCHECK(!path_to_match_.empty());
   if (!OpenForInfo(path_to_match_, &file_, comparison_type_)) {
@@ skipping 38 matching lines @@
 
   // Open the program and see if it references the expected file.
   base::File file;
   BY_HANDLE_FILE_INFORMATION info = {};
 
   return (OpenForInfo(path, &file, comparison_type_) && GetInfo(file, &info) &&
           info.dwVolumeSerialNumber == file_info_.dwVolumeSerialNumber &&
           info.nFileIndexHigh == file_info_.nFileIndexHigh &&
           info.nFileIndexLow == file_info_.nFileIndexLow);
 }