Commit 05b2a8ed authored by Wez's avatar Wez Committed by Commit Bot

Disable some more flaky FlingControllerTest tests.

Disable the ControllerHandlesTouchscreenGestureFling and
ControllerSendsWheelEndWhenTouchpadFlingIsOver tests, which appear to
have timing-dependent flakiness.

Bug: 836996
Change-Id: I42b7b23489b4cf2dbb2fdacf89c77bd102ac03a8
Reviewed-on: https://chromium-review.googlesource.com/1166328Reviewed-by: default avatarSahel Sharify <sahel@chromium.org>
Reviewed-by: default avatarDave Tapuska <dtapuska@chromium.org>
Commit-Queue: Wez <wez@chromium.org>
Cr-Commit-Position: refs/heads/master@{#581671}
parent 40bc1181
......@@ -195,7 +195,8 @@ TEST_F(FlingControllerTest, DISABLED_ControllerHandlesTouchpadGestureFling) {
EXPECT_EQ(0.f, last_sent_wheel_.delta_y);
}
TEST_F(FlingControllerTest, ControllerHandlesTouchscreenGestureFling) {
// TODO(https://crbug.com/836996): Timing-dependent flakes on some platforms.
TEST_F(FlingControllerTest, DISABLED_ControllerHandlesTouchscreenGestureFling) {
base::TimeTicks progress_time = base::TimeTicks::Now();
SimulateFlingStart(blink::kWebGestureDeviceTouchscreen,
gfx::Vector2dF(1000, 0));
......@@ -221,7 +222,9 @@ TEST_F(FlingControllerTest, ControllerHandlesTouchscreenGestureFling) {
EXPECT_EQ(WebInputEvent::kGestureScrollEnd, last_sent_gesture_.GetType());
}
TEST_F(FlingControllerTest, ControllerSendsWheelEndWhenTouchpadFlingIsOver) {
// TODO(https://crbug.com/836996): Timing-dependent flakes on some platforms.
TEST_F(FlingControllerTest,
DISABLED_ControllerSendsWheelEndWhenTouchpadFlingIsOver) {
base::TimeTicks progress_time = base::TimeTicks::Now();
SimulateFlingStart(blink::kWebGestureDeviceTouchpad, gfx::Vector2dF(100, 0));
EXPECT_TRUE(FlingInProgress());
......
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