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

chrome/installer/util/browser_distribution.h

Index: chrome/installer/util/browser_distribution.h
diff --git a/chrome/installer/util/browser_distribution.h b/chrome/installer/util/browser_distribution.h
index 7ec1eacd79bcc443d426aa1f41ea32818716e951..47d790870cd79e931c8133811acb232ddfa3e700 100644
--- a/chrome/installer/util/browser_distribution.h
+++ b/chrome/installer/util/browser_distribution.h
@@ -121,6 +121,8 @@ class BrowserDistribution {
 
   virtual bool CanCreateDesktopShortcuts();
 
+  virtual string16 GetIconExe();

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

I don't really like this new call, I feel if this existed it should return a
full path and since me and grt want to move towards a BrowserDistribution class
that only returns constants I don't like adding this.

The icon index itself is a constant so that's different, the path to the exe is
built differently depending on the state the caller has and I think it should
remain the caller's responsibility to build that path.

At best this could return a constant relative path to the install root, but this
easily becomes confusing to read at the call site imo (I'm thinking system-level
installs where User Data and Application aren't in the same folder) and some
path logic needs to be applied by the caller anyways so I don't really see the
benefit (not even mentioning that the caller will have to make sure the path
they are adding the relative path to is at the correct depth for this relative
path to add up... meh).

[erikwright (departed), 2012/11/13 14:55:34]

Which .exe the icon is in is also constant.
The question is whether the caller has to apply a single path logic for all
cases or whether the caller needs to have an if/else block checking specific
products.

See my other comment in this CL for alternatives.

[grt (UTC plus 2), 2012/11/13 16:19:32]

GetIconExe -> GetIconFile or somesuch, since icons can be in any PE Image file
(.exe or .dll or whatever extension you want).

also, please please please don't follow the terrible precedent of not
documenting methods in this file.

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

I would prefer to have a method to get the full path.  However, chrome_exe is
being passed all over the place in ShellUtils, so some refactor is needed.  What
I have is a compromise to avoid changing too many things.  Pending decisions.

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

Renamed to GetIconFile(), and added comments.

+
   virtual int GetIconIndex();
 
   virtual bool GetChromeChannel(string16* channel);