New issue
Advanced search Search tips

Issue 846656 link

Starred by 1 user

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 3
Type: Bug



Sign in to add a comment

Layout Test external/wpt/css/selectors/focus-visible-002-manual.html is failing

Project Member Reported by olka@chromium.org, May 25 2018

Issue description

The following layout test is flaky on Win and Linux:

external/wpt/css/selectors/focus-visible-002-manual.html

Dashboard:
https://test-results.appspot.com/dashboards/flakiness_dashboard.html#testType=webkit_layout_tests&tests=focus-visible-002-manual.html
 

Comment 1 by olka@chromium.org, May 25 2018

Labels: -OS-Windows
Owner: olka@chromium.org
Status: Started (was: Untriaged)
Project Member

Comment 2 by bugdroid1@chromium.org, May 25 2018

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

commit 62ca3b416f9f385fe8d9130a3e49c162b753e8e1
Author: Olga Sharonova <olka@chromium.org>
Date: Fri May 25 12:04:08 2018

Adjusting expectation for external/wpt/css/selectors/focus-visible-002-manual.html

(flaky).

NOTRY=true
TBR=sky@chromium.org

Bug: 846656
Change-Id: I711631f169c28c525ed8ce97ae5799ca6a07adae
Reviewed-on: https://chromium-review.googlesource.com/1073410
Reviewed-by: Olga Sharonova <olka@chromium.org>
Commit-Queue: Olga Sharonova <olka@chromium.org>
Cr-Commit-Position: refs/heads/master@{#561836}
[modify] https://crrev.com/62ca3b416f9f385fe8d9130a3e49c162b753e8e1/third_party/WebKit/LayoutTests/TestExpectations

Comment 3 by olka@chromium.org, May 25 2018

Cc: aboxhall@chromium.org
Labels: -Sheriff-Chromium Test-Disabled
Owner: ----
Status: Available (was: Started)

Comment 4 by cnardi@chromium.org, May 25 2018

Cc: tkent@chromium.org wangxianzhu@chromium.org
 Issue 846603  has been merged into this issue.
Cc: -wangxianzhu@chromium.org
Owner: aboxhall@chromium.org
Status: Assigned (was: Available)
Assigning to the author of the test.
Labels: -Pri-1 Pri-2
Reducing priority since the test expectations has been updated.

Comment 7 by kenrb@chromium.org, Jun 22 2018

I am going to disable this test on all platforms, because I need to land a change that turns this test from flaky to reliably failing.

I think what is happening is that the select element menu is popping up underneath the mouse cursor and stealing some of the events getting sent by mouseClickInTarget. Currently this usually works okay because in the content layer we have been locking mouseUp events to the widget surface that received the mouseDown. That behavior is wrong and I am removing it in https://chromium-review.googlesource.com/c/chromium/src/+/1099183. This test needs to be modified to not rely on the incorrect behavior.
Owner: ----
Status: Untriaged (was: Assigned)
aboxhall@ is not on CSS nor Blink at this time (to my knowledge)
Labels: -Pri-2 Pri-3
Status: Available (was: Untriaged)
Summary: Layout Test external/wpt/css/selectors/focus-visible-002-manual.html is failing (was: Layout Test external/wpt/css/selectors/focus-visible-002-manual.html is flaky (times out))

Sign in to add a comment