New issue
Advanced search Search tips

Issue 695835 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner:
Closed: Feb 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Chrome
Pri: 1
Type: Bug



Sign in to add a comment

PrintPreviewDialogControllerBrowserTest.ReloadInitiatorTab flaky

Project Member Reported by treib@chromium.org, Feb 24 2017

Issue description

browser_tests failing on chromium.chromiumos/Linux ChromiumOS Tests (dbg)(1)

Builders failed on: 
- Linux ChromiumOS Tests (dbg)(1): 
  https://build.chromium.org/p/chromium.chromiumos/builders/Linux%20ChromiumOS%20Tests%20%28dbg%29%281%29


Particular test that is flaky: PrintPreviewDialogControllerBrowserTest.ReloadInitiatorTab

Earliest known failure:
https://uberchromegw.corp.google.com/i/chromium.chromiumos/builders/Linux%20ChromiumOS%20Tests%20%28dbg%29%281%29/builds/22813

Flakiness analysis:
https://findit-for-me.appspot.com/waterfall/list-flakes?test_name=PrintPreviewDialogControllerBrowserTest.ReloadInitiatorTab

Suspect: https://codereview.chromium.org/2701473007
 

Comment 1 by treib@chromium.org, Feb 24 2017

Revert in the CQ: https://codereview.chromium.org/2715733003/

Comment 2 by treib@chromium.org, Feb 24 2017

Cc: lazyboy@chromium.org treib@chromium.org rbpotter@chromium.org ma...@chromium.org hajimehoshi@chromium.org
 Issue 695760  has been merged into this issue.

Comment 3 by treib@chromium.org, Feb 24 2017

Labels: OS-Linux
Summary: PrintPreviewDialogControllerBrowserTest.ReloadInitiatorTab flaky on Linux and ChromeOS (was: browser_tests failing on chromium.chromiumos/Linux ChromiumOS Tests (dbg)(1))

Comment 4 by treib@chromium.org, Feb 24 2017

Revert has landed as #452800. Let's see if that helps...

Comment 6 by treib@chromium.org, Feb 24 2017

Summary: PrintPreviewDialogControllerBrowserTest.ReloadInitiatorTab flaky (was: PrintPreviewDialogControllerBrowserTest.ReloadInitiatorTab flaky on Linux and ChromeOS)

Comment 7 by treib@chromium.org, Feb 24 2017

 Issue 695764  has been merged into this issue.

Comment 8 by treib@chromium.org, Feb 24 2017

 Bug 695780  (flakiness in DeviceLocalAccountExternalPolicyLoaderTest.ForceInstallListSet) *might* be caused by the same CL.

Comment 9 by treib@chromium.org, Feb 24 2017

Status: Fixed (was: Started)
No more flakes; seems like the revert helped.

Comment 10 by treib@chromium.org, Feb 24 2017

 Issue 695780  has been merged into this issue.
Sorry for the breakage.
Strangely, I did not see this test failing in any of the tryjobs before landing the CL.
And I though (wrongly) that the other one (in  bug 695780 ) stoped failing after the last patchset.
I fixed these two and created https://codereview.chromium.org/2720513003/.

Sign in to add a comment