Chrome OS: New window on teleported browser window should show on current desktop

chrome/browser/ui/browser_navigator.cc

Index: chrome/browser/ui/browser_navigator.cc
diff --git a/chrome/browser/ui/browser_navigator.cc b/chrome/browser/ui/browser_navigator.cc
index f7714d42f7fde86c2987c6a939fedbce25dd32dd..cf6f21325e2c05e1603bac6ae932fce719657ea1 100644
--- a/chrome/browser/ui/browser_navigator.cc
+++ b/chrome/browser/ui/browser_navigator.cc
@@ -39,7 +39,7 @@
 #include "extensions/features/features.h"
 
 #if defined(USE_ASH)
-#include "chrome/browser/ui/ash/multi_user/multi_user_window_manager.h"
+#include "chrome/browser/ui/ash/multi_user/multi_user_util.h"
 #include "components/signin/core/account_id/account_id.h"
 #endif
 
@@ -437,26 +437,10 @@ void Navigate(NavigateParams* params) {
     return;
 
 #if defined(USE_ASH)
-  if (source_browser && source_browser != params->browser) {
-    // When the newly created browser was spawned by a browser which visits
-    // another user's desktop, it should be shown on the same desktop as the
-    // originating one. (This is part of the desktop separation per profile).
-    MultiUserWindowManager* manager = MultiUserWindowManager::GetInstance();
-    // Some unit tests have no manager instantiated.
-    if (manager) {
-      aura::Window* src_window = source_browser->window()->GetNativeWindow();
-      aura::Window* new_window = params->browser->window()->GetNativeWindow();
-      const AccountId& src_account_id =
-          manager->GetUserPresentingWindow(src_window);
-      if (src_account_id != manager->GetUserPresentingWindow(new_window)) {
-        // Once the window gets presented, it should be shown on the same
-        // desktop as the desktop of the creating browser. Note that this
-        // command will not show the window if it wasn't shown yet by the
-        // browser creation.
-        manager->ShowWindowForUser(new_window, src_account_id);
-      }
-    }
-  }
+  // In ChromeOS multiprofile scenario we might need to teleport the window back
+  // to the current user desktop.
+  aura::Window* new_window = params->browser->window()->GetNativeWindow();
+  multi_user_util::MoveWindowToCurrentDesktop(new_window);

[sky, 2017/01/20 01:03:08]

This seems wrong to me. When we create the browser don't we supply context that
corresponds to the active desktop so that it appears in the right place?

[oshima, 2017/01/24 21:22:58]

I agree. warx@, please see my comment in the bug.

[Mr4D (OOO till 08-26), 2017/01/25 18:08:44]

If I remember it correctly there was the possibility that a chrome window on a
desktop of a (could be hidden) user spawned another window and that should show
up on the desktop on where the chrome window is. So e.g.: User clicks on query -
which processes the result for a second in which the user switches to another
desktop. Then the resulting window shows up - but it should be presented on the
desktop of the originating browser - not the current one.

 #endif
 
   // Navigate() must not return early after this point.