Issue metadata
Sign in to add a comment
|
chrome_public_test_apk (FullscreenManagerTest) failing consistently on chromium.memory/Android CFI |
||||||||||||||||||||
Issue descriptionFiled 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
,
Nov 2
There are also issue 901188 and 901187 from Findit saying that these tests are flaky, however, here they failed 5 times in a row.
,
Nov 2
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) .......... ===========
,
Nov 2
Android sheriff observers similar issues in issue 901278.
,
Nov 2
There is also issue 872264 from Aug 8, which has similar error log for testHideBrowserControlsAfterFlingBoosting
,
Nov 2
Adding bokan@ who added both tests.
,
Nov 2
I can't find any obvious culprit, disabling the tests.
,
Nov 2
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.
,
Nov 2
The pending CL is https://chromium-review.googlesource.com/c/chromium/src/+/1314470.
,
Nov 2
Issue 901279 has been merged into this issue.
,
Nov 2
,
Nov 2
Issue 901187 has been merged into this issue.
,
Nov 2
Issue 901188 has been merged into this issue.
,
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
,
Nov 2
Issue 901278 has been merged into this issue.
,
Nov 2
bokan@, please have a look!
,
Nov 2
,
Nov 3
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).
,
Nov 3
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).
,
Nov 5
Removing from sheriff queue since the tests were disabled.
,
Nov 5
Sorry - had more urgent issues last week. Will investigate this week. |
|||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||
Comment 1 by vitaliii@chromium.org
, Nov 2