New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 622696 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner: ----
Closed: Jun 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 2
Type: Bug



Sign in to add a comment

inspector/sources/debugger-async/async-await/async-callstack-async-await.html is flaky on WebKit Win7 (dbg)

Project Member Reported by hbos@chromium.org, Jun 23 2016

Issue description

inspector/sources/debugger-async/async-await/async-callstack-async-await.html is flaky on WebKit Win7 (dbg).

Normally, I would disable the test in TestExpectations to avoid the flake. But it's already added as [ Failure Timeout ]: https://cs.chromium.org/chromium/src/third_party/WebKit/LayoutTests/TestExpectations?q=inspector/sources/debugger-async/async-await/async-callstack-async-await.html&sq=package:chromium&l=701&dr=C
... and still we get Timeout failures, I don't get it.

Related issue: https://bugs.chromium.org/p/chromium/issues/detail?id=621515

Recent builds:
https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Win7%20%28dbg%29?numbuilds=200

Sample flake:
https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Win7%20%28dbg%29/builds/6158

 
The virtual suite isnt under TestExpectations, only the non-virtual suite.

Maybe splitting the test apart would avoid the timeout? I can probably do that tomorrow.

Comment 2 by hbos@chromium.org, Jun 23 2016

Cool, thanks!
Project Member

Comment 3 by bugdroid1@chromium.org, Jun 24 2016

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

commit 459aa22b3344f3f8097e05cf57cc2631ec0ab4b1
Author: caitpotter88 <caitpotter88@gmail.com>
Date: Fri Jun 24 23:46:47 2016

[LayoutTests] Split up inspector/sources/debugger-async/async-await/async-callstack-async-await.html

Split up test in order to minimize Timeout flakes

R=hbos@chromium.org, dgozman@chromium.org, littledan@chromium.org
BUG= 622696 ,  621515 

Review-Url: https://codereview.chromium.org/2099723002
Cr-Commit-Position: refs/heads/master@{#402015}

[modify] https://crrev.com/459aa22b3344f3f8097e05cf57cc2631ec0ab4b1/third_party/WebKit/LayoutTests/TestExpectations
[delete] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/third_party/WebKit/LayoutTests/inspector/sources/debugger-async/async-await/async-callstack-async-await-expected.txt
[add] https://crrev.com/459aa22b3344f3f8097e05cf57cc2631ec0ab4b1/third_party/WebKit/LayoutTests/inspector/sources/debugger-async/async-await/async-callstack-async-await1-expected.txt
[rename] https://crrev.com/459aa22b3344f3f8097e05cf57cc2631ec0ab4b1/third_party/WebKit/LayoutTests/inspector/sources/debugger-async/async-await/async-callstack-async-await1.html
[add] https://crrev.com/459aa22b3344f3f8097e05cf57cc2631ec0ab4b1/third_party/WebKit/LayoutTests/inspector/sources/debugger-async/async-await/async-callstack-async-await2-expected.txt
[copy] https://crrev.com/459aa22b3344f3f8097e05cf57cc2631ec0ab4b1/third_party/WebKit/LayoutTests/inspector/sources/debugger-async/async-await/async-callstack-async-await2.html

Comment 4 by hbos@chromium.org, Jun 27 2016

Status: Fixed (was: Untriaged)
Thanks!

Sign in to add a comment