Add default icon to app_host.exe, and use it in shortcuts during installation.

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 69eba10625936f6c9a5cdde40b374efb59130c7e..e49105f7949275667a43283bbd2ce84dd9c4abdc 100644
--- a/chrome/installer/util/chrome_app_host_distribution.cc
+++ b/chrome/installer/util/chrome_app_host_distribution.cc
@@ -22,11 +22,6 @@
 
 namespace {
 
-#if defined(GOOGLE_CHROME_BUILD)
-const int kAppListIconIndex = 5;
-#else
-const int kAppListIconIndex = 1;
-#endif
 const wchar_t kChromeAppHostGuid[] = L"{FDA71E6F-AC4C-4a00-8B70-9958A68906BF}";
 
 }  // namespace
@@ -139,8 +134,8 @@ bool ChromeAppHostDistribution::CanCreateDesktopShortcuts() {
   return true;
 }
 
-int ChromeAppHostDistribution::GetIconIndex() {

[gab, 2012/11/13 14:15:12]

Although you use the default in browser_distribution.cc, I think I'm still in
favor of overriding GetIconIndex() here, makes it more readable and easier to
change later; also isn't there a different icon for Chromium vs Chrome
AppLauncher.

Same comment applies for other distribution where you removed the override.

[huangs, 2012/11/13 20:41:54]

Done, but this is the only distribution where the override was removed.

-  return kAppListIconIndex;
+string16 ChromeAppHostDistribution::GetIconExe() {
+  return installer::kChromeAppHostExe;
 }
 
 bool ChromeAppHostDistribution::GetCommandExecuteImplClsid(