Trying to activate a window in a workspace other than the current is ignored while a system modal d…

ash/wm/workspace_controller.cc

Index: ash/wm/workspace_controller.cc
diff --git a/ash/wm/workspace_controller.cc b/ash/wm/workspace_controller.cc
index c880663ac8c3c7de9b4c9084f953f65a5a55f842..590abb16cbc4f8f88b29a08ad264ea80174599e7 100644
--- a/ash/wm/workspace_controller.cc
+++ b/ash/wm/workspace_controller.cc
@@ -4,6 +4,7 @@
 
 #include "ash/wm/workspace_controller.h"
 
+#include "ash/shell.h"
 #include "ash/wm/window_util.h"
 #include "ash/wm/workspace/workspace_manager.h"
 #include "ui/aura/client/activation_client.h"
@@ -58,5 +59,17 @@ void WorkspaceController::OnWindowActivated(aura::Window* gained_active,
   }
 }
 
+void WorkspaceController::OnAttemptToReactivateWindow(
+    aura::Window* request_active, aura::Window* actual_active) {

[sky, 2013/04/23 21:36:42]

same comment here.

[sschmitz, 2013/04/24 01:23:43]

Done.

+  if (!request_active || request_active == actual_active)

[sky, 2013/04/23 21:36:42]

Do we really pass in NULL in some case?

[sschmitz, 2013/04/24 01:23:43]

Not anymore.
Done.

+    return;
+  if (ash::Shell::GetInstance()->IsSystemModalWindowOpen()) {
+    // Handle the case that the requested window was not activated because

[sky, 2013/04/23 21:36:42]

How about something like:
While a system model dialog is showing requests to activate
a window other than that of the modal dialog fail. We only switch the active
workspace when activation changes. The result is the active workspace never
changes while a system modal dialog is showing. This code ensures the active
workspace changes even though activation doesn't change away from the system
model dialog.

[sschmitz, 2013/04/24 01:23:43]

Done.

+    // of a system model window. We still want to activate the workspace
+    // of the requested window.
+    workspace_manager_->SetActiveWorkspaceByWindow(request_active);
+  }
+}
+
 }  // namespace internal
 }  // namespace ash