Add a flag to control whether there is a shortcut for the app list / launcher in the Windows taskba…

chrome/browser/shell_integration_win.cc

Index: chrome/browser/shell_integration_win.cc
diff --git a/chrome/browser/shell_integration_win.cc b/chrome/browser/shell_integration_win.cc
index 874faaabdcb673eb766ef40fe1f65622aea1bdd6..d671ff23b587145bb0bf928d63b6d75cb10c8fb2 100644
--- a/chrome/browser/shell_integration_win.cc
+++ b/chrome/browser/shell_integration_win.cc
@@ -41,6 +41,12 @@ using content::BrowserThread;
 
 namespace {
 
+#if defined(GOOGLE_CHROME_BUILD)
+const wchar_t kAppListAppName[] = L"ChromeAppList";
+#else
+const wchar_t kAppListAppName[] = L"ChromiumAppList";
+#endif
+
 // Helper function for ShellIntegration::GetAppId to generates profile id
 // from profile path. "profile_id" is composed of sanitized basenames of
 // user data dir and profile dir joined by a ".".
@@ -139,6 +145,8 @@ bool GetExpectedAppId(const FilePath& chrome_exe,
   } else if (command_line.HasSwitch(switches::kAppId)) {
     app_name = UTF8ToUTF16(web_app::GenerateApplicationNameFromExtensionId(
         command_line.GetSwitchValueASCII(switches::kAppId)));
+  } else if (command_line.HasSwitch(switches::kShowAppList)) {
+    app_name = kAppListAppName;
   } else {
     BrowserDistribution* dist = BrowserDistribution::GetDistribution();
     app_name = ShellUtil::GetBrowserModelId(
@@ -411,6 +419,12 @@ string16 ShellIntegration::GetChromiumModelIdForProfile(
       profile_path);
 }
 
+string16 ShellIntegration::GetAppListAppModelIdForProfile(
+    const FilePath& profile_path) {
+  return ShellIntegration::GetAppModelIdForProfile(kAppListAppName,

[gab, 2012/11/01 22:18:11]

huangs@ is introducing something very similar in
http://codereview.chromium.org/11359013/, your appid should start from the
distribution's BaseAppId, then optionally (but I highly recommend it to avoid
trouble later, e.g. say you ever want to support system-level app launcher
installs) add a per-user suffix, and finally add the profile part of the appid.

What I suggested to huangs@ and erikwright@ is that you modify
ShellUtil::GetBrowserModelId to be more generic (i.e. not browser only -- IIRC
might just need a name change...).

[benwells, 2012/11/01 22:59:08]

Yes I saw that CL this morning. I had assumed that going through
ShellIntegration would get the proper suffix, but I just traced through and it
doesn't.

Proposal: leave this as is for now, to enable the dogfooding to commence, and
update to use a properly suffixed AppModelId after Sam's change lands.

[gab, 2012/11/02 03:47:15]

The problem that you will run into however is that, although you can update the
shortcut saved in user_data_dir, you can't update the shortcut on the taskbar
itself (at least not easily); so if you prefer for it to be easy to keep
updating your dogfood users to stable without having shortcut issues (i.e.
appids dont match) then it's easier to have the right appid from the beginning.

+                                                   profile_path);
+}
+
 string16 ShellIntegration::GetChromiumIconPath() {
   // Determine the app path. If we can't determine what that is, we have
   // bigger fish to fry...