Implement installation of the Chrome App Host.

chrome/installer/util/chromium_binaries_distribution.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.
 //
 // This file declares a class that contains various method related to branding.
 
 #include "chrome/installer/util/google_chrome_binaries_distribution.h"
 
 #include "base/logging.h"
+#include "chrome/installer/util/l10n_string_util.h"
+
+#include "installer_util_strings.h"  // NOLINT
 
 namespace {
 
 const wchar_t kChromiumBinariesName[] = L"Chromium Binaries";
-
 }  // namespace
 
 ChromiumBinariesDistribution::ChromiumBinariesDistribution()
     : BrowserDistribution(CHROME_BINARIES),
       browser_distribution_(
           BrowserDistribution::GetSpecificDistribution(CHROME_BROWSER)) {
 }
 
 string16 ChromiumBinariesDistribution::GetAppGuid() {
   return string16();
 }
 
 string16 ChromiumBinariesDistribution::GetBaseAppName() {
-  NOTREACHED();
-  return string16();
+  return L"Chromium Binaries";
+}
+
+string16 ChromiumBinariesDistribution::GetBaseAppId() {
+  return L"Chromium Binaries";
 }
 
 string16 ChromiumBinariesDistribution::GetAppShortCutName() {
-  return kChromiumBinariesName;
+  const string16& product_name =
+      installer::GetLocalizedString(IDS_PRODUCT_BINARIES_NAME_BASE);
+  return product_name;
 }
 
 string16 ChromiumBinariesDistribution::GetAlternateApplicationName() {
-  NOTREACHED();
-  return string16();
-}
-
-string16 ChromiumBinariesDistribution::GetBaseAppId() {
-  NOTREACHED();
-  return string16();
+  const string16& product_name =
+      installer::GetLocalizedString(IDS_PRODUCT_BINARIES_NAME_BASE);
+  return product_name;
 }
 
 string16 ChromiumBinariesDistribution::GetInstallSubDir() {
   return browser_distribution_->GetInstallSubDir();
 }
 
 string16 ChromiumBinariesDistribution::GetPublisherName() {
-  NOTREACHED();
-  return string16();
+  const string16& publisher_name =
+      installer::GetLocalizedString(IDS_ABOUT_VERSION_COMPANY_NAME_BASE);
+  return publisher_name;
 }
 
 string16 ChromiumBinariesDistribution::GetAppDescription() {
-  NOTREACHED();
-  return string16();
+  return L"Shared components for Chromium products.";
 }
 
 string16 ChromiumBinariesDistribution::GetLongAppDescription() {
-  NOTREACHED();
-  return string16();
+  return L"Shared components for Chromium products.";
 }
 
 std::string ChromiumBinariesDistribution::GetSafeBrowsingName() {
-  NOTREACHED();
-  return std::string();
+  return "chromiumbinaries";
 }
 
 string16 ChromiumBinariesDistribution::GetStateKey() {
   return string16(L"Software\\").append(kChromiumBinariesName);
 }
 
 string16 ChromiumBinariesDistribution::GetStateMediumKey() {
   return string16(L"Software\\").append(kChromiumBinariesName);
 }
 
 string16 ChromiumBinariesDistribution::GetUninstallLinkName() {
-  NOTREACHED();
-  return string16();
+  return L"Uninstall Chromium Binaries";

[grt (UTC plus 2), 2012/07/12 18:37:10]

why?

[Jói, 2012/07/13 14:07:41]

No localization on this or other user-visible messages here?

[erikwright (departed), 2012/07/16 20:13:11]

Some of these NOTREACHED were being hit even though the values are never
actually used.

In the end I fixed it up so no one ever calls the methods.

[erikwright (departed), 2012/07/16 20:13:11]

They were not actually user-visible. In any case, removed.

 }
 
 string16 ChromiumBinariesDistribution::GetUninstallRegPath() {
-  NOTREACHED();
-  return string16();
+  return L"Software\\Microsoft\\Windows\\CurrentVersion\\Uninstall\\"

[grt (UTC plus 2), 2012/07/12 18:37:10]

the binaries show up in add/remove?

[erikwright (departed), 2012/07/16 20:13:11]

Done.

+         L"Google Chromium Binaries";

[grt (UTC plus 2), 2012/07/12 18:37:10]

this shouldn't have "Google" in the name, although i wonder why we need a value
here at all.

[erikwright (departed), 2012/07/16 20:13:11]

Done.

 }
 
 string16 ChromiumBinariesDistribution::GetVersionKey() {
   return string16(L"Software\\").append(kChromiumBinariesName);
 }
 
 bool ChromiumBinariesDistribution::CanSetAsDefault() {
-  NOTREACHED();
   return false;
 }
 
-int ChromiumBinariesDistribution::GetIconIndex() {
-  NOTREACHED();
-  return 0;
+bool ChromiumBinariesDistribution::CanCreateDesktopShortcuts() {
+  return false;
+}
+
+bool ChromiumBinariesDistribution::GetDelegateExecuteHandlerData(
+    string16* handler_class_uuid,
+    string16* type_lib_uuid,
+    string16* type_lib_version,
+    string16* interface_uuid) {
+  return false;
 }
 
 bool ChromiumBinariesDistribution::GetChromeChannel(string16* channel) {
-  NOTREACHED();
   return false;
 }