New issue
Advanced search Search tips

Issue 592183 link

Starred by 2 users

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Bug



Sign in to add a comment

Layout Test usb/usbDevice.html is slow

Project Member Reported by dpranke@chromium.org, Mar 5 2016

Issue description

The following layout test is flaky on all platforms:

usb/usbDevice.html

http://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=usb%2FusbDevice.html

I have no idea when this started. It appears to always time out during the main run and then pass on a retry.

scheib@, can you figure out who should look at this?
 
Cc: -scheib@chromium.org
Owner: reillyg@chromium.org
Status: Assigned (was: Untriaged)
Project Member

Comment 3 by bugdroid1@chromium.org, Mar 8 2016

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

commit ee57f6c40d33921df8364e4710a73620b57ccdb5
Author: reillyg <reillyg@chromium.org>
Date: Tue Mar 08 02:53:39 2016

Mark usb/usbDevice.html as a slow LayoutTest.

I don't believe this test actually fails on bots. It simply does a
bunch of work and needs some extra time to run. Mark it as slow to see
if that makes the timeouts go away. If not then there is a real bug in
the test.

BUG=592183

Review URL: https://codereview.chromium.org/1775643002

Cr-Commit-Position: refs/heads/master@{#379707}

[modify] https://crrev.com/ee57f6c40d33921df8364e4710a73620b57ccdb5/third_party/WebKit/LayoutTests/SlowTests
[modify] https://crrev.com/ee57f6c40d33921df8364e4710a73620b57ccdb5/third_party/WebKit/LayoutTests/TestExpectations

Cc: roc...@chromium.org
Summary: Layout Test usb/usbDevice.html is slow (was: Layout Test usb/usbDevice.html is flaky)
I've determined that this test is simply slow and rockot@ is investigating whether excessive thread hopping in the Mojo layer is partially to blame.

Comment 6 by sshru...@google.com, May 18 2016

Labels: Test-Layout

Comment 7 by sshru...@google.com, May 18 2016

Components: -Blink>LayoutTests Blink
Deprecating component:Blink>LayoutTests, to use label Test=Layout instead. Merging these to component:Blink for the Blink rotation to pick up and re-triage as appropriate. 
Components: -Blink Blink>USB

Comment 9 by scheib@chromium.org, Apr 20 2017

Labels: Pri-3
Issues not modified in last 50 days aren't on track to ship in next release.
Cc: -roc...@chromium.org rockot@google.com

Sign in to add a comment