Create/Delete windows profile shortcuts

chrome/browser/chrome_browser_main_win.cc

Index: chrome/browser/chrome_browser_main_win.cc
===================================================================
--- chrome/browser/chrome_browser_main_win.cc	(revision 150342)
+++ chrome/browser/chrome_browser_main_win.cc	(working copy)
@@ -25,7 +25,7 @@
 #include "chrome/browser/media_gallery/media_device_notifications_window_win.h"
 #include "chrome/browser/metrics/metrics_service.h"
 #include "chrome/browser/profiles/profile_info_cache.h"
-#include "chrome/browser/profiles/profile_shortcut_manager_win.h"
+#include "chrome/browser/profiles/profile_shortcut_manager.h"
 #include "chrome/browser/ui/simple_message_box.h"
 #include "chrome/browser/ui/uninstall_browser_prompt.h"
 #include "chrome/common/chrome_constants.h"
@@ -132,11 +132,11 @@
             dist, ShellUtil::CURRENT_USER, ShellUtil::SHORTCUT_NO_OPTIONS)) {
       VLOG(1) << "Failed to delete desktop shortcut.";
     }
-    if (!ShellUtil::RemoveChromeDesktopShortcutsWithAppendedNames(
+    /*if (!ShellUtil::RemoveChromeDesktopShortcutsWithAppendedNames(
         ProfileShortcutManagerWin::GenerateShortcutsFromProfiles(
             ProfileInfoCache::GetProfileNames()))) {
       VLOG(1) << "Failed to delete desktop profiles shortcuts.";
-    }
+    }*/

[SteveT, 2012/08/09 18:07:39]

Is this temporary? (will this code be uncommented when this patch is committed?)

We generally try to avoid hunks of commented-out code unless they're actually
for documentation.

[Halli, 2012/08/09 18:12:30]

I just missed this one when deleting old code, thanks!
On 2012/08/09 18:07:39, SteveT wrote:

     if (!ShellUtil::RemoveChromeQuickLaunchShortcut(dist,
                                                     ShellUtil::CURRENT_USER)) {
       VLOG(1) << "Failed to delete quick launch shortcut.";