New issue
Advanced search Search tips

Issue 860731 link

Starred by 2 users

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 2
Type: Bug



Sign in to add a comment

fast/scroll-snap/animate-fling-to-snap-points.html is flaky

Project Member Reported by pdr@chromium.org, Jul 6

Issue description

This test is flaky:
fast/scroll-snap/animate-fling-to-snap-points.html
https://test-results.appspot.com/dashboards/flakiness_dashboard.html#testType=webkit_layout_tests&tests=fast%2Fscroll-snap%2Fanimate-fling-to-snap-points.html%20

When it fails, the failure text is:
---------------------8<---------------------
This is a testharness.js-based test.
PASS Without fling enabled, the scroll ends at the closest snap point to the scroll destination.
FAIL With fling enabled, the scroll ends at the closest snap point to the fling destination. promise_test: Unhandled rejection with value: "Reaches the maximum frames."
Harness: the test ran to completion.
---------------------8<---------------------
 
Probably best to update the text expectations first to indicate the flakiness while looking for the root cause.
Sure thing, I will mark this test as flaky (Failure Pass).
Project Member

Comment 3 by bugdroid1@chromium.org, Jul 6

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/63d152d1231834aebb51e65ed3b2bcc706a6501a

commit 63d152d1231834aebb51e65ed3b2bcc706a6501a
Author: Philip Rogers <pdr@chromium.org>
Date: Fri Jul 06 18:17:49 2018

Mark animate-fling-to-snap-points.html as flaky

This test is flaky on all platforms:
https://test-results.appspot.com/dashboards/flakiness_dashboard.html#testType=webkit_layout_tests&tests=fast%2Fscroll-snap%2Fanimate-fling-to-snap-points.html%20

This patch marks it as flaky while the root cause is investigated.

TBR=sunyunjia@chromium.org

No-Try: true
Bug: 860731
Change-Id: I182631656d5d1be0c38f1f04ee3780d08e2a6ebc
Reviewed-on: https://chromium-review.googlesource.com/1128049
Commit-Queue: Philip Rogers <pdr@chromium.org>
Reviewed-by: Philip Rogers <pdr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#573015}
[modify] https://crrev.com/63d152d1231834aebb51e65ed3b2bcc706a6501a/third_party/WebKit/LayoutTests/TestExpectations

Triage checkin - this test is still (very) flaky - on certain botst, most runs fail.

Comment 5 by sunyunjia@chromium.org, Jan 17 (5 days ago)

Labels: -Pri-1 Pri-2

Sign in to add a comment