Adding App Launcher shortcuts on install.

chrome/installer/util/chrome_app_host_distribution.cc

Index: chrome/installer/util/chrome_app_host_distribution.cc
diff --git a/chrome/installer/util/chrome_app_host_distribution.cc b/chrome/installer/util/chrome_app_host_distribution.cc
index 112678e430f84ec1fb76b1c7d2747661fc97bced..37a446d8dcf87fd373143de1d09111e6ac3cee6a 100644
--- a/chrome/installer/util/chrome_app_host_distribution.cc
+++ b/chrome/installer/util/chrome_app_host_distribution.cc
@@ -48,6 +48,11 @@ string16 ChromeAppHostDistribution::GetAlternateApplicationName() {
   return product_name;
 }
 
+string16 ChromeAppHostDistribution::GetBaseAppId() {
+  // See AppListController::GetAppModelId().
+  return L"ChromeAppList";

[erikwright (departed), 2012/11/01 01:21:50]

Is a TODO required to use the correct App Model ID incorporating the
user-data-dir, right?

[gab, 2012/11/01 04:46:19]

This is the "base AppId", i.e. the one to begin all the appids generated by
other methods. See methods in shell_util.cc; this is correct (compare with
google_chrome_distribution.cc.

[huangs, 2012/11/01 19:20:38]

Expanded comment, and defined constant for this.

+}
+
 string16 ChromeAppHostDistribution::GetInstallSubDir() {
   return BrowserDistribution::GetSpecificDistribution(
       BrowserDistribution::CHROME_BINARIES)->GetInstallSubDir();
@@ -124,7 +129,7 @@ bool ChromeAppHostDistribution::CanSetAsDefault() {
 }
 
 bool ChromeAppHostDistribution::CanCreateDesktopShortcuts() {
-  return false;
+  return true;

[gab, 2012/11/01 04:46:19]

I don't think this has any impact anymore (i.e. fine to change it but we should
add to the installer fixit
(https://docs.google.com/a/google.com/document/d/1Js17FLROy5L83Q8Qp9G6l6hRq1fH...
to either make those have an impact again if they should or remove them (I feel
the latter is correct now))

[huangs, 2012/11/01 19:20:38]

Will edit once I have access.  Planning to add:
- BrowserDistribution::CanCreateDesktopShortcuts() (and overrides) are not
useful now.  Either make it useful or remove it.

 }
 
 bool ChromeAppHostDistribution::GetCommandExecuteImplClsid(