Do not create shortcuts on Windows if the "no-first-run" switch is on.

chrome/browser/profiles/profile_manager.cc

Index: chrome/browser/profiles/profile_manager.cc
===================================================================
--- chrome/browser/profiles/profile_manager.cc	(revision 111737)
+++ chrome/browser/profiles/profile_manager.cc	(working copy)
@@ -214,7 +214,9 @@
 ProfileManager::~ProfileManager() {
   BrowserList::RemoveObserver(this);
 #if defined(OS_WIN)
-  profile_info_cache_->RemoveObserver(profile_shortcut_manager_.get());
+  const CommandLine& command_line = *CommandLine::ForCurrentProcess();
+  if (!command_line.HasSwitch(switches::kNoFirstRun))

[robertshield, 2011/11/30 20:07:48]

I would replace the condition with 
if (profile_shortcut_manager_.get()) {}

What we really want to do is remove the profile_shortcut_manager_ from the
observer list if it exists.

[Miranda Callahan, 2011/11/30 20:18:24]

Indeed! Fixed.

+    profile_info_cache_->RemoveObserver(profile_shortcut_manager_.get());
 #endif
 }
 
@@ -593,8 +595,11 @@
     profile_info_cache_.reset(new ProfileInfoCache(
         g_browser_process->local_state(), user_data_dir_));
 #if defined(OS_WIN)
-    profile_shortcut_manager_.reset(new ProfileShortcutManagerWin());
-    profile_info_cache_->AddObserver(profile_shortcut_manager_.get());
+    const CommandLine& command_line = *CommandLine::ForCurrentProcess();
+    if (!command_line.HasSwitch(switches::kNoFirstRun)) {
+      profile_shortcut_manager_.reset(new ProfileShortcutManagerWin());
+      profile_info_cache_->AddObserver(profile_shortcut_manager_.get());
+    }
 #endif
   }
   return *profile_info_cache_.get();