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 93 matching lines @@
 #if defined(OS_MACOSX)
 #include "base/mac/mac_util.h"
 #include "chrome/browser/ui/cocoa/keystone_infobar_delegate.h"
 #endif
 
 #if defined(TOOLKIT_GTK)
 #include "chrome/browser/ui/gtk/gtk_util.h"
 #endif
 
 #if defined(OS_WIN)
+#include "base/win/metro.h"
 #include "base/win/windows_version.h"
 #endif
 
 using content::ChildProcessSecurityPolicy;
 using content::WebContents;
 using extensions::Extension;
 using protector::ProtectedPrefsWatcher;
 using protector::ProtectorService;
 using protector::ProtectorServiceFactory;
 
@@ skipping 167 matching lines @@
     DCHECK_EQ(type, content::NOTIFICATION_WEB_CONTENTS_DESTROYED);
     contents_ = NULL;
   }
 
   content::WebContents* contents_;
   content::NotificationRegistrar registrar_;
 
   DISALLOW_COPY_AND_ASSIGN(WebContentsCloseObserver);
 };
 
+// Platform specific helper to activate an existing Browser instance for the
+// profile passed in. Defaults to returning false. For Windows 8 in metro mode
+// we activate an existing browser if the startup urls contains only the new
+// tab page.
+#if defined(OS_WIN) && !defined(USE_AURA)
+bool PlatformActivateBrowserForProfile(
+    Profile* profile, const std::vector<GURL>& startup_urls) {
+  if (!base::win::IsMetroProcess())
+    return false;
+
+  // We activate an existing browser window if we are opening just the new tab
+  // page in metro mode.
+  if (startup_urls.size() > 1)
+    return false;
+
+  if (startup_urls[0] != GURL(chrome::kChromeUINewTabURL))
+    return false;
+
+  Browser* browser = browser::FindBrowserWithProfile(
+      profile, chrome::HOST_DESKTOP_TYPE_NATIVE);
+
+  if (!browser)
+    return false;
+
+  browser->window()->Show();
+}
+#else
+bool PlatformActivateBrowserForProfile(
+    Profile* profile, const std::vector<GURL>& startup_urls) {
+  return false;
+}
+#endif
+
 }  // namespace
 
 StartupBrowserCreatorImpl::StartupBrowserCreatorImpl(
     const FilePath& cur_dir,
     const CommandLine& command_line,
     chrome::startup::IsFirstRun is_first_run)
     : cur_dir_(cur_dir),
       command_line_(command_line),
       profile_(NULL),
       browser_creator_(NULL),
@@ skipping 299 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 (PlatformActivateBrowserForProfile(profile_, adjust_urls))

[sky, 2012/10/16 15:46:30]

Name this OpenStartupURLsInExistingBrowser and move the implementation to
startup_browser_creator_impl_win.

[ananta, 2012/10/16 18:44:49]

The function does not open the startup urls. It activates the existing browser
instance if the url being opened is the  new tab page. I moved the functionality
to the startup_browser_creator_impl_win.cc file as you suggested. This is
implemented via a static member function
StartupBrowserCreatorImpl::ActivateBrowserForProfile.

[sky, 2012/10/16 21:11:08]

Logically this method is called during startup to process the startup URLs.
That's why I suggested naming it OpenStartupURLsInExistingBrowser.
PlatformActivateBrowserForProfile is named specified to what we want to do on
windows. You may as well call it ActivateBrowserForProfileOnMetro.
OpenStartupURLsInExistingBrowser is a much more descriptive name.

[ananta, 2012/10/16 21:17:44]

Fair enough. Done.

+      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));
   }
 }