New issue
Advanced search Search tips

Issue 599730 link

Starred by 1 user

Issue metadata

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



Sign in to add a comment

SitePerProcessBrowserTest.NestedPopupMenuTest and SequentialFocusNavigation frequently time out

Project Member Reported by benwells@chromium.org, Apr 1 2016

Issue description

Project Member

Comment 1 by bugdroid1@chromium.org, Apr 1 2016

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

commit 103e61ce16a81cfee6154c41ee79829c34bf7a03
Author: benwells <benwells@chromium.org>
Date: Fri Apr 01 01:38:53 2016

Disable two flakily timing out SitePerProcessBrowserTests

TBR=jyasskin@chromium.org
BUG=599730

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

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

[modify] https://crrev.com/103e61ce16a81cfee6154c41ee79829c34bf7a03/content/browser/site_per_process_browsertest.cc

Comment 2 by creis@chromium.org, Apr 1 2016

Owner: alex...@chromium.org
Alex is looking into this (as continuation from  issue 599344 ).
Project Member

Comment 3 by bugdroid1@chromium.org, Apr 13 2016

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

commit 94f63b7fb807642fd3373d0ae43b14f2015c8694
Author: alexmos <alexmos@chromium.org>
Date: Wed Apr 13 19:17:25 2016

OOPIF: Move SequentialFocusNavigation test to interactive_ui_tests.

The test was recently disabled due to becoming flaky.  This CL
converts it to an interactive test, so that focus is not tinkered with
during the test.  The prerequisite work for this was in issues  559273 
and 596508.

BUG=599730

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

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

[modify] https://crrev.com/94f63b7fb807642fd3373d0ae43b14f2015c8694/chrome/browser/site_per_process_interactive_browsertest.cc
[modify] https://crrev.com/94f63b7fb807642fd3373d0ae43b14f2015c8694/content/browser/site_per_process_browsertest.cc

Cc: -kenrb@chromium.org
Owner: kenrb@chromium.org
Status: Assigned (was: Untriaged)
I don't see any more flakes for SequentialFocusNavigation since I moved it to interactive_ui_tests in r387062, so I'm reassigning this to Ken now to fix the NestedPopupMenuTest test.

Sign in to add a comment