Commit 8dd6dae3 authored by Daniel Libby's avatar Daniel Libby Committed by Chromium LUCI CQ

Re-enable TouchSelectionController scale factor tests on Windows

The alleged bug no longer hits, so these tests can be re-enabled and the
bug closed (last reference to this in the source tree).

Bug: 545547
Change-Id: I74469684932ba2bb61a8ac91a522024ac24dc328
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2570540
Auto-Submit: Daniel Libby <dlibby@microsoft.com>
Commit-Queue: Dave Tapuska <dtapuska@chromium.org>
Reviewed-by: default avatarDave Tapuska <dtapuska@chromium.org>
Cr-Commit-Position: refs/heads/master@{#833301}
parent f9cd66f5
...@@ -882,18 +882,9 @@ class TouchSelectionControllerClientAuraScaleFactorTest ...@@ -882,18 +882,9 @@ class TouchSelectionControllerClientAuraScaleFactorTest
} }
}; };
#if defined(OS_WIN)
// High DPI tests are disabled on Windows due to crbug.com/545547.
#define MAYBE_SelectionHandleCoordinates DISABLED_SelectionHandleCoordinates
#define MAYBE_InsertionHandleCoordinates DISABLED_InsertionHandleCoordinates
#else
#define MAYBE_SelectionHandleCoordinates SelectionHandleCoordinates
#define MAYBE_InsertionHandleCoordinates InsertionHandleCoordinates
#endif
// Tests that selection handles are properly positioned at 2x DSF. // Tests that selection handles are properly positioned at 2x DSF.
IN_PROC_BROWSER_TEST_F(TouchSelectionControllerClientAuraScaleFactorTest, IN_PROC_BROWSER_TEST_F(TouchSelectionControllerClientAuraScaleFactorTest,
MAYBE_SelectionHandleCoordinates) { SelectionHandleCoordinates) {
// Set the test page up. // Set the test page up.
ASSERT_NO_FATAL_FAILURE(StartTestWithPage("/touch_selection.html")); ASSERT_NO_FATAL_FAILURE(StartTestWithPage("/touch_selection.html"));
InitSelectionController(); InitSelectionController();
...@@ -989,7 +980,7 @@ IN_PROC_BROWSER_TEST_F(TouchSelectionControllerClientAuraScaleFactorTest, ...@@ -989,7 +980,7 @@ IN_PROC_BROWSER_TEST_F(TouchSelectionControllerClientAuraScaleFactorTest,
// Tests that insertion handles are properly positioned at 2x DSF. // Tests that insertion handles are properly positioned at 2x DSF.
IN_PROC_BROWSER_TEST_F(TouchSelectionControllerClientAuraScaleFactorTest, IN_PROC_BROWSER_TEST_F(TouchSelectionControllerClientAuraScaleFactorTest,
MAYBE_InsertionHandleCoordinates) { InsertionHandleCoordinates) {
// Set the test page up. // Set the test page up.
ASSERT_NO_FATAL_FAILURE(StartTestWithPage("/touch_selection.html")); ASSERT_NO_FATAL_FAILURE(StartTestWithPage("/touch_selection.html"));
InitSelectionController(); InitSelectionController();
......
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