When user presses Ctrl-N, we should use the current active window's profile to determine on which p…

chrome/browser/ui/ash/chrome_new_window_delegate.cc

 // Copyright 2013 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/ash/chrome_new_window_delegate.h"
 
 #include "ash/wm/window_util.h"
 #include "base/macros.h"
 #include "chrome/browser/profiles/profile_manager.h"
 #include "chrome/browser/sessions/tab_restore_service_factory.h"
@@ skipping 79 matching lines @@
     chrome::ScopedTabbedBrowserDisplayer displayer(
         ProfileManager::GetActiveUserProfile());
     browser = displayer.browser();
     chrome::NewTab(browser);
   }
 
   browser->SetFocusToLocationBar(false);
 }
 
 void ChromeNewWindowDelegate::NewWindow(bool is_incognito) {
-  Profile* profile = ProfileManager::GetActiveUserProfile();
+  Profile* profile;
+  Browser* browser = GetBrowserForActiveWindow();
+  if (browser && browser->profile())
+    profile = browser->profile();
+  else
+    profile = ProfileManager::GetActiveUserProfile();

[oshima, 2016/05/06 18:44:14]

optional: conditional operator? 

[xdai1, 2016/05/06 20:25:32]

Done.

   chrome::NewEmptyWindow(is_incognito ? profile->GetOffTheRecordProfile()
                                       : profile);
 }
 
 void ChromeNewWindowDelegate::RestoreTab() {
   if (tab_restore_helper_.get()) {
     DCHECK(!tab_restore_helper_->tab_restore_service()->IsLoaded());
     return;
   }
 
@@ skipping 16 matching lines @@
   }
 }
 
 void ChromeNewWindowDelegate::ShowTaskManager() {
   chrome::OpenTaskManager(NULL);
 }
 
 void ChromeNewWindowDelegate::OpenFeedbackPage() {
   chrome::OpenFeedbackDialog(GetBrowserForActiveWindow());
 }