Separated alt-tab window cycle from overview mode

ash/wm/window_cycle_controller_unittest.cc

+// Copyright (c) 2012 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 "ash/wm/window_cycle_controller.h"
+
+#include <algorithm>
+
+#include "ash/session/session_state_delegate.h"
+#include "ash/shelf/shelf.h"
+#include "ash/shelf/shelf_widget.h"
+#include "ash/shell.h"
+#include "ash/shell_window_ids.h"
+#include "ash/test/ash_test_base.h"
+#include "ash/test/shelf_test_api.h"
+#include "ash/test/shelf_view_test_api.h"
+#include "ash/test/test_shelf_delegate.h"
+#include "ash/test/test_shell_delegate.h"
+#include "ash/wm/window_cycle_list.h"
+#include "ash/wm/window_state.h"
+#include "ash/wm/window_util.h"
+#include "base/memory/scoped_ptr.h"
+#include "ui/aura/client/aura_constants.h"
+#include "ui/aura/client/screen_position_client.h"
+#include "ui/aura/env.h"
+#include "ui/aura/test/test_windows.h"
+#include "ui/aura/window.h"
+#include "ui/aura/window_event_dispatcher.h"
+#include "ui/gfx/rect.h"
+#include "ui/gfx/screen.h"
+
+namespace ash {
+
+using aura::test::CreateTestWindowWithId;
+using aura::test::TestWindowDelegate;
+using aura::Window;
+
+class WindowCycleControllerTest : public test::AshTestBase {
+ public:
+  WindowCycleControllerTest() {}
+  virtual ~WindowCycleControllerTest() {}
+
+  virtual void SetUp() OVERRIDE {
+    test::AshTestBase::SetUp();
+    ASSERT_TRUE(test::TestShelfDelegate::instance());
+
+    shelf_view_test_.reset(new test::ShelfViewTestAPI(
+        test::ShelfTestAPI(Shelf::ForPrimaryDisplay()).shelf_view()));
+    shelf_view_test_->SetAnimationDuration(1);
+  }
+
+  aura::Window* CreatePanelWindow() {
+    gfx::Rect rect(100, 100);
+    aura::Window* window = CreateTestWindowInShellWithDelegateAndType(
+        NULL, ui::wm::WINDOW_TYPE_PANEL, 0, rect);
+    test::TestShelfDelegate::instance()->AddShelfItem(window);
+    shelf_view_test_->RunMessageLoopUntilAnimationsDone();
+    return window;
+  }
+
+ private:
+  scoped_ptr<test::ShelfViewTestAPI> shelf_view_test_;
+
+  DISALLOW_COPY_AND_ASSIGN(WindowCycleControllerTest);
+};
+
+TEST_F(WindowCycleControllerTest, HandleCycleWindowBaseCases) {

[tdanderson, 2014/05/01 18:51:39]

Include TEST=... in the issue description with any tests that are brand new, not
just ones that have been changed slightly and moved to a new file. If there are
no new tests, say TEST=none. This makes it easier for your reviewers.

What are the new things I should be looking at in this file? Also, do you have a
test that explicitly checks that OM is exited on Alt+Tab?

[Nina, 2014/05/01 20:41:16]

This unittest class existed in the past. I had to do the following modifications
for it to work with the "new" window_cycle_controller:

* Remove tests related to one button window cycle (we now use that key for the
overview mode).

* Modify usage of HandleWindowCycle function to only take one argument.

* Bring some tests from WindowSelector:
** SelectingHidesAppList
** CyclePreservesMinimization
** CyclePanels
** CyclePanelsDestroyed
** CyclePanelsDestroyed

Apparently panels didn't exist when WindowCycleController was used.

So, I don't really know if we should include TEST=… maybe
TEST=WindowCycleControllerTest.* ?

[tdanderson, 2014/05/01 21:58:33]

Ok, so if you're not adding new tests, then you can say TEST=none. However,
please move the above explanation of what you did into the issue description so
that anyone who looks at this gets a nice high-level summary of what you changed
without needing to dive into the diffs.

+  WindowCycleController* controller =
+      Shell::GetInstance()->window_cycle_controller();
+
+  // Cycling doesn't crash if there are no windows.
+  controller->HandleCycleWindow(WindowCycleController::FORWARD);
+
+  // Create a single test window.
+  scoped_ptr<Window> window0(CreateTestWindowInShellWithId(0));
+  wm::ActivateWindow(window0.get());
+  EXPECT_TRUE(wm::IsActiveWindow(window0.get()));
+
+  // Cycling works for a single window, even though nothing changes.
+  controller->HandleCycleWindow(WindowCycleController::FORWARD);
+  EXPECT_TRUE(wm::IsActiveWindow(window0.get()));
+}
+
+// Verifies if there is only one window and it isn't active that cycling
+// activates it.
+TEST_F(WindowCycleControllerTest, SingleWindowNotActive) {
+  WindowCycleController* controller =
+      Shell::GetInstance()->window_cycle_controller();
+
+  // Create a single test window.
+  scoped_ptr<Window> window0(CreateTestWindowInShellWithId(0));
+  wm::ActivateWindow(window0.get());
+  EXPECT_TRUE(wm::IsActiveWindow(window0.get()));
+
+  // Rotate focus, this should move focus to another window that isn't part of
+  // the default container.
+  Shell::GetInstance()->RotateFocus(Shell::FORWARD);
+  EXPECT_FALSE(wm::IsActiveWindow(window0.get()));
+
+  // Cycling should activate the window.
+  controller->HandleCycleWindow(WindowCycleController::FORWARD);
+  EXPECT_TRUE(wm::IsActiveWindow(window0.get()));
+}
+
+TEST_F(WindowCycleControllerTest, HandleCycleWindow) {
+  WindowCycleController* controller =
+      Shell::GetInstance()->window_cycle_controller();
+
+  // Set up several windows to use to test cycling.  Create them in reverse
+  // order so they are stacked 0 over 1 over 2.
+  scoped_ptr<Window> window2(CreateTestWindowInShellWithId(2));
+  scoped_ptr<Window> window1(CreateTestWindowInShellWithId(1));
+  scoped_ptr<Window> window0(CreateTestWindowInShellWithId(0));
+  wm::ActivateWindow(window0.get());
+
+  // Simulate pressing and releasing Alt-tab.
+  EXPECT_TRUE(wm::IsActiveWindow(window0.get()));
+  controller->HandleCycleWindow(WindowCycleController::FORWARD);
+
+  // Window lists should return the topmost window in front.
+  ASSERT_TRUE(controller->windows());
+  ASSERT_EQ(3u, controller->windows()->windows().size());
+  ASSERT_EQ(window0.get(), controller->windows()->windows()[0]);
+  ASSERT_EQ(window1.get(), controller->windows()->windows()[1]);
+  ASSERT_EQ(window2.get(), controller->windows()->windows()[2]);
+
+  controller->AltKeyReleased();
+  EXPECT_TRUE(wm::IsActiveWindow(window1.get()));
+
+  // Pressing and releasing Alt-tab again should cycle back to the most-
+  // recently-used window in the current child order.
+  controller->HandleCycleWindow(WindowCycleController::FORWARD);
+  controller->AltKeyReleased();
+  EXPECT_TRUE(wm::IsActiveWindow(window0.get()));
+
+  // Pressing Alt-tab multiple times without releasing Alt should cycle through
+  // all the windows and wrap around.
+  controller->HandleCycleWindow(WindowCycleController::FORWARD);
+  EXPECT_TRUE(controller->IsCycling());
+  EXPECT_TRUE(wm::IsActiveWindow(window1.get()));
+
+  controller->HandleCycleWindow(WindowCycleController::FORWARD);
+  EXPECT_TRUE(controller->IsCycling());
+  EXPECT_TRUE(wm::IsActiveWindow(window2.get()));
+
+  controller->HandleCycleWindow(WindowCycleController::FORWARD);
+  EXPECT_TRUE(controller->IsCycling());
+  EXPECT_TRUE(wm::IsActiveWindow(window0.get()));
+
+  controller->AltKeyReleased();
+  EXPECT_FALSE(controller->IsCycling());
+  EXPECT_TRUE(wm::IsActiveWindow(window0.get()));
+
+  // Reset our stacking order.
+  wm::ActivateWindow(window2.get());
+  wm::ActivateWindow(window1.get());
+  wm::ActivateWindow(window0.get());
+
+  // Likewise we can cycle backwards through all the windows.
+  controller->HandleCycleWindow(WindowCycleController::BACKWARD);
+  EXPECT_TRUE(wm::IsActiveWindow(window2.get()));
+  controller->HandleCycleWindow(WindowCycleController::BACKWARD);
+  EXPECT_TRUE(wm::IsActiveWindow(window1.get()));
+  controller->HandleCycleWindow(WindowCycleController::BACKWARD);
+  EXPECT_TRUE(wm::IsActiveWindow(window0.get()));
+  controller->AltKeyReleased();
+  EXPECT_TRUE(wm::IsActiveWindow(window0.get()));
+
+  // When the screen is locked, cycling window does not take effect.
+  Shell::GetInstance()->session_state_delegate()->LockScreen();
+  EXPECT_TRUE(wm::IsActiveWindow(window0.get()));
+  controller->HandleCycleWindow(WindowCycleController::FORWARD);
+  EXPECT_TRUE(wm::IsActiveWindow(window0.get()));
+  controller->HandleCycleWindow(WindowCycleController::BACKWARD);
+  EXPECT_TRUE(wm::IsActiveWindow(window0.get()));
+
+  Shell::GetInstance()->session_state_delegate()->UnlockScreen();
+  EXPECT_TRUE(wm::IsActiveWindow(window0.get()));
+  controller->HandleCycleWindow(WindowCycleController::FORWARD);
+  EXPECT_TRUE(wm::IsActiveWindow(window1.get()));
+  controller->HandleCycleWindow(WindowCycleController::FORWARD);
+  EXPECT_TRUE(wm::IsActiveWindow(window2.get()));
+
+  // When a modal window is active, cycling window does not take effect.
+  aura::Window* modal_container =
+      ash::Shell::GetContainer(
+          Shell::GetPrimaryRootWindow(),
+          kShellWindowId_SystemModalContainer);
+  scoped_ptr<Window> modal_window(
+      CreateTestWindowWithId(-2, modal_container));
+  modal_window->SetProperty(aura::client::kModalKey, ui::MODAL_TYPE_SYSTEM);
+  wm::ActivateWindow(modal_window.get());
+  EXPECT_TRUE(wm::IsActiveWindow(modal_window.get()));
+  controller->HandleCycleWindow(WindowCycleController::FORWARD);
+  EXPECT_TRUE(wm::IsActiveWindow(modal_window.get()));
+  EXPECT_FALSE(wm::IsActiveWindow(window0.get()));
+  EXPECT_FALSE(wm::IsActiveWindow(window1.get()));
+  EXPECT_FALSE(wm::IsActiveWindow(window2.get()));
+  controller->HandleCycleWindow(WindowCycleController::BACKWARD);
+  EXPECT_TRUE(wm::IsActiveWindow(modal_window.get()));
+  EXPECT_FALSE(wm::IsActiveWindow(window0.get()));
+  EXPECT_FALSE(wm::IsActiveWindow(window1.get()));
+  EXPECT_FALSE(wm::IsActiveWindow(window2.get()));
+}
+
+// Cycles between a maximized and normal window.
+TEST_F(WindowCycleControllerTest, MaximizedWindow) {
+  // Create a couple of test windows.
+  scoped_ptr<Window> window0(CreateTestWindowInShellWithId(0));
+  scoped_ptr<Window> window1(CreateTestWindowInShellWithId(1));
+  wm::WindowState* window1_state = wm::GetWindowState(window1.get());
+  window1_state->Maximize();
+  window1_state->Activate();
+  EXPECT_TRUE(window1_state->IsActive());
+
+  // Rotate focus, this should move focus to window0.
+  WindowCycleController* controller =
+      Shell::GetInstance()->window_cycle_controller();
+  controller->HandleCycleWindow(WindowCycleController::FORWARD);
+  EXPECT_TRUE(wm::GetWindowState(window0.get())->IsActive());
+
+  // One more time.
+  controller->HandleCycleWindow(WindowCycleController::FORWARD);
+  EXPECT_TRUE(window1_state->IsActive());
+}
+
+// Cycles to a minimized window.
+TEST_F(WindowCycleControllerTest, Minimized) {
+  // Create a couple of test windows.
+  scoped_ptr<Window> window0(CreateTestWindowInShellWithId(0));
+  scoped_ptr<Window> window1(CreateTestWindowInShellWithId(1));
+  wm::WindowState* window0_state = wm::GetWindowState(window0.get());
+  wm::WindowState* window1_state = wm::GetWindowState(window1.get());
+
+  window1_state->Minimize();
+  window0_state->Activate();
+  EXPECT_TRUE(window0_state->IsActive());
+
+  // Rotate focus, this should move focus to window1 and unminimize it.
+  WindowCycleController* controller =
+      Shell::GetInstance()->window_cycle_controller();
+  controller->HandleCycleWindow(WindowCycleController::FORWARD);
+  EXPECT_FALSE(window1_state->IsMinimized());
+  EXPECT_TRUE(window1_state->IsActive());
+
+  // One more time back to w0.
+  controller->HandleCycleWindow(WindowCycleController::FORWARD);
+  EXPECT_TRUE(window0_state->IsActive());
+}
+
+TEST_F(WindowCycleControllerTest, AlwaysOnTopWindow) {
+  WindowCycleController* controller =
+      Shell::GetInstance()->window_cycle_controller();
+
+  // Set up several windows to use to test cycling.
+  scoped_ptr<Window> window0(CreateTestWindowInShellWithId(0));
+  scoped_ptr<Window> window1(CreateTestWindowInShellWithId(1));
+
+  Window* top_container =
+      Shell::GetContainer(
+          Shell::GetPrimaryRootWindow(),
+          kShellWindowId_AlwaysOnTopContainer);
+  scoped_ptr<Window> window2(CreateTestWindowWithId(2, top_container));
+  wm::ActivateWindow(window0.get());
+
+  // Simulate pressing and releasing Alt-tab.
+  EXPECT_TRUE(wm::IsActiveWindow(window0.get()));
+  controller->HandleCycleWindow(WindowCycleController::FORWARD);
+
+  // Window lists should return the topmost window in front.
+  ASSERT_TRUE(controller->windows());
+  ASSERT_EQ(3u, controller->windows()->windows().size());
+  EXPECT_EQ(window0.get(), controller->windows()->windows()[0]);
+  EXPECT_EQ(window2.get(), controller->windows()->windows()[1]);
+  EXPECT_EQ(window1.get(), controller->windows()->windows()[2]);
+
+  controller->AltKeyReleased();
+  EXPECT_TRUE(wm::IsActiveWindow(window2.get()));
+
+  controller->HandleCycleWindow(WindowCycleController::FORWARD);
+  EXPECT_TRUE(wm::IsActiveWindow(window0.get()));
+
+  controller->AltKeyReleased();
+
+  controller->HandleCycleWindow(WindowCycleController::FORWARD);
+  EXPECT_TRUE(wm::IsActiveWindow(window2.get()));
+
+  controller->HandleCycleWindow(WindowCycleController::FORWARD);
+  EXPECT_TRUE(wm::IsActiveWindow(window1.get()));
+
+  controller->HandleCycleWindow(WindowCycleController::FORWARD);
+  EXPECT_TRUE(wm::IsActiveWindow(window0.get()));
+}
+
+TEST_F(WindowCycleControllerTest, AlwaysOnTopMultiWindow) {
+  WindowCycleController* controller =
+      Shell::GetInstance()->window_cycle_controller();
+
+  // Set up several windows to use to test cycling.
+  scoped_ptr<Window> window0(CreateTestWindowInShellWithId(0));
+  scoped_ptr<Window> window1(CreateTestWindowInShellWithId(1));
+
+  Window* top_container =
+      Shell::GetContainer(
+          Shell::GetPrimaryRootWindow(),
+          kShellWindowId_AlwaysOnTopContainer);
+  scoped_ptr<Window> window2(CreateTestWindowWithId(2, top_container));
+  scoped_ptr<Window> window3(CreateTestWindowWithId(3, top_container));
+  wm::ActivateWindow(window0.get());
+
+  // Simulate pressing and releasing Alt-tab.
+  EXPECT_TRUE(wm::IsActiveWindow(window0.get()));
+  controller->HandleCycleWindow(WindowCycleController::FORWARD);
+
+  // Window lists should return the topmost window in front.
+  ASSERT_TRUE(controller->windows());
+  ASSERT_EQ(4u, controller->windows()->windows().size());
+  EXPECT_EQ(window0.get(), controller->windows()->windows()[0]);
+  EXPECT_EQ(window3.get(), controller->windows()->windows()[1]);
+  EXPECT_EQ(window2.get(), controller->windows()->windows()[2]);
+  EXPECT_EQ(window1.get(), controller->windows()->windows()[3]);
+
+  controller->AltKeyReleased();
+  EXPECT_TRUE(wm::IsActiveWindow(window3.get()));
+
+  controller->HandleCycleWindow(WindowCycleController::FORWARD);
+  EXPECT_TRUE(wm::IsActiveWindow(window0.get()));
+
+  controller->AltKeyReleased();
+
+  controller->HandleCycleWindow(WindowCycleController::FORWARD);
+  EXPECT_TRUE(wm::IsActiveWindow(window3.get()));
+
+  controller->HandleCycleWindow(WindowCycleController::FORWARD);
+  EXPECT_TRUE(wm::IsActiveWindow(window2.get()));
+
+  controller->HandleCycleWindow(WindowCycleController::FORWARD);
+  EXPECT_TRUE(wm::IsActiveWindow(window1.get()));
+
+  controller->HandleCycleWindow(WindowCycleController::FORWARD);
+  EXPECT_TRUE(wm::IsActiveWindow(window0.get()));
+}
+
+TEST_F(WindowCycleControllerTest, AlwaysOnTopMultipleRootWindows) {
+  if (!SupportsMultipleDisplays())
+    return;
+
+  // Set up a second root window
+  UpdateDisplay("1000x600,600x400");
+  aura::Window::Windows root_windows = Shell::GetAllRootWindows();
+  ASSERT_EQ(2U, root_windows.size());
+
+  WindowCycleController* controller =
+      Shell::GetInstance()->window_cycle_controller();
+
+  Shell::GetInstance()->set_target_root_window(root_windows[0]);
+
+  // Create two windows in the primary root.
+  scoped_ptr<Window> window0(CreateTestWindowInShellWithId(0));
+  EXPECT_EQ(root_windows[0], window0->GetRootWindow());
+  Window* top_container0 =
+      Shell::GetContainer(
+          root_windows[0],
+          kShellWindowId_AlwaysOnTopContainer);
+  scoped_ptr<Window> window1(CreateTestWindowWithId(1, top_container0));
+  EXPECT_EQ(root_windows[0], window1->GetRootWindow());
+
+  // And two on the secondary root.
+  Shell::GetInstance()->set_target_root_window(root_windows[1]);
+  scoped_ptr<Window> window2(CreateTestWindowInShellWithId(2));
+  EXPECT_EQ(root_windows[1], window2->GetRootWindow());
+
+  Window* top_container1 =
+      Shell::GetContainer(
+          root_windows[1],
+          kShellWindowId_AlwaysOnTopContainer);
+  scoped_ptr<Window> window3(CreateTestWindowWithId(3, top_container1));
+  EXPECT_EQ(root_windows[1], window3->GetRootWindow());
+
+  // Move the active root window to the secondary.
+  Shell::GetInstance()->set_target_root_window(root_windows[1]);
+
+  wm::ActivateWindow(window2.get());
+
+  EXPECT_EQ(root_windows[0], window0->GetRootWindow());
+  EXPECT_EQ(root_windows[0], window1->GetRootWindow());
+  EXPECT_EQ(root_windows[1], window2->GetRootWindow());
+  EXPECT_EQ(root_windows[1], window3->GetRootWindow());
+
+  // Simulate pressing and releasing Alt-tab.
+  EXPECT_TRUE(wm::IsActiveWindow(window2.get()));
+  controller->HandleCycleWindow(WindowCycleController::FORWARD);
+
+  // Window lists should return the topmost window in front.
+  ASSERT_TRUE(controller->windows());
+  ASSERT_EQ(4u, controller->windows()->windows().size());
+  EXPECT_EQ(window2.get(), controller->windows()->windows()[0]);
+  EXPECT_EQ(window3.get(), controller->windows()->windows()[1]);
+  EXPECT_EQ(window1.get(), controller->windows()->windows()[2]);
+  EXPECT_EQ(window0.get(), controller->windows()->windows()[3]);
+
+  controller->AltKeyReleased();
+  EXPECT_TRUE(wm::IsActiveWindow(window3.get()));
+
+  controller->HandleCycleWindow(WindowCycleController::FORWARD);
+  EXPECT_TRUE(wm::IsActiveWindow(window2.get()));
+
+  controller->AltKeyReleased();
+
+  controller->HandleCycleWindow(WindowCycleController::FORWARD);
+  EXPECT_TRUE(wm::IsActiveWindow(window3.get()));
+
+  controller->HandleCycleWindow(WindowCycleController::FORWARD);
+  EXPECT_TRUE(wm::IsActiveWindow(window1.get()));
+
+  controller->HandleCycleWindow(WindowCycleController::FORWARD);
+  EXPECT_TRUE(wm::IsActiveWindow(window0.get()));
+
+  controller->HandleCycleWindow(WindowCycleController::FORWARD);
+  EXPECT_TRUE(wm::IsActiveWindow(window2.get()));
+}
+
+TEST_F(WindowCycleControllerTest, MostRecentlyUsed) {
+  WindowCycleController* controller =
+      Shell::GetInstance()->window_cycle_controller();
+
+  // Set up several windows to use to test cycling.
+  scoped_ptr<Window> window0(CreateTestWindowInShellWithId(0));
+  scoped_ptr<Window> window1(CreateTestWindowInShellWithId(1));
+  scoped_ptr<Window> window2(CreateTestWindowInShellWithId(2));
+
+  wm::ActivateWindow(window0.get());
+
+  // Simulate pressing and releasing Alt-tab.
+  EXPECT_TRUE(wm::IsActiveWindow(window0.get()));
+  controller->HandleCycleWindow(WindowCycleController::FORWARD);
+
+  // Window lists should return the topmost window in front.
+  ASSERT_TRUE(controller->windows());
+  ASSERT_EQ(3u, controller->windows()->windows().size());
+  EXPECT_EQ(window0.get(), controller->windows()->windows()[0]);
+  EXPECT_EQ(window2.get(), controller->windows()->windows()[1]);
+  EXPECT_EQ(window1.get(), controller->windows()->windows()[2]);
+
+  controller->HandleCycleWindow(WindowCycleController::FORWARD);
+  controller->AltKeyReleased();
+  EXPECT_TRUE(wm::IsActiveWindow(window1.get()));
+
+
+  controller->HandleCycleWindow(WindowCycleController::FORWARD);
+  EXPECT_TRUE(wm::IsActiveWindow(window0.get()));
+
+  controller->AltKeyReleased();
+
+  controller->HandleCycleWindow(WindowCycleController::FORWARD);
+  EXPECT_TRUE(wm::IsActiveWindow(window1.get()));
+
+  controller->HandleCycleWindow(WindowCycleController::FORWARD);
+  EXPECT_TRUE(wm::IsActiveWindow(window2.get()));
+
+  controller->HandleCycleWindow(WindowCycleController::FORWARD);
+  EXPECT_TRUE(wm::IsActiveWindow(window0.get()));
+}
+
+// Tests that beginning window selection hides the app list.
+TEST_F(WindowCycleControllerTest, SelectingHidesAppList) {
+  WindowCycleController* controller =
+      Shell::GetInstance()->window_cycle_controller();
+
+  gfx::Rect bounds(0, 0, 400, 400);
+  scoped_ptr<aura::Window> window0(CreateTestWindowInShellWithId(0));
+  scoped_ptr<aura::Window> window1(CreateTestWindowInShellWithId(1));
+  Shell::GetInstance()->ToggleAppList(NULL);
+  EXPECT_TRUE(Shell::GetInstance()->GetAppListTargetVisibility());
+  controller->HandleCycleWindow(WindowCycleController::FORWARD);
+  EXPECT_FALSE(Shell::GetInstance()->GetAppListTargetVisibility());
+}
+
+// Tests that cycling through windows shows and minimizes windows as they
+// are passed.
+TEST_F(WindowCycleControllerTest, CyclePreservesMinimization) {
+  WindowCycleController* controller =
+      Shell::GetInstance()->window_cycle_controller();
+
+  gfx::Rect bounds(0, 0, 400, 400);
+  scoped_ptr<aura::Window> window0(CreateTestWindowInShellWithId(0));
+  scoped_ptr<aura::Window> window1(CreateTestWindowInShellWithId(1));
+  wm::ActivateWindow(window1.get());
+  wm::GetWindowState(window1.get())->Minimize();
+  wm::ActivateWindow(window0.get());
+  EXPECT_TRUE(wm::IsWindowMinimized(window1.get()));
+
+  // On window 2.
+  controller->HandleCycleWindow(WindowCycleController::FORWARD);
+  EXPECT_FALSE(wm::IsWindowMinimized(window1.get()));
+
+  // Back on window 1.
+  controller->HandleCycleWindow(WindowCycleController::FORWARD);
+  EXPECT_TRUE(wm::IsWindowMinimized(window1.get()));
+
+  controller->AltKeyReleased();
+
+  EXPECT_TRUE(wm::IsWindowMinimized(window1.get()));
+}
+
+// Tests cycles between panel and normal windows.
+TEST_F(WindowCycleControllerTest, CyclePanels) {
+  WindowCycleController* controller =
+      Shell::GetInstance()->window_cycle_controller();
+
+  scoped_ptr<aura::Window> window0(CreateTestWindowInShellWithId(0));
+  scoped_ptr<aura::Window> panel0(CreatePanelWindow());
+  scoped_ptr<aura::Window> panel1(CreatePanelWindow());
+  wm::ActivateWindow(window0.get());
+  wm::ActivateWindow(panel1.get());
+  wm::ActivateWindow(panel0.get());
+  EXPECT_TRUE(wm::IsActiveWindow(panel0.get()));
+
+  controller->HandleCycleWindow(WindowCycleController::FORWARD);
+  controller->AltKeyReleased();
+  EXPECT_TRUE(wm::IsActiveWindow(panel1.get()));
+
+  // Cycling again should select the most recently used panel.
+  controller->HandleCycleWindow(WindowCycleController::FORWARD);
+  controller->AltKeyReleased();
+  EXPECT_TRUE(wm::IsActiveWindow(panel0.get()));
+
+  // Cycling twice again should select the first window.
+  controller->HandleCycleWindow(WindowCycleController::FORWARD);
+  controller->HandleCycleWindow(WindowCycleController::FORWARD);
+  controller->AltKeyReleased();
+  EXPECT_TRUE(wm::IsActiveWindow(window0.get()));
+}
+
+// Tests cycles between panel and normal windows.
+TEST_F(WindowCycleControllerTest, CyclePanelsDestroyed) {
+  WindowCycleController* controller =
+      Shell::GetInstance()->window_cycle_controller();
+
+  scoped_ptr<aura::Window> window0(CreateTestWindowInShellWithId(0));
+  scoped_ptr<aura::Window> window1(CreateTestWindowInShellWithId(1));
+  scoped_ptr<aura::Window> window2(CreateTestWindowInShellWithId(2));
+  scoped_ptr<aura::Window> panel0(CreatePanelWindow());
+  scoped_ptr<aura::Window> panel1(CreatePanelWindow());
+  wm::ActivateWindow(window2.get());
+  wm::ActivateWindow(panel1.get());
+  wm::ActivateWindow(panel0.get());
+  wm::ActivateWindow(window1.get());
+  wm::ActivateWindow(window0.get());
+  EXPECT_TRUE(wm::IsActiveWindow(window0.get()));
+
+  // Cycling once highlights window2.
+  controller->HandleCycleWindow(WindowCycleController::FORWARD);
+  // All panels are destroyed.
+  panel0.reset();
+  panel1.reset();
+  // Cycling again should now select window3.
+  controller->HandleCycleWindow(WindowCycleController::FORWARD);
+  controller->AltKeyReleased();
+  EXPECT_TRUE(wm::IsActiveWindow(window2.get()));
+}
+
+// Tests cycles between panel and normal windows.
+TEST_F(WindowCycleControllerTest, CycleMruPanelDestroyed) {
+  WindowCycleController* controller =
+      Shell::GetInstance()->window_cycle_controller();
+
+  scoped_ptr<aura::Window> window0(CreateTestWindowInShellWithId(0));
+  scoped_ptr<aura::Window> window1(CreateTestWindowInShellWithId(1));
+  scoped_ptr<aura::Window> panel0(CreatePanelWindow());
+  scoped_ptr<aura::Window> panel1(CreatePanelWindow());
+  wm::ActivateWindow(panel1.get());
+  wm::ActivateWindow(panel0.get());
+  wm::ActivateWindow(window1.get());
+  wm::ActivateWindow(window0.get());
+  EXPECT_TRUE(wm::IsActiveWindow(window0.get()));
+
+  // Cycling once highlights window2.
+  controller->HandleCycleWindow(WindowCycleController::FORWARD);
+
+  // Panel 1 is the next item as the MRU panel, removing it should make panel 2
+  // the next window to be selected.
+  panel0.reset();
+  // Cycling again should now select window3.
+  controller->HandleCycleWindow(WindowCycleController::FORWARD);
+  controller->AltKeyReleased();
+  EXPECT_TRUE(wm::IsActiveWindow(panel1.get()));
+}
+
+}  // namespace ash