New issue
Advanced search Search tips

Issue 867628 link

Starred by 1 user

Issue metadata

Status: Duplicate
Merged: issue 874162
Owner:
Closed: Aug 21
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

middleClickAutoscroll-click-hyperlink.html / middleClickAutoscroll-nested-divs-forbidden.html flaky on WebKit Mac10.13 (retina)

Project Member Reported by imch...@chromium.org, Jul 25

Issue description

The following 2 tests are flaky, example: https://ci.chromium.org/buildbot/chromium.webkit/WebKit%20Mac10.13%20%28retina%29/101

virtual/user-activation-v2/fast/events/middleClickAutoscroll-click-hyperlink.html
virtual/user-activation-v2/fast/events/middleClickAutoscroll-nested-divs-forbidden.html
 
Cc: mustaq@chromium.org
Project Member

Comment 3 by bugdroid1@chromium.org, Jul 25

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

commit 6b166a1b60efb899ead3128eaf91df6620b5629c
Author: Derek Cheng <imcheng@chromium.org>
Date: Wed Jul 25 21:39:20 2018

[Sheriff] Update expectations for a few layout tests.

virtual/user-activation-v2/fast/events/middleClickAutoscroll-click-hyperlink.html
virtual/user-activation-v2/fast/events/middleClickAutoscroll-nested-divs-forbidden.html
virtual/new-remote-playback-pipeline/media/controls/text-track-menu-pointer-selection.html

Tbr: hajimehoshi
Bug:  866785 , 867628 
Change-Id: I938fe40c98ffbad9ef697556bcb456243b8b50da
Reviewed-on: https://chromium-review.googlesource.com/1150697
Reviewed-by: Derek Cheng <imcheng@chromium.org>
Commit-Queue: Derek Cheng <imcheng@chromium.org>
Cr-Commit-Position: refs/heads/master@{#578073}
[modify] https://crrev.com/6b166a1b60efb899ead3128eaf91df6620b5629c/third_party/WebKit/LayoutTests/TestExpectations

Owner: sunyunjia@chromium.org
Status: Assigned (was: Untriaged)
The test middleClickAutoscroll-click-hyperlink.html doesn't seem to have middle-click autoscroll.  Not sure what's being tested here!

sunyunjia@: It looks like an old test, please consider updating or even removing it.
Project Member

Comment 5 by bugdroid1@chromium.org, Aug 8

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

commit fc330293ae3900dbae89b2a10318a2aafa53dd0b
Author: Avi Drissman <avi@chromium.org>
Date: Wed Aug 08 16:30:30 2018

Disable failing tests.

BUG= 867628 , 746128
TBR=hubbe@chromium.org

Change-Id: I71f6dba4710bf7c3ffb0f14ed430d6419f59e0ed
Reviewed-on: https://chromium-review.googlesource.com/1167468
Commit-Queue: Avi Drissman <avi@chromium.org>
Reviewed-by: Avi Drissman <avi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#581588}
[modify] https://crrev.com/fc330293ae3900dbae89b2a10318a2aafa53dd0b/third_party/WebKit/LayoutTests/TestExpectations

Mergedinto: 874162
Status: Duplicate (was: Assigned)

Sign in to add a comment