New issue
Advanced search Search tips

Issue 716763 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: May 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

"org.chromium.android_webview.test.crash.VisualStateCallbackTest#testVisualStateCallbackNotCalledAfterRendererGone" is flaky

Project Member Reported by chromium...@appspot.gserviceaccount.com, Apr 29 2017

Issue description

"org.chromium.android_webview.test.crash.VisualStateCallbackTest#testVisualStateCallbackNotCalledAfterRendererGone" 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=ahVzfmNocm9taXVtLXRyeS1mbGFrZXNyfAsSBUZsYWtlInFvcmcuY2hyb21pdW0uYW5kcm9pZF93ZWJ2aWV3LnRlc3QuY3Jhc2guVmlzdWFsU3RhdGVDYWxsYmFja1Rlc3QjdGVzdFZpc3VhbFN0YXRlQ2FsbGJhY2tOb3RDYWxsZWRBZnRlclJlbmRlcmVyR29uZQw.

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
 

Comment 1 by suzyh@chromium.org, May 1 2017

Cc: michaelbai@chromium.org tobiasjs@chromium.org
Components: Mobile>WebView
Labels: -Sheriff-Chromium
Adding @RetryOnFailure as per sheriffing instructions in https://codereview.chromium.org/2851133002
Project Member

Comment 2 by bugdroid1@chromium.org, May 1 2017

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

commit 7f022a2ccb293c5b31446f3f5af8be79f46555ba
Author: suzyh <suzyh@chromium.org>
Date: Mon May 01 03:19:02 2017

Set flaky testVisualStateCa... to retry on failure

Add @RetryOnFailure annotation for
testVisualStateCallbackNotCalledAfterRendererGone

TBR=tobiasjs@chromium.org
BUG= 716763 

Review-Url: https://codereview.chromium.org/2851133002
Cr-Commit-Position: refs/heads/master@{#468273}

[modify] https://crrev.com/7f022a2ccb293c5b31446f3f5af8be79f46555ba/android_webview/javatests/src/org/chromium/android_webview/test/crash/VisualStateCallbackTest.java

Checked the failure log, restarting render process took 2 sec, and there was blocking gc, device seemed overload at that time.

From https://test-results.appspot.com/dashboards/flakiness_dashboard.html#testType=android_webview_test_apk&showAllRuns=true&tests=org.chromium.android_webview.test.crash.VisualStateCallbackTest%23testVisualStateCallbackNotCalledAfterRendererGone

The slowest running time is from 4-13 seconds, we might extend waiting time.

Comment 4 by ti...@chromium.org, May 4 2017

Cc: -michaelbai@chromium.org
Labels: -Pri-1 Pri-2
Owner: michaelbai@chromium.org
Status: Assigned (was: Untriaged)
Status: Fixed (was: Assigned)

Sign in to add a comment