If chrome in desktop mode on Windows 8 defers to Windows 8 metro chrome, then we only want to creat…

chrome/browser/ui/startup/startup_browser_creator_impl.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/ui/startup/startup_browser_creator_impl.h"
 
 #include <algorithm>
 #include <vector>
 
 #include "base/bind.h"
@@ skipping 599 matching lines @@
         return;
       }
     }
   }
 
   // Session startup didn't occur, open the urls.
   Browser* browser = NULL;
   std::vector<GURL> adjust_urls = urls_to_open;
   if (adjust_urls.empty()) {
     AddStartupURLs(&adjust_urls);
+    if (StartupBrowserCreatorImpl::OpenStartupURLsInExistingBrowser(
+            profile_, adjust_urls))
+      return;
   } else if (!command_line_.HasSwitch(switches::kOpenInNewWindow)) {
     // Always open a list of urls in a window on the native desktop.
     browser = browser::FindBrowserWithProfile(profile_,
                                               chrome::HOST_DESKTOP_TYPE_NATIVE);
   }
-
   // This will launch a browser; prevent session restore.
   in_synchronous_profile_launch = true;
   browser = OpenURLsInBrowser(browser, process_startup, adjust_urls);
   in_synchronous_profile_launch = false;
   AddInfoBarsIfNecessary(browser, is_process_startup);
 }
 
 bool StartupBrowserCreatorImpl::ProcessStartupURLs(
     const std::vector<GURL>& urls_to_open) {
   VLOG(1) << "StartupBrowserCreatorImpl::ProcessStartupURLs";
@@ skipping 390 matching lines @@
         // New:
         prefs->GetString(prefs::kHomePage),
         prefs->GetBoolean(prefs::kHomePageIsNewTabPage),
         prefs->GetBoolean(prefs::kShowHomeButton),
         // Backup:
         backup_homepage,
         backup_homepage_is_ntp,
         backup_show_home_button));
   }
 }
+
+#if !defined(OS_WIN) || defined(USE_AURA)
+// static
+bool StartupBrowserCreatorImpl::OpenStartupURLsInExistingBrowser(
+    Profile* profile, const std::vector<GURL>& startup_urls) {

[sky, 2012/10/16 22:06:15]

Same thing about wrapping here. Also, why the USE_AURA ifdef? Don't we want this
on windows with aura? I'm assuming at some point we'll have aura working with
metro and then we would want this logic, right?

[ananta, 2012/10/16 22:14:58]

Fixed the wrapping. Regarding AURA this code path is mostly invoked from desktop
chrome which delegates to metro. Under AURA it is very likely that desktop
chrome would put a viewer on similar lines as metro. We can revisit this once we
have details flushed out on that.

+  return false;
+}
+#endif
+