Minimized windows should not be activated by another user in multiprofile scenario.

chrome/browser/ui/ash/session_state_delegate_chromeos.cc

 // Copyright (c) 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/session_state_delegate_chromeos.h"
 
 #include "ash/multi_profile_uma.h"
 #include "ash/session/session_state_observer.h"
 #include "ash/system/chromeos/multi_user/user_switch_util.h"
 #include "base/bind.h"
 #include "base/callback.h"
 #include "base/command_line.h"
 #include "base/logging.h"
 #include "base/prefs/pref_service.h"
 #include "chrome/browser/chromeos/login/lock/screen_locker.h"
 #include "chrome/browser/chromeos/login/ui/user_adding_screen.h"
 #include "chrome/browser/chromeos/login/users/multi_profile_user_controller.h"
 #include "chrome/browser/chromeos/profiles/profile_helper.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/profiles/profile_manager.h"
+#include "chrome/browser/ui/ash/multi_user/multi_user_util.h"
 #include "chrome/browser/ui/ash/multi_user/multi_user_window_manager.h"
 #include "chrome/common/pref_names.h"
 #include "chromeos/chromeos_switches.h"
 #include "chromeos/dbus/dbus_thread_manager.h"
 #include "chromeos/dbus/session_manager_client.h"
 #include "chromeos/login/login_state.h"
 #include "components/user_manager/user.h"
 #include "components/user_manager/user_manager.h"
 #include "google_apis/gaia/gaia_auth_util.h"
 
@@ skipping 26 matching lines @@
       user_manager::UserManager::Get()->GetLRULoggedInUsers()[index];
   CHECK(user);
   return chromeos::ProfileHelper::Get()->GetProfileByUserUnsafe(user);
 }
 
 content::BrowserContext*
 SessionStateDelegateChromeos::GetBrowserContextForWindow(
     aura::Window* window) {
   const std::string& user_id =
       chrome::MultiUserWindowManager::GetInstance()->GetWindowOwner(window);
-  const user_manager::User* user =
-      user_manager::UserManager::Get()->FindUser(user_id);
-  DCHECK(user);
-  return chromeos::ProfileHelper::Get()->GetProfileByUserUnsafe(user);
+  return user_id.empty() ? NULL
+                         : multi_user_util::GetProfileFromUserID(user_id);

[xdai1, 2015/04/10 22:56:07]

I think it might be more cleaner to get profile this way? The user_id will be
empty string if the window is special (for example, the system tray bubble
window, login window, etc)

[oshima, 2015/04/10 23:44:36]

lgtm

+}
+
+content::BrowserContext*
+SessionStateDelegateChromeos::GetUserPresentingBrowserContextForWindow(
+    aura::Window* window) {
+  const std::string& user_id =
+      chrome::MultiUserWindowManager::GetInstance()->GetUserPresentingWindow(
+          window);
+  return user_id.empty() ? NULL
+                         : multi_user_util::GetProfileFromUserID(user_id);
 }
 
 int SessionStateDelegateChromeos::GetMaximumNumberOfLoggedInUsers() const {
   // We limit list of logged in users to 10 due to memory constraints.
   // Note that 10 seems excessive, but we want to test how many users are
   // actually added to a session.
   // TODO(nkostylev): Adjust this limitation based on device capabilites.
   // http://crbug.com/230865
   return 10;
 }
@@ skipping 203 matching lines @@
 }
 
 void DoSwitchUser(const std::string& user_id) {
   user_manager::UserManager::Get()->SwitchActiveUser(user_id);
 }
 
 void SessionStateDelegateChromeos::TryToSwitchUser(
     const std::string& user_id) {
   ash::TrySwitchingActiveUser(base::Bind(&DoSwitchUser, user_id));
 }