New issue
Advanced search Search tips

Issue 901280 link

Starred by 2 users

Issue metadata

Status: Assigned
Owner:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 1
Type: Bug
Flaky-Test: org.chromium.chrome.browser.fullscreen.FullscreenManagerTest#testManualHidingShowingBrowserControls



Sign in to add a comment

chrome_public_test_apk (FullscreenManagerTest) failing consistently on chromium.memory/Android CFI

Project Member Reported by sheriff-...@appspot.gserviceaccount.com, Nov 2

Issue description

Filed by sheriff-o-matic@appspot.gserviceaccount.com on behalf of vitaliii@google.com

chrome_public_test_apk failing on chromium.memory/Android CFI

Builders failed on: 
- Android CFI: 
  https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Android%20CFI


 
Failing tests are
org.chromium.chrome.browser.fullscreen.FullscreenManagerTest#testManualHidingShowingBrowserControls
org.chromium.chrome.browser.fullscreen.FullscreenManagerTest#testHideBrowserControlsAfterFlingBoosting

The first (known) failure is in https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Android%20CFI/3495
There are also  issue 901188  and  901187  from Findit saying that these tests are flaky, however, here they failed 5 times in a row.
Logs from https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Android%20CFI/3495
===========
java.lang.AssertionError: Didn't get expected ScrollEnd gestures
	at org.junit.Assert.fail(Assert.java:88)
	at org.chromium.chrome.browser.fullscreen.FullscreenManagerTestUtils.waitForBrowserControlsToBeMoveable(FullscreenManagerTestUtils.java:201)
	at org.chromium.chrome.browser.fullscreen.FullscreenManagerTest.testManualHidingShowingBrowserControls(FullscreenManagerTest.java:245)
	at java.lang.reflect.Method.invoke(Method.java)
	at org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50)
	at org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
	at org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:52)
	at org.junit.internal.runners.statements.InvokeMethod.evaluate(InvokeMethod.java:17)
..........
===========
java.lang.AssertionError: Didn't get expected ScrollEnd gestures
	at org.junit.Assert.fail(Assert.java:88)
	at org.chromium.chrome.browser.fullscreen.FullscreenManagerTestUtils.waitForBrowserControlsToBeMoveable(FullscreenManagerTestUtils.java:201)
	at org.chromium.chrome.browser.fullscreen.FullscreenManagerTest.testHideBrowserControlsAfterFlingBoosting(FullscreenManagerTest.java:263)
	at java.lang.reflect.Method.invoke(Method.java)
	at org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50)
	at org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
	at org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:52)
	at org.junit.internal.runners.statements.InvokeMethod.evaluate(InvokeMethod.java:17)
	at org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:26)
..........
===========
Android sheriff observers similar issues in issue 901278.
There is also issue 872264 from Aug 8, which has similar error log for testHideBrowserControlsAfterFlingBoosting 
Cc: bokan@chromium.org vitaliii@chromium.org
Adding bokan@ who added both tests.
I can't find any obvious culprit, disabling the tests.
Moreover in https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Android%20CFI/3500 only org.chromium.chrome.browser.fullscreen.FullscreenManagerTest#testHideBrowserControlsAfterFlingBoosting fails, so they indeed look flaky.
 Issue 901279  has been merged into this issue.
Cc: -bokan@chromium.org
Components: Tests>Flaky
Labels: OS-Android Type-Bug-Regression
Owner: bokan@chromium.org
Status: Assigned (was: Available)
 Issue 901187  has been merged into this issue.
 Issue 901188  has been merged into this issue.
Project Member

Comment 14 by bugdroid1@chromium.org, Nov 2

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

commit 4c5496274240356e0f6325c04aca69fce7c855b6
Author: vitaliii <vitaliii@chromium.org>
Date: Fri Nov 02 11:26:49 2018

Disable 2 FullscreenManagerTest tests.

Disable
org.chromium.chrome.browser.fullscreen.FullscreenManagerTest#testManualHidingShowingBrowserControls
org.chromium.chrome.browser.fullscreen.FullscreenManagerTest#testHideBrowserControlsAfterFlingBoosting
because flaky.

TBR=bokan@chromium.org

Bug: 901280
Change-Id: I5eb8a5c418ac0b83440d26fd2dad07e112a32501
Reviewed-on: https://chromium-review.googlesource.com/c/1314470
Reviewed-by: vitaliii <vitaliii@chromium.org>
Commit-Queue: vitaliii <vitaliii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#604889}
[modify] https://crrev.com/4c5496274240356e0f6325c04aca69fce7c855b6/chrome/android/javatests/src/org/chromium/chrome/browser/fullscreen/FullscreenManagerTest.java

Issue 901278 has been merged into this issue.
Labels: -Sheriff-Chromium
bokan@, please have a look!
Cc: -vitaliii@chromium.org
Project Member

Comment 18 by Findit, Nov 3

Flaky-Test: org.chromium.chrome.browser.fullscreen.FullscreenManagerTest#testHideBrowserControlsAfterFlingBoosting
Labels: Type-Bug Test-Flaky Test-Findit-Detected Sheriff-Chromium

org.chromium.chrome.browser.fullscreen.FullscreenManagerTest#testHideBrowserControlsAfterFlingBoosting is flaky.

Findit has detected 4 new flake occurrences of this test. List
of all flake occurrences can be found at:
https://findit-for-me.appspot.com/flake/occurrences?key=ag9zfmZpbmRpdC1mb3ItbWVykgELEgVGbGFrZSKGAWNocm9taXVtQGNocm9tZV9wdWJsaWNfdGVzdF9hcGtAb3JnLmNocm9taXVtLmNocm9tZS5icm93c2VyLmZ1bGxzY3JlZW4uRnVsbHNjcmVlbk1hbmFnZXJUZXN0I3Rlc3RIaWRlQnJvd3NlckNvbnRyb2xzQWZ0ZXJGbGluZ0Jvb3N0aW5nDA.

Since this test is still flaky, this issue has been moved back onto the Sheriff
Bug Queue if it's not already there.

This flaky test was previously tracked in  bug 901187 .

If the result above is wrong, please file a bug using this link:
https://bugs.chromium.org/p/chromium/issues/entry?status=Unconfirmed&labels=Pri-1,Test-Findit-Wrong&components=Tools%3ETest%3EFindit%3EFlakiness&summary=%5BFindit%5D%20Flake%20Detection%20-%20Wrong%20result%20for%20org.chromium.chrome.browser.fullscreen.FullscreenManagerTest#testHideBrowserControlsAfterFlingBoosting&comment=Link%20to%20flake%20occurrences%3A%20https://findit-for-me.appspot.com/flake/occurrences?key=ag9zfmZpbmRpdC1mb3ItbWVykgELEgVGbGFrZSKGAWNocm9taXVtQGNocm9tZV9wdWJsaWNfdGVzdF9hcGtAb3JnLmNocm9taXVtLmNocm9tZS5icm93c2VyLmZ1bGxzY3JlZW4uRnVsbHNjcmVlbk1hbmFnZXJUZXN0I3Rlc3RIaWRlQnJvd3NlckNvbnRyb2xzQWZ0ZXJGbGluZ0Jvb3N0aW5nDA

Automatically posted by the findit-for-me app (https://goo.gl/Ot9f7N).
Project Member

Comment 19 by Findit, Nov 3

Flaky-Test: org.chromium.chrome.browser.fullscreen.FullscreenManagerTest#testManualHidingShowingBrowserControls

org.chromium.chrome.browser.fullscreen.FullscreenManagerTest#testManualHidingShowingBrowserControls is flaky.

Findit has detected 4 new flake occurrences of this test. List
of all flake occurrences can be found at:
https://findit-for-me.appspot.com/flake/occurrences?key=ag9zfmZpbmRpdC1mb3ItbWVyjwELEgVGbGFrZSKDAWNocm9taXVtQGNocm9tZV9wdWJsaWNfdGVzdF9hcGtAb3JnLmNocm9taXVtLmNocm9tZS5icm93c2VyLmZ1bGxzY3JlZW4uRnVsbHNjcmVlbk1hbmFnZXJUZXN0I3Rlc3RNYW51YWxIaWRpbmdTaG93aW5nQnJvd3NlckNvbnRyb2xzDA.

Since this test is still flaky, this issue has been moved back onto the Sheriff
Bug Queue if it's not already there.

This flaky test was previously tracked in  bug 901188 .

If the result above is wrong, please file a bug using this link:
https://bugs.chromium.org/p/chromium/issues/entry?status=Unconfirmed&labels=Pri-1,Test-Findit-Wrong&components=Tools%3ETest%3EFindit%3EFlakiness&summary=%5BFindit%5D%20Flake%20Detection%20-%20Wrong%20result%20for%20org.chromium.chrome.browser.fullscreen.FullscreenManagerTest#testManualHidingShowingBrowserControls&comment=Link%20to%20flake%20occurrences%3A%20https://findit-for-me.appspot.com/flake/occurrences?key=ag9zfmZpbmRpdC1mb3ItbWVyjwELEgVGbGFrZSKDAWNocm9taXVtQGNocm9tZV9wdWJsaWNfdGVzdF9hcGtAb3JnLmNocm9taXVtLmNocm9tZS5icm93c2VyLmZ1bGxzY3JlZW4uRnVsbHNjcmVlbk1hbmFnZXJUZXN0I3Rlc3RNYW51YWxIaWRpbmdTaG93aW5nQnJvd3NlckNvbnRyb2xzDA

Automatically posted by the findit-for-me app (https://goo.gl/Ot9f7N).
Labels: -Sheriff-Chromium
Removing from sheriff queue since the tests were disabled.
Sorry - had more urgent issues last week. Will investigate this week.

Sign in to add a comment