New issue
Advanced search Search tips

Issue 868921 link

Starred by 1 user

Issue metadata

Status: Duplicate
Merged: issue 662160
Owner: ----
Closed: Jul 30
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug



Sign in to add a comment

"WebNavigationApiTest.UserAction" is flaky

Project Member Reported by chromium...@appspot.gserviceaccount.com, Jul 30

Issue description

"WebNavigationApiTest.UserAction" is flaky.

This issue was created automatically by the chromium-try-flakes app. Please find the right owner to fix the respective test/step and assign this issue to them. If the step/test is infrastructure-related, please add Infra-Troopers label and change issue status to Untriaged. When done, please remove the issue from Sheriff Bug Queue by removing the Sheriff-Chromium label.

We have detected 3 recent flakes. List of all flakes can be found at https://chromium-try-flakes.appspot.com/all_flake_occurrences?key=ahVzfmNocm9taXVtLXRyeS1mbGFrZXNyKgsSBUZsYWtlIh9XZWJOYXZpZ2F0aW9uQXBpVGVzdC5Vc2VyQWN0aW9uDA.

Flaky tests should be disabled within 30 minutes unless culprit CL is found and reverted. Please see more details here: https://sites.google.com/a/chromium.org/dev/developers/tree-sheriffs/sheriffing-bug-queues#triaging-auto-filed-flakiness-bugs
 
Mergedinto: 662160
Status: Duplicate (was: Untriaged)
Project Member

Comment 2 by bugdroid1@chromium.org, Jul 30

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

commit bbabfa4fc9d13342562a1f252376fcbba0cfa880
Author: Peter Boström <pbos@chromium.org>
Date: Mon Jul 30 17:37:27 2018

Revert "Enable WebNavigationApiTest.UserAction test."

This reverts commit 98cfabe9418bb39ec82ce941432303e302a10463.

Reason for revert: Still reporting as flaky.

Bug:  chromium:868921 , chromium:662160

Original change's description:
> Enable WebNavigationApiTest.UserAction test.
> 
> There have been various fixes in navigation code that have helped deflake
> webNavigationApi tests. The goal of this CL is to enable the
> WebNavigationApiTest.UserAction test and see if it has benefited from
> these fixes and is no longer flaky.
> 
> Bug: 662160
> Change-Id: Ia463d32f425b9e366c5879d4f297a521c8c28260
> Reviewed-on: https://chromium-review.googlesource.com/1153394
> Reviewed-by: Alex Moshchuk <alexmos@chromium.org>
> Commit-Queue: Nasko Oskov <nasko@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#578835}

TBR=nasko@chromium.org,alexmos@chromium.org

# Not skipping CQ checks because original CL landed > 1 day ago.

Bug: 662160
Change-Id: I57be2c1d1224720820efeb0204950b15f75d6731
Reviewed-on: https://chromium-review.googlesource.com/1155072
Reviewed-by: Peter Boström <pbos@chromium.org>
Commit-Queue: Peter Boström <pbos@chromium.org>
Cr-Commit-Position: refs/heads/master@{#579080}
[modify] https://crrev.com/bbabfa4fc9d13342562a1f252376fcbba0cfa880/chrome/browser/extensions/api/web_navigation/web_navigation_apitest.cc

Sign in to add a comment