New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 653130 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner: ----
Closed: Dec 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug



Sign in to add a comment

"org.chromium.android_webview.test.ClientOnReceivedError2Test#testUserGestureForIframeSubresource" is flaky

Project Member Reported by chromium...@appspot.gserviceaccount.com, Oct 5 2016

Issue description

"org.chromium.android_webview.test.ClientOnReceivedError2Test#testUserGestureForIframeSubresource" 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 4 recent flakes. List of all flakes can be found at https://chromium-try-flakes.appspot.com/all_flake_occurrences?key=ahVzfmNocm9taXVtLXRyeS1mbGFrZXNyawsSBUZsYWtlImBvcmcuY2hyb21pdW0uYW5kcm9pZF93ZWJ2aWV3LnRlc3QuQ2xpZW50T25SZWNlaXZlZEVycm9yMlRlc3QjdGVzdFVzZXJHZXN0dXJlRm9ySWZyYW1lU3VicmVzb3VyY2UM.

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 boliu@chromium.org, Oct 5 2016

Components: Mobile>WebView
Labels: -Sheriff-Chromium
Adding RetryOnFailure: https://codereview.chromium.org/2393033003/
Project Member

Comment 2 by bugdroid1@chromium.org, Oct 5 2016

Comment 3 by boliu@chromium.org, Oct 5 2016

 Issue 653184  has been merged into this issue.
Project Member

Comment 4 by bugdroid1@chromium.org, Oct 27 2016

Labels: merge-merged-2840
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/88dc583d32e0ec09b9afacb2dfa71ef531b30a80

commit 88dc583d32e0ec09b9afacb2dfa71ef531b30a80
Author: boliu <boliu@chromium.org>
Date: Wed Oct 05 16:51:19 2016

aw: Mark testUserGestureForIframeSubresource as retry

BUG= 653130 
TBR=sgurun@chromium.org

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

[modify] https://crrev.com/88dc583d32e0ec09b9afacb2dfa71ef531b30a80/android_webview/javatests/src/org/chromium/android_webview/test/ClientOnReceivedError2Test.java

Comment 5 by dimu@google.com, Nov 4 2016

Labels: -merge-merged-2840
[Automated comment] removing mislabelled merge-merged-2840
Huh, this passes consistently on Nexus 6 on N (20/20) but on Nexus 5 on Lollipop it flakes a lot. I'll try to take a look.
Cc: jbudorick@chromium.org boliu@chromium.org
Actually, after turning on the 'Developer Settings -> Stay Awake' setting my Nexus 5 running Lollipop also passes 20/20 test runs. Could it be that our trybots do not have that Setting enabled?

Who knows about bot setups?
Cc: gsennton@chromium.org

Comment 9 by sgu...@chromium.org, Nov 21 2016

Cc: stip@chromium.org
I think it could be a problem specific to that one device. I think stip could know ?
They should have that set (https://codesearch.chromium.org/chromium/src/build/android/pylib/device_settings.py?rcl=0&l=159), but we've seen a few related similar issues lately.
Right, so WDYT, should we just try marking these tests as non-flaky and see what happens?
Project Member

Comment 13 by bugdroid1@chromium.org, Dec 22 2016

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

commit 09c88537d125e970cfc1252735dda30b424a6f10
Author: boliu <boliu@chromium.org>
Date: Thu Dec 22 22:30:56 2016

Remove retry for testUserGestureForIframeSubresource

Theory is due to misconfigured device. See bug for details.

BUG= 653130 

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

[modify] https://crrev.com/09c88537d125e970cfc1252735dda30b424a6f10/android_webview/javatests/src/org/chromium/android_webview/test/ClientOnReceivedError2Test.java

Comment 14 by boliu@chromium.org, Dec 22 2016

Status: Fixed (was: Untriaged)

Sign in to add a comment