Commit a62b3627 authored by gab@chromium.org's avatar gab@chromium.org

Use GetActiveDesktop instead of hardcoding HOST_DESKTOP_TYPE_NATIVE in tests.

BUG=179830
TEST=Fixes StartupBrowserCreatorTest.OpenURLsPopup for Ash browser tests.

Review URL: https://chromiumcodereview.appspot.com/19702003

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@213373 0039d316-1c4b-4281-b951-d872f2087c98
parent 204d8ee0
...@@ -180,8 +180,7 @@ IN_PROC_BROWSER_TEST_F(StartupBrowserCreatorTest, OpenURLsPopup) { ...@@ -180,8 +180,7 @@ IN_PROC_BROWSER_TEST_F(StartupBrowserCreatorTest, OpenURLsPopup) {
StartupBrowserCreatorImpl launch(base::FilePath(), dummy, first_run); StartupBrowserCreatorImpl launch(base::FilePath(), dummy, first_run);
// This should create a new window, but re-use the profile from |popup|. If // This should create a new window, but re-use the profile from |popup|. If
// it used a NULL or invalid profile, it would crash. // it used a NULL or invalid profile, it would crash.
launch.OpenURLsInBrowser(popup, false, urls, launch.OpenURLsInBrowser(popup, false, urls, chrome::GetActiveDesktop());
chrome::HOST_DESKTOP_TYPE_NATIVE);
ASSERT_NE(popup, observer.added_browser_); ASSERT_NE(popup, observer.added_browser_);
BrowserList::RemoveObserver(&observer); BrowserList::RemoveObserver(&observer);
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment