New issue
Advanced search Search tips

Issue 811708 link

Starred by 1 user
Status: Fixed
Owner: ----
Closed: Feb 2018
Cc:
rbyers@chromium.org
dtapu...@chromium.org
eirage@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 1
Type: Bug



Sign in to add a comment

Layout Test fast/events/hit-test-counts.html is flaky on Win dbg

Project Member Reported by treib@chromium.org, Feb 13 2018 
The following layout test is flaky on Win dbg

fast/events/hit-test-counts.html

Probable cause:

Unknown. Seems to have started on 2018-02-12, see flakiness dashboard:
https://test-results.appspot.com/dashboards/flakiness_dashboard.html#testType=webkit_layout_tests&tests=fast%2Fevents%2Fhit-test-counts

CCing the last person who touched this: eirage, can you either take a look or redirect as appropriate?

Comment 1 by treib@chromium.org, Feb 13 2018 

Logs look like this:

01:20:59.758 4332 worker/2 fast/events/hit-test-counts.html output stderr lines:
01:20:59.758 4332   [4016:6160:0213/012057.309:INFO:media_foundation_video_encode_accelerator_win.cc(370)] Windows versions earlier than 8 are not supported.
01:20:59.759 5436 [4645/5685] fast/events/hit-test-counts.html failed unexpectedly (text diff)
01:20:59.759 4332 worker/2 fast/events/hit-test-counts.html failed:
01:20:59.759 4332 worker/2  text diff

Which doesn't really tell me much.

Comment 2 by treib@chromium.org, Feb 13 2018 

Pending CL to mark as flaky: https://crrev.com/c/915346
Project Member

Comment 3 by bugdroid1@chromium.org, Feb 13 2018 

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

commit a2ba02870355fcc3dc2d9a2f0dcf8ee830337a9c
Author: Marc Treib <treib@chromium.org>
Date: Tue Feb 13 12:03:45 2018

Mark fast/events/hit-test-counts.html as flaky on Win Debug

TBR=eirage

Bug:  811708 
Change-Id: I6a9fa22fd0deab2077490378c2bca8f06dd4c046
Reviewed-on: https://chromium-review.googlesource.com/915346
Reviewed-by: Marc Treib <treib@chromium.org>
Commit-Queue: Marc Treib <treib@chromium.org>
Cr-Commit-Position: refs/heads/master@{#536339}
[modify] https://crrev.com/a2ba02870355fcc3dc2d9a2f0dcf8ee830337a9c/third_party/WebKit/LayoutTests/TestExpectations

Comment 4 by treib@chromium.org, Feb 13 2018

Cc: dtapu...@chromium.org rbyers@chromium.org
Labels: -Pri-2 -Sheriff-Chromium Test-Disabled Pri-1
Project Member

Comment 5 by bugdroid1@chromium.org, Feb 13 2018 

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

commit ad9f14e2f05a55da16249935532d2326df155c26
Author: Ella Ge <eirage@chromium.org>
Date: Tue Feb 13 17:00:00 2018

hit-test-counts test clear count before run test

A MouseMove will be sent when initialize a layout test. The hit-test
from the initial mousemove causes the first "Initial" counts record
incorrectly.
In this CL, clear counts before run each tests

Bug:  811708 
Change-Id: Ie032d68627d14a668247c01b02489ba107f31f0e
Reviewed-on: https://chromium-review.googlesource.com/915985
Reviewed-by: Dave Tapuska <dtapuska@chromium.org>
Commit-Queue: Ella Ge <eirage@chromium.org>
Cr-Commit-Position: refs/heads/master@{#536390}
[modify] https://crrev.com/ad9f14e2f05a55da16249935532d2326df155c26/third_party/WebKit/LayoutTests/TestExpectations
[modify] https://crrev.com/ad9f14e2f05a55da16249935532d2326df155c26/third_party/WebKit/LayoutTests/fast/events/hit-test-counts.html

Comment 6 by eirage@chromium.org, Feb 13 2018

Status: Fixed (was: Untriaged)
►

Sign in to add a comment