New issue
Advanced search Search tips

Issue 807471 link

Starred by 1 user

Issue metadata

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



Sign in to add a comment

HostedAppProcessModelTest.FromOutsideHostedApp/1 flaky on Win 7

Project Member Reported by bsep@chromium.org, Jan 30 2018

Issue description

Comment 1 by bsep@chromium.org, Jan 31 2018

Cc: rdevlin....@chromium.org
Labels: Test-Disabled
Owner: alex...@chromium.org
Status: Assigned (was: Untriaged)
Disabling as https://chromium-review.googlesource.com/c/chromium/src/+/894736

It's been flaky for a while. Assigning to the test author to look at it.
Project Member

Comment 2 by bugdroid1@chromium.org, Jan 31 2018

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

commit d1d6d70eef59566f0e7b09d535a27ff8489e605d
Author: Bret Sepulveda <bsep@chromium.org>
Date: Wed Jan 31 01:05:15 2018

Disable HostedAppProcessModelTest.FromOutsideHostedApp/1 on Win 7.

The test is timeout-flaky.

TBR=rdevlin.cronin@chromium.org

Bug: 807471
Change-Id: Ie70d0217c21ce9ed3ae1c1a79b6a030dd70886f5
Reviewed-on: https://chromium-review.googlesource.com/894736
Reviewed-by: Bret Sepulveda <bsep@chromium.org>
Commit-Queue: Bret Sepulveda <bsep@chromium.org>
Cr-Commit-Position: refs/heads/master@{#533132}
[modify] https://crrev.com/d1d6d70eef59566f0e7b09d535a27ff8489e605d/chrome/browser/ui/extensions/hosted_app_browsertest.cc

Looks like the test is just too long for the slower debug bots.  Looking at https://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=HostedAppProcessModelTest.FromOutsideHostedApp the successful runs on Win7 Tests (dbg)(1) were all just slightly below the 45 second cutoff.  

I'll see if I can break this test up into smaller tests, after I take care of a few higher-priority items.  We'll maintain coverage on non-Windows platforms in the meantime, and we could probably even re-enable it for Win release builds.

Sign in to add a comment