Ensure we have a valid profile when using

BrowserInit::LaunchWithProfile::OpenURLsInBrowser() directly and the browser
passed in is a popup window.

Original Review URL: http://codereview.chromium.org/160331

BUG=http://crbug.com/17520
TEST=see bug.

Review URL: http://codereview.chromium.org/164341

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@23077 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 60 insertions, 0 deletions

diff --git a/chrome/browser/browser_init_browsertest.cc b/chrome/browser/browser_init_browsertest.cc
new file mode 100644
index 0000000..7a32141
--- /dev/null
+++ b/chrome/browser/browser_init_browsertest.cc
@@ -0,0 +1,60 @@
+// Copyright (c) 2009 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/browser.h"
+#include "chrome/browser/browser_init.h"
+#include "chrome/browser/browser_list.h"
+#include "chrome/browser/browser_window.h"
+#include "chrome/browser/views/frame/browser_view.h"
+#include "chrome/test/in_process_browser_test.h"
+#include "testing/gtest/include/gtest/gtest.h"
+
+namespace {
+
+class BrowserInitTest : public InProcessBrowserTest {
+};
+
+class OpenURLsPopupObserver : public BrowserList::Observer {
+ public:
+  OpenURLsPopupObserver() : added_browser_(NULL) { }
+
+  virtual void OnBrowserAdded(const Browser* browser) {
+    added_browser_ = browser;
+  }
+
+  virtual void OnBrowserRemoving(const Browser* browser) { }
+
+  const Browser* added_browser_;
+};
+
+// Test that when there is a popup as the active browser any requests to
+// BrowserInit::LaunchWithProfile::OpenURLsInBrowser don't crash because
+// there's no explicit profile given.
+IN_PROC_BROWSER_TEST_F(BrowserInitTest, OpenURLsPopup) {
+  std::vector<GURL> urls;
+  urls.push_back(GURL("http://www.google.com"));
+  urls.push_back(GURL("http://dev.chromium.org"));
+
+  // Note that in our testing we do not ever query the BrowserList for the "last
+  // active" browser. That's because the browsers are set as "active" by
+  // platform UI toolkit messages, and those messages are not sent during unit
+  // testing sessions.
+
+  OpenURLsPopupObserver observer;
+  BrowserList::AddObserver(&observer);
+
+  Browser* popup = Browser::CreateForPopup(browser()->profile());
+  ASSERT_EQ(popup->type(), Browser::TYPE_POPUP);
+  ASSERT_EQ(popup, observer.added_browser_);
+
+  CommandLine dummy((std::wstring()));
+  BrowserInit::LaunchWithProfile launch(std::wstring(), dummy);
+  // This should create a new window, but re-use the profile from |popup|. If
+  // it used a NULL or invalid profile, it would crash.
+  launch.OpenURLsInBrowser(popup, false, urls);
+  ASSERT_NE(popup, observer.added_browser_);
+  BrowserList::RemoveObserver(&observer);
+}
+
+}  // namespace