Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(495)

Unified Diff: chrome/browser/app_controller_mac.mm

Issue 2685333005: ash: fix regression where ctrl+n put new window on wrong desktop (Closed)
Patch Set: Rebase to ToT Created 3 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | chrome/browser/captive_portal/captive_portal_browsertest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/app_controller_mac.mm
diff --git a/chrome/browser/app_controller_mac.mm b/chrome/browser/app_controller_mac.mm
index 4f248343f7f799dd5ff33036365c7cdfb5f55228..84165c5d9ece4a370c5d4b8dec91b86e22bde931 100644
--- a/chrome/browser/app_controller_mac.mm
+++ b/chrome/browser/app_controller_mac.mm
@@ -799,7 +799,7 @@ class AppControllerProfileObserver : public ProfileAttributesStorage::Observer {
// downloads page if the user chooses to wait.
Browser* browser = chrome::FindBrowserWithProfile(profiles[i]);
if (!browser) {
- browser = new Browser(Browser::CreateParams(profiles[i]));
+ browser = new Browser(Browser::CreateParams(profiles[i], true));
browser->window()->Show();
}
DCHECK(browser);
@@ -1282,7 +1282,7 @@ class AppControllerProfileObserver : public ProfileAttributesStorage::Observer {
Browser* browser = chrome::GetLastActiveBrowser();
// if no browser window exists then create one with no tabs to be filled in
if (!browser) {
- browser = new Browser(Browser::CreateParams([self lastProfile]));
+ browser = new Browser(Browser::CreateParams([self lastProfile], true));
browser->window()->Show();
}
« no previous file with comments | « no previous file | chrome/browser/captive_portal/captive_portal_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698