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

Issue 922098 link

Starred by 2 users

Issue metadata

Status: Untriaged
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Android , Windows , iOS , Chrome , Mac , Fuchsia
Pri: 3
Type: Bug



Sign in to add a comment

Reduce default Run() timeout in tests, and apply it more widely.

Project Member Reported by w...@chromium.org, Jan 15

Issue description

 Issue 918724  introduced a default timeout on RunLoop::Run() calls made on the main thread of any test that uses ScopedTaskEnvironment.  This could be improved by:

1. Reducing the default Run() timeout from action_max_timeout() to e.g. action_timeout().

2. Enabling Run() timeouts in base::TestSuite, so that tests that don't use ScopedTaskEnvironment still get them.

3. Enabling some form of timeout on Run[UntilIdle]() even when running under MOCK_TIME, to trap tests which run with mocked time that hang busy-looping over tasks.
 
Labels: Hotlist-GoodFirstBug

Sign in to add a comment