Move app launcher and chrome apps shortcut strings into the installer

chrome/browser/web_applications/web_app_win.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.
 
 #include "chrome/browser/web_applications/web_app.h"
 
 #include <shlobj.h>
 
 #include "base/command_line.h"
 #include "base/file_util.h"
 #include "base/logging.h"
 #include "base/md5.h"
 #include "base/path_service.h"
 #include "base/stringprintf.h"
 #include "base/utf_string_conversions.h"
 #include "base/win/shortcut.h"
 #include "base/win/windows_version.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/installer/launcher_support/chrome_launcher_support.h"
 #include "chrome/installer/util/browser_distribution.h"
+#include "chrome/installer/util/install_util.h"
 #include "chrome/installer/util/util_constants.h"
 #include "content/public/browser/browser_thread.h"
 #include "ui/gfx/icon_util.h"
 #include "ui/gfx/image/image.h"
 #include "ui/gfx/image/image_family.h"
 
 namespace {
 
 const base::FilePath::CharType kIconChecksumFileExt[] =
     FILE_PATH_LITERAL(".ico.md5");
@@ skipping 130 matching lines @@
 }
 
 bool CreatePlatformShortcuts(
     const base::FilePath& web_app_path,
     const ShellIntegration::ShortcutInfo& shortcut_info,
     const ShellIntegration::ShortcutLocations& creation_locations) {
   DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::FILE));
 
   // Shortcut paths under which to create shortcuts.
   std::vector<base::FilePath> shortcut_paths =
-      GetShortcutPaths(creation_locations);
+      GetShortcutPaths(creation_locations, ShellUtil::CURRENT_USER);
 
   bool pin_to_taskbar = creation_locations.in_quick_launch_bar &&
                         (base::win::GetVersion() >= base::win::VERSION_WIN7);
 
   // Create/update the shortcut in the web app path for the "Pin To Taskbar"
   // option in Win7. We use the web app path shortcut because we will overwrite
   // it rather than appending unique numbers if the shortcut already exists.
   // This prevents pinned apps from having unique numbers in their names.
   if (pin_to_taskbar)
     shortcut_paths.push_back(web_app_path);
@@ skipping 103 matching lines @@
 void DeletePlatformShortcuts(
     const base::FilePath& web_app_path,
     const ShellIntegration::ShortcutInfo& shortcut_info) {
   DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::FILE));
 
   // Get all possible locations for shortcuts.
   ShellIntegration::ShortcutLocations all_shortcut_locations;
   all_shortcut_locations.in_applications_menu = true;
   all_shortcut_locations.in_quick_launch_bar = true;
   all_shortcut_locations.on_desktop = true;
-  std::vector<base::FilePath> shortcut_paths = GetShortcutPaths(
-      all_shortcut_locations);
+  std::vector<base::FilePath> shortcut_paths =
+      GetShortcutPaths(all_shortcut_locations, ShellUtil::CURRENT_USER);
   if (base::win::GetVersion() >= base::win::VERSION_WIN7)
     shortcut_paths.push_back(web_app_path);
 
   for (std::vector<base::FilePath>::const_iterator i = shortcut_paths.begin();
        i != shortcut_paths.end(); ++i) {
     std::vector<base::FilePath> shortcut_files =
         MatchingShortcutsForProfileAndExtension(*i, shortcut_info.profile_path,
             shortcut_info.title);
     for (std::vector<base::FilePath>::const_iterator j = shortcut_files.begin();
          j != shortcut_files.end(); ++j) {
       // Any shortcut could have been pinned, either by chrome or the user, so
       // they are all unpinned.
       base::win::TaskbarUnpinShortcutLink(j->value().c_str());
       file_util::Delete(*j, false);
     }
   }
 }
 
 std::vector<base::FilePath> GetShortcutPaths(
-    const ShellIntegration::ShortcutLocations& creation_locations) {
+    const ShellIntegration::ShortcutLocations& creation_locations,
+    const ShellUtil::ShellChange install_level) {
   // Shortcut paths under which to create shortcuts.
   std::vector<base::FilePath> shortcut_paths;
   // Locations to add to shortcut_paths.
   struct {
     bool use_this_location;
-    int location_id;
+    ShellUtil::ShortcutLocation location_id;
     const wchar_t* subdir;
   } locations[] = {
     {
       creation_locations.on_desktop,
-      base::DIR_USER_DESKTOP,
+      ShellUtil::SHORTCUT_LOCATION_DESKTOP,
       NULL
     }, {
       creation_locations.in_applications_menu,
-      base::DIR_START_MENU,
+      ShellUtil::SHORTCUT_LOCATION_START_MENU,

[gab, 2013/04/26 15:21:03]

SHORTCUT_LOCATION_START_MENU_CHROME_DIR sounds good to me.

Which shortcuts are not going in the Chrome dir? Only Chrome and Applist are in
the Google Chrome folder and all apps are outside of it?

[calamity, 2013/04/29 07:20:05]

mgiuca@ is making chrome app start menu shortcuts get created under a top-level
folder tentatively named 'Chrome Apps'.

In fact, there's probably no reason to provide full access to the start menu.
I'll just add a TODO for him to add another value to the
ShellUtil::ShortcutLocations enum.

He's also brought to my attention that having an arbitrary subdirectory breaks
the shortcut cleanup paradigm of "Search everywhere we could have made
shortcuts".

Do you think it's still worth changing the enum value name?

       creation_locations.applications_menu_subdir.empty() ? NULL :
           creation_locations.applications_menu_subdir.c_str()
     }, {
-      creation_locations.in_quick_launch_bar,
-      // For Win7, in_quick_launch_bar means pinning to taskbar. Use
-      // base::PATH_START as a flag for this case.
-      (base::win::GetVersion() >= base::win::VERSION_WIN7) ?
-          base::PATH_START : base::DIR_APP_DATA,
-      (base::win::GetVersion() >= base::win::VERSION_WIN7) ?
-          NULL : L"Microsoft\\Internet Explorer\\Quick Launch"
+      // For Win7, in_quick_launch_bar means pinning to taskbar. This needs to

[gab, 2013/04/26 15:21:03]

I don't think that this comment is correct.
Quick Launch is NOT the taskbar, they are different folders and both exist on
Win7. Unless what is meant here is that the option in the UI is changed to say
"taskbar", but it is still |in_quick_launch_bar| that gets set to true if the
user sets that for web app shortcuts??

[calamity, 2013/04/29 07:20:05]

Yes, in_quick_launch_bar is used as "pin to taskbar" in
ShellIntegration::ShortcutLocations for Win7+. It is ignored by non-windows
platforms.

+      // be handled by callers.
+      creation_locations.in_quick_launch_bar &&
+          base::win::GetVersion() < base::win::VERSION_WIN7,
+      ShellUtil::SHORTCUT_LOCATION_QUICK_LAUNCH,
+      NULL
     }
   };
+
+  BrowserDistribution* dist = BrowserDistribution::GetDistribution();
   // Populate shortcut_paths.
   for (int i = 0; i < arraysize(locations); ++i) {
     if (locations[i].use_this_location) {
       base::FilePath path;
 
-      // Skip the Win7 case.
-      if (locations[i].location_id == base::PATH_START)
-        continue;
-
-      if (!PathService::Get(locations[i].location_id, &path)) {
+      if (!ShellUtil::GetShortcutPath(locations[i].location_id,
+                                      dist,
+                                      install_level,
+                                      &path)) {
+        NOTREACHED();
         continue;
       }
 
       if (locations[i].subdir != NULL)
         path = path.Append(locations[i].subdir);
       shortcut_paths.push_back(path);
     }
   }
   return shortcut_paths;
 }
 
 }  // namespace internals
 
 }  // namespace web_app