New issue
Advanced search Search tips

Issue 594189 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Mar 2016
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

virtual/spv2/fast/overflow/lots-of-sibling-inline-boxes.html is flaky

Project Member Reported by alex...@chromium.org, Mar 11 2016

Issue description

This test has been timing out pretty consistently on a bunch of WebKit debug bots:
https://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=fast%2Foverflow%2Flots-of-sibling-inline-boxes.html

This is a slow test, but note that the fast/overflow/lots-of-sibling-inline-boxes.html has the SLOW expectation, but virtual/spv2/fast/overflow/lots-of-sibling-inline-boxes.html does not, so maybe that just needs to be fixed. 
 
Yeah - fast/overflow/lots-of-sibling-inline-boxes.html takes 4.5 seconds on a Release build:

https://build.chromium.org/p/tryserver.blink/builders/linux_blink_rel/builds/83647/steps/webkit_tests%20%28with%20patch%29/logs/stdio
I also filed issue 594216 to see if virtual tests should be inheriting SLOW expectations from their non-virtual counterparts.
Project Member

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

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

commit 658061b938c58c9ea432bbf293b3a9d34a1c5457
Author: alexmos <alexmos@chromium.org>
Date: Fri Mar 11 19:28:57 2016

Add SLOW expectation for virtual/spv2/fast/overflow/lots-of-sibling-inline-boxes.html

BUG= 594189 
NOTRY=true

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

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

[modify] https://crrev.com/658061b938c58c9ea432bbf293b3a9d34a1c5457/third_party/WebKit/LayoutTests/SlowTests

Cc: -alex...@chromium.org
Owner: alex...@chromium.org
Status: Fixed (was: Untriaged)
Looking at the flakiness dashboard, adding SLOW seems to have solved this, so marking as fixed.

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

Labels: Test-Layout

Sign in to add a comment