Remove support for non-browser products from InstallationState and ProductState.

chrome/installer/util/installation_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/util/installation_state.h"
 
 #include "base/logging.h"
 #include "base/strings/string_util.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/version.h"
 #include "base/win/registry.h"
+#include "chrome/installer/util/browser_distribution.h"
 #include "chrome/installer/util/google_update_constants.h"
 #include "chrome/installer/util/install_util.h"
 
 namespace installer {
 
 ProductState::ProductState()
     : uninstall_command_(base::CommandLine::NO_PROGRAM),
       eula_accepted_(0),
       usagestats_(0),
       msi_(false),
       multi_install_(false),
       has_eula_accepted_(false),
       has_oem_install_(false),
       has_usagestats_(false) {
 }
 
 ProductState::~ProductState() {
 }
 
-bool ProductState::Initialize(bool system_install,
-                              BrowserDistribution::Type type) {
-  return Initialize(system_install,
-                    BrowserDistribution::GetSpecificDistribution(type));
-}
-
-// Initializes |commands| from the "Commands" subkey of |version_key|.
-// Returns false if there is no "Commands" subkey or on error.
-// static
-bool ProductState::InitializeCommands(const base::win::RegKey& version_key,
-                                      AppCommands* commands) {
-  static const DWORD kAccess =
-      KEY_ENUMERATE_SUB_KEYS | KEY_QUERY_VALUE | KEY_WOW64_32KEY;
-  base::win::RegKey commands_key;
-
-  if (commands_key.Open(version_key.Handle(), google_update::kRegCommandsKey,
-                        kAccess) == ERROR_SUCCESS)
-    return commands->Initialize(commands_key, KEY_WOW64_32KEY);
-  return false;
-}
-
-bool ProductState::Initialize(bool system_install,
-                              BrowserDistribution* distribution) {
+bool ProductState::Initialize(bool system_install) {
   static const DWORD kAccess = KEY_QUERY_VALUE | KEY_WOW64_32KEY;
+  const BrowserDistribution* distribution =
+      BrowserDistribution::GetDistribution();
   const std::wstring version_key(distribution->GetVersionKey());
   const std::wstring state_key(distribution->GetStateKey());
   const HKEY root_key = system_install ? HKEY_LOCAL_MACHINE : HKEY_CURRENT_USER;
   base::win::RegKey key;
 
   // Clear the runway.
   Clear();
 
   // Read from the Clients key.
   if (key.Open(root_key, version_key.c_str(), kAccess) == ERROR_SUCCESS) {
@@ skipping 23 matching lines @@
   // Read from the ClientState key.
   if (key.Open(root_key, state_key.c_str(), kAccess) == ERROR_SUCCESS) {
     std::wstring setup_path;
     std::wstring uninstall_arguments;
     // "ap" will be absent if not managed by Google Update.
     channel_.Initialize(key);
 
     // Read in the brand code, it may be absent
     key.ReadValue(google_update::kRegBrandField, &brand_);
 
-    // "UninstallString" will be absent for the multi-installer package.
     key.ReadValue(kUninstallStringField, &setup_path);
-    // "UninstallArguments" will be absent for the multi-installer package.
     key.ReadValue(kUninstallArgumentsField, &uninstall_arguments);
     InstallUtil::ComposeCommandLine(setup_path, uninstall_arguments,
                                     &uninstall_command_);
 
     // "usagestats" may be absent, 0 (false), or 1 (true).  On the chance that
     // different values are permitted in the future, we'll simply hold whatever
     // we find.
     has_usagestats_ = (key.ReadValueDW(google_update::kRegUsageStatsField,
                                        &usagestats_) == ERROR_SUCCESS);
     // "oeminstall" may be present with any value or absent.
     has_oem_install_ = (key.ReadValue(google_update::kRegOemInstallField,
                                       &oem_install_) == ERROR_SUCCESS);
     // "eulaaccepted" may be absent, 0 or 1.
     has_eula_accepted_ = (key.ReadValueDW(google_update::kRegEULAAceptedField,
                                           &eula_accepted_) == ERROR_SUCCESS);
     // "msi" may be absent, 0 or 1
     DWORD dw_value = 0;
     msi_ = (key.ReadValueDW(google_update::kRegMSIField,
                             &dw_value) == ERROR_SUCCESS) && (dw_value != 0);
-    // Multi-install is implied or is derived from the command-line.
-    if (distribution->GetType() == BrowserDistribution::CHROME_BINARIES)
-      multi_install_ = true;
-    else
-      multi_install_ = uninstall_command_.HasSwitch(switches::kMultiInstall);
+    // Multi-install is a legacy option that is read for the sole purpose of
+    // migrating clients away from it.
+    multi_install_ = uninstall_command_.HasSwitch(switches::kMultiInstall);
   }
 
   // Read from the ClientStateMedium key.  Values here override those in
   // ClientState.
   if (system_install &&
       key.Open(root_key, distribution->GetStateMediumKey().c_str(), kAccess) ==
           ERROR_SUCCESS) {
     DWORD dword_value = 0;
 
     if (key.ReadValueDW(google_update::kRegUsageStatsField,
                         &dword_value) == ERROR_SUCCESS) {
       has_usagestats_ = true;
       usagestats_ = dword_value;
     }
 
     if (key.ReadValueDW(google_update::kRegEULAAceptedField,
                         &dword_value) == ERROR_SUCCESS) {
       has_eula_accepted_ = true;
       eula_accepted_ = dword_value;
     }
   }
 
-  return version_.get() != NULL;
+  return version_.get() != nullptr;
 }
 
 base::FilePath ProductState::GetSetupPath() const {
   return uninstall_command_.GetProgram();
 }
 
 const base::Version& ProductState::version() const {
-  DCHECK(version_.get() != NULL);
+  DCHECK(version_);
   return *version_;
 }
 
 ProductState& ProductState::CopyFrom(const ProductState& other) {
   channel_.set_value(other.channel_.value());
-  version_.reset(
-      other.version_.get() ? new base::Version(*other.version_) : NULL);
-  old_version_.reset(
-      other.old_version_.get() ? new base::Version(*other.old_version_) : NULL);
+  version_.reset(other.version_.get() ? new base::Version(*other.version_)
+                                      : nullptr);
+  old_version_.reset(other.old_version_.get()
+                         ? new base::Version(*other.old_version_)
+                         : nullptr);
   brand_ = other.brand_;
   rename_cmd_ = other.rename_cmd_;
   uninstall_command_ = other.uninstall_command_;
   oem_install_ = other.oem_install_;
   commands_.CopyFrom(other.commands_);
   eula_accepted_ = other.eula_accepted_;
   usagestats_ = other.usagestats_;
   msi_ = other.msi_;
   multi_install_ = other.multi_install_;
   has_eula_accepted_ = other.has_eula_accepted_;
@@ skipping 38 matching lines @@
 }
 
 bool ProductState::GetUsageStats(DWORD* usagestats) const {
   DCHECK(usagestats);
   if (!has_usagestats_)
     return false;
   *usagestats = usagestats_;
   return true;
 }
 
-InstallationState::InstallationState() {
+// Initializes |commands| from the "Commands" subkey of |version_key|.

[huangs, 2017/01/09 09:09:10]

Seems this is only called from within the file?  Maybe move to local namespace
{} ?

[grt (UTC plus 2), 2017/01/09 10:11:45]

Done.

+// Returns false if there is no "Commands" subkey or on error.
+// static
+bool ProductState::InitializeCommands(const base::win::RegKey& version_key,
+                                      AppCommands* commands) {
+  static const DWORD kAccess =
+      KEY_ENUMERATE_SUB_KEYS | KEY_QUERY_VALUE | KEY_WOW64_32KEY;
+  base::win::RegKey commands_key;
+
+  if (commands_key.Open(version_key.Handle(), google_update::kRegCommandsKey,
+                        kAccess) == ERROR_SUCCESS)

[huangs, 2017/01/09 09:09:10]

NIT: Not sure if we should use {}? I'll leave to your discretion.

[grt (UTC plus 2), 2017/01/09 10:11:45]

I like them there. Thanks.

+    return commands->Initialize(commands_key, KEY_WOW64_32KEY);
+  return false;
 }
 
-// static
-int InstallationState::IndexFromDistType(BrowserDistribution::Type type) {
-  static_assert(BrowserDistribution::CHROME_BROWSER == CHROME_BROWSER_INDEX,
-                "unexpected_chrome_browser_distribution_value_");
-  static_assert(BrowserDistribution::CHROME_FRAME == CHROME_FRAME_INDEX,
-                "unexpected_chrome_frame_distribution_value_");
-  static_assert(BrowserDistribution::CHROME_BINARIES == CHROME_BINARIES_INDEX,
-                "unexpected_chrome_frame_distribution_value_");
-  DCHECK(type == BrowserDistribution::CHROME_BROWSER ||
-         type == BrowserDistribution::CHROME_FRAME ||
-         type == BrowserDistribution::CHROME_BINARIES);
-  return type;
+InstallationState::InstallationState() {}
+
+void InstallationState::Initialize() {
+  user_chrome_.Initialize(false);

[huangs, 2017/01/09 09:09:10]

// TODO(grt): Do something with return value, or remove if not needed.

[grt (UTC plus 2), 2017/01/09 10:11:45]

Return value of ProductState::Initialize isn't used here, but is used by other
callers. Leaving as-is.

+  system_chrome_.Initialize(true);
 }
 
-void InstallationState::Initialize() {
-  BrowserDistribution* distribution;
-
-  distribution = BrowserDistribution::GetSpecificDistribution(
-      BrowserDistribution::CHROME_BROWSER);
-  user_products_[CHROME_BROWSER_INDEX].Initialize(false, distribution);
-  system_products_[CHROME_BROWSER_INDEX].Initialize(true, distribution);
-
-  distribution = BrowserDistribution::GetSpecificDistribution(
-      BrowserDistribution::CHROME_FRAME);
-  user_products_[CHROME_FRAME_INDEX].Initialize(false, distribution);
-  system_products_[CHROME_FRAME_INDEX].Initialize(true, distribution);
-
-  distribution = BrowserDistribution::GetSpecificDistribution(
-      BrowserDistribution::CHROME_BINARIES);
-  user_products_[CHROME_BINARIES_INDEX].Initialize(false, distribution);
-  system_products_[CHROME_BINARIES_INDEX].Initialize(true, distribution);
+const ProductState* InstallationState::GetProductState(
+    bool system_install) const {
+  const ProductState* product_state =
+      GetNonVersionedProductState(system_install);
+  return product_state->version_.get() ? product_state : nullptr;
 }
 
 const ProductState* InstallationState::GetNonVersionedProductState(
-    bool system_install,
-    BrowserDistribution::Type type) const {
-  const ProductState& product_state = (system_install ? system_products_ :
-      user_products_)[IndexFromDistType(type)];
-  return &product_state;
+    bool system_install) const {
+  return system_install ? &system_chrome_ : &user_chrome_;
 }
 
-const ProductState* InstallationState::GetProductState(
-    bool system_install,
-    BrowserDistribution::Type type) const {
-  const ProductState* product_state =
-      GetNonVersionedProductState(system_install, type);
-  return product_state->version_.get() == NULL ? NULL : product_state;
-}
 }  // namespace installer