Commit 97c6e448 authored by Wei Li's avatar Wei Li Committed by Commit Bot

Re-enable EscapingMouseLockAndFullscreen on Win

This test showed to be flaky only on Linux bots. It should be safe to be
re-enabled on Windows.

Bug: 976883
Change-Id: I69d28a3b31ce31af2529a16750b5b65423b85e51
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2387180Reviewed-by: default avatarPeter Boström <pbos@chromium.org>
Commit-Queue: Wei Li <weili@chromium.org>
Cr-Commit-Position: refs/heads/master@{#803693}
parent 6b1a5028
......@@ -332,8 +332,14 @@ IN_PROC_BROWSER_TEST_F(FullscreenControllerInteractiveTest, EscapingMouseLock) {
// Disabled due to flakiness.
// TODO(crbug.com/976883): Fix and re-enable this.
// Tests mouse lock and fullscreen modes can be escaped with ESC key.
#if defined(OS_WIN)
#define MAYBE_EscapingMouseLockAndFullscreen EscapingMouseLockAndFullscreen
#else
#define MAYBE_EscapingMouseLockAndFullscreen \
DISABLED_EscapingMouseLockAndFullscreen
#endif
IN_PROC_BROWSER_TEST_F(FullscreenControllerInteractiveTest,
DISABLED_EscapingMouseLockAndFullscreen) {
MAYBE_EscapingMouseLockAndFullscreen) {
ASSERT_TRUE(embedded_test_server()->Start());
ui_test_utils::NavigateToURL(
browser(), embedded_test_server()->GetURL(kFullscreenMouseLockHTML));
......@@ -385,7 +391,6 @@ IN_PROC_BROWSER_TEST_F(FullscreenControllerInteractiveTest,
// Times out sometimes on Linux. http://crbug.com/135115
// Mac: http://crbug.com/103912
// Windows: Failing flakily on try jobs also.
// Tests mouse lock then fullscreen in same request.
#if defined(OS_WIN)
#define MAYBE_MouseLockAndFullscreen MouseLockAndFullscreen
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment