ash: fix regression where ctrl+n put new window on wrong desktop

chrome/browser/ui/tab_contents/tab_contents_iterator_unittest.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/tab_contents/tab_contents_iterator.h"
 
 #include <stddef.h>
 
 #include "build/build_config.h"
 #include "chrome/browser/browser_process.h"
@@ skipping 30 matching lines @@
 
 }  // namespace
 
 TEST_F(BrowserListTest, TabContentsIteratorVerifyCount) {
   // Make sure we have 1 window to start with.
   EXPECT_EQ(1U, BrowserList::GetInstance()->size());
 
   EXPECT_EQ(0U, CountAllTabs());
 
   // Create more browsers/windows.
-  Browser::CreateParams native_params(profile());
+  Browser::CreateParams native_params(profile(), true);
   std::unique_ptr<Browser> browser2(
       chrome::CreateBrowserWithTestWindowForParams(&native_params));
   // Create browser 3 and 4 on the Ash desktop (the TabContentsIterator
   // shouldn't see the difference).
-  Browser::CreateParams ash_params(profile());
+  Browser::CreateParams ash_params(profile(), true);
   std::unique_ptr<Browser> browser3(
       chrome::CreateBrowserWithTestWindowForParams(&ash_params));
   std::unique_ptr<Browser> browser4(
       chrome::CreateBrowserWithTestWindowForParams(&ash_params));
 
   // Sanity checks.
   EXPECT_EQ(4U, BrowserList::GetInstance()->size());
   EXPECT_EQ(0, browser()->tab_strip_model()->count());
   EXPECT_EQ(0, browser2->tab_strip_model()->count());
   EXPECT_EQ(0, browser3->tab_strip_model()->count());
@@ skipping 20 matching lines @@
   EXPECT_EQ(42U, CountAllTabs());
   // Close all remaining tabs to keep all the destructors happy.
   browser3->tab_strip_model()->CloseAllTabs();
 }
 
 TEST_F(BrowserListTest, TabContentsIteratorVerifyBrowser) {
   // Make sure we have 1 window to start with.
   EXPECT_EQ(1U, BrowserList::GetInstance()->size());
 
   // Create more browsers/windows.
-  Browser::CreateParams native_params(profile());
+  Browser::CreateParams native_params(profile(), true);
   std::unique_ptr<Browser> browser2(
       chrome::CreateBrowserWithTestWindowForParams(&native_params));
   // Create browser 3 on the Ash desktop (the TabContentsIterator shouldn't see
   // the difference).
-  Browser::CreateParams ash_params(profile());
+  Browser::CreateParams ash_params(profile(), true);
   std::unique_ptr<Browser> browser3(
       chrome::CreateBrowserWithTestWindowForParams(&ash_params));
 
   // Sanity checks.
   EXPECT_EQ(3U, BrowserList::GetInstance()->size());
   EXPECT_EQ(0, browser()->tab_strip_model()->count());
   EXPECT_EQ(0, browser2->tab_strip_model()->count());
   EXPECT_EQ(0, browser3->tab_strip_model()->count());
 
   EXPECT_EQ(0U, CountAllTabs());
@@ skipping 83 matching lines @@
 
   chrome::AttemptRestart();
   EXPECT_TRUE(testing_pref_service.GetBoolean(prefs::kWasRestarted));
 
   // Cancel the effects of us calling chrome::AttemptRestart. Otherwise tests
   // ran after this one will fail.
   browser_shutdown::SetTryingToQuit(false);
 
   testing_browser_process->SetLocalState(NULL);
 }