New issue
Advanced search Search tips

Issue 854143 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Jun 2018
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows
Pri: 1
Type: Bug
Flaky-Test: TargetDomainCreateAndDeleteBrowserContextTest.RunAsyncTest



Sign in to add a comment

TargetDomainCreateAndDeleteBrowserContextTest.RunAsyncTest is Flaky

Project Member Reported by Findit, Jun 19 2018

Issue description

Findit has detected a flake at test TargetDomainCreateAndDeleteBrowserContextTest.RunAsyncTest.

Culprit (70.0% confidence): https://chromium-review.googlesource.com/q/I6e391fadbbf47aa6427a16495bdca3c88d456b05
Regression range: None

Analysis: https://findit-for-me.appspot.com/waterfall/flake?key=ag9zfmZpbmRpdC1mb3ItbWVyxAELEhdNYXN0ZXJGbGFrZUFuYWx5c2lzUm9vdCKNAWNocm9taXVtLndpbi9XaW43IFRlc3RzIChkYmcpKDEpLzY5ODIxL2hlYWRsZXNzX2Jyb3dzZXJ0ZXN0cy9WR0Z5WjJWMFJHOXRZV2x1UTNKbFlYUmxRVzVrUkdWc1pYUmxRbkp2ZDNObGNrTnZiblJsZUhSVVpYTjBMbEoxYmtGemVXNWpWR1Z6ZEE9PQwLEhNNYXN0ZXJGbGFrZUFuYWx5c2lzGAEM

If this result was incorrect, apply the label Test-Findit-Wrong, mark the bug as Untriaged and the component Tools>Test>Findit>Flakiness.
 
Project Member

Comment 1 by Findit, Jun 19 2018

Findit identified the culprit r568213 with confidence 70.0% in the config "chromium.win / Win7 Tests (dbg)(1)"
based on the flakiness trend:

https://findit-for-me.appspot.com/waterfall/flake?key=ag9zfmZpbmRpdC1mb3ItbWVyxAELEhdNYXN0ZXJGbGFrZUFuYWx5c2lzUm9vdCKNAWNocm9taXVtLndpbi9XaW43IFRlc3RzIChkYmcpKDEpLzY5ODIxL2hlYWRsZXNzX2Jyb3dzZXJ0ZXN0cy9WR0Z5WjJWMFJHOXRZV2x1UTNKbFlYUmxRVzVrUkdWc1pYUmxRbkp2ZDNObGNrTnZiblJsZUhSVVpYTjBMbEoxYmtGemVXNWpWR1Z6ZEE9PQwLEhNNYXN0ZXJGbGFrZUFuYWx5c2lzGAEM


Automatically posted by the findit-for-me app (https://goo.gl/Ot9f7N).
Feedback is welcome! Please use component Tools>Test>FindIt>Flakiness

Comment 2 by treib@chromium.org, Jun 19 2018

Cc: pfeldman@chromium.org
 Issue 854142  has been merged into this issue.

Comment 3 by treib@chromium.org, Jun 19 2018

Cc: -pfeldman@chromium.org
Labels: OS-Linux OS-Windows
Owner: pfeldman@chromium.org
Status: Assigned (was: Available)
Since FindIt agrees that this is likely the culprit, I'll go ahead and revert.
Project Member

Comment 4 by bugdroid1@chromium.org, Jun 19 2018

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

commit 7e6092a29f787a7243d3dff4d6626dc3f62404d7
Author: Marc Treib <treib@chromium.org>
Date: Tue Jun 19 13:00:31 2018

Revert "Headless: don't run unload hooks upon web contents close."

This reverts commit 0d53513f26f09e7f8654a30452e2676addb01a30.

Reason for revert: Flaky tests, see bug

Bug:  854143 

Original change's description:
> Headless: don't run unload hooks upon web contents close.
> 
> Change-Id: I6e391fadbbf47aa6427a16495bdca3c88d456b05
> Reviewed-on: https://chromium-review.googlesource.com/1105041
> Reviewed-by: Dmitry Gozman <dgozman@chromium.org>
> Commit-Queue: Pavel Feldman <pfeldman@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#568213}

TBR=dgozman@chromium.org,pfeldman@chromium.org,eseckler@chromium.org

Change-Id: I252909f78e6d33b98819f6510f780997327a6901
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://chromium-review.googlesource.com/1106077
Reviewed-by: Marc Treib <treib@chromium.org>
Commit-Queue: Marc Treib <treib@chromium.org>
Cr-Commit-Position: refs/heads/master@{#568417}
[modify] https://crrev.com/7e6092a29f787a7243d3dff4d6626dc3f62404d7/headless/lib/browser/headless_web_contents_impl.cc
[modify] https://crrev.com/7e6092a29f787a7243d3dff4d6626dc3f62404d7/headless/lib/browser/headless_web_contents_impl.h

Comment 5 by treib@chromium.org, Jun 19 2018

Labels: -Sheriff-Chromium

Comment 6 by treib@chromium.org, Jun 19 2018

 Issue 854188  has been merged into this issue.
Status: Fixed (was: Assigned)

Sign in to add a comment