Don't handle the tab press event in the window cycle event filter.

ash/wm/window_cycle_controller_unittest.cc

 // Copyright 2014 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/common/wm/window_cycle_controller.h"
 
 #include <algorithm>
 #include <memory>
 
 #include "ash/aura/wm_window_aura.h"
 #include "ash/common/focus_cycler.h"
 #include "ash/common/scoped_root_window_for_new_windows.h"
 #include "ash/common/session/session_state_delegate.h"
 #include "ash/common/shell_window_ids.h"
 #include "ash/common/wm/window_cycle_list.h"
 #include "ash/common/wm/window_state.h"
+#include "ash/common/wm/wm_event.h"
 #include "ash/common/wm_shell.h"
 #include "ash/shelf/shelf.h"
 #include "ash/shelf/shelf_widget.h"
 #include "ash/shell.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_state_aura.h"
@@ skipping 581 matching lines @@
   EXPECT_EQ(1u, key_count.GetCountAndReset());
   generator.PressKey(ui::VKEY_TAB, ui::EF_ALT_DOWN);
   EXPECT_EQ(0u, key_count.GetCountAndReset());
   generator.ReleaseKey(ui::VKEY_TAB, ui::EF_ALT_DOWN);
   EXPECT_EQ(0u, key_count.GetCountAndReset());
   generator.ReleaseKey(ui::VKEY_MENU, ui::EF_NONE);
   EXPECT_TRUE(wm::GetWindowState(w1.get())->IsActive());
   EXPECT_EQ(0u, key_count.GetCountAndReset());
 }
 
+// Tests that we can cycle past fullscreen windows: https://crbug.com/622396.
+// Fullscreen windows are special in that they are allowed to handle alt+tab
+// keypresses, which means the window cycle event filter should not handle
+// the tab press else it prevents cycling past that window.
+TEST_F(WindowCycleControllerTest, TabPastFullscreenWindow) {
+  std::unique_ptr<Window> w0(CreateTestWindowInShellWithId(0));
+  std::unique_ptr<Window> w1(CreateTestWindowInShellWithId(1));
+  wm::WMEvent maximize_event(wm::WM_EVENT_FULLSCREEN);
+  wm::GetWindowState(w1.get())->Activate();
+  wm::GetWindowState(w1.get())->OnWMEvent(&maximize_event);
+  EXPECT_TRUE(wm::GetWindowState(w1.get())->IsFullscreen());
+  wm::GetWindowState(w0.get())->Activate();
+
+  ui::test::EventGenerator& generator = GetEventGenerator();

[varkha, 2016/08/17 18:06:57]

nit: Could swap the line before and above this comment.

[flackr, 2016/08/17 19:07:19]

Done.

+  EXPECT_TRUE(wm::GetWindowState(w0.get())->IsActive());
+  generator.PressKey(ui::VKEY_MENU, ui::EF_NONE);
+
+  generator.PressKey(ui::VKEY_TAB, ui::EF_ALT_DOWN);
+  generator.ReleaseKey(ui::VKEY_TAB, ui::EF_ALT_DOWN);
+  EXPECT_TRUE(wm::GetWindowState(w1.get())->IsActive());
+
+  // Because w1 is full-screen, the event should be passed to the browser
+  // window to handle it (which if the browser doesn't handle it will pass on
+  // the alt+tab to continue cycling).
+  KeyEventCounter key_count;
+  w1->AddPreTargetHandler(&key_count);
+  generator.PressKey(ui::VKEY_TAB, ui::EF_ALT_DOWN);
+  EXPECT_EQ(1u, key_count.GetCountAndReset());
+}
+
 }  // namespace ash