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

Issue 599192 link

Starred by 3 users

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug



Sign in to add a comment

org.chromium.android_webview.test.AwQuotaManagerBridgeTest#testDeleteOriginWithAppCache flaky timeouts

Project Member Reported by grt@chromium.org, Mar 30 2016

Issue description

https://build.chromium.org/p/tryserver.chromium.android/builders/linux_android_rel_ng/builds/45324:

C  777.579s Main  [FAIL] org.chromium.android_webview.test.AwQuotaManagerBridgeTest#testDeleteOriginWithAppCache:
C  777.579s Main  junit.framework.AssertionFailedError: Criteria not met in allotted time.
 

Comment 1 by boliu@chromium.org, Apr 1 2016

C  777.579s Main  [FAIL] org.chromium.android_webview.test.AwQuotaManagerBridgeTest#testDeleteOriginWithAppCache:
C  777.579s Main  junit.framework.AssertionFailedError: Criteria not met in allotted time.
C  777.579s Main  	at org.chromium.content.browser.test.util.CriteriaHelper.pollInstrumentationThread(CriteriaHelper.java:74)
C  777.579s Main  	at org.chromium.android_webview.test.AwTestBase.pollInstrumentationThread(AwTestBase.java:499)
C  777.579s Main  	at org.chromium.android_webview.test.AwQuotaManagerBridgeTest.testDeleteOriginWithAppCache(AwQuotaManagerBridgeTest.java:197)
C  777.579s Main  	at java.lang.reflect.Method.invokeNative(Native Method)
C  777.579s Main  	at android.test.InstrumentationTestCase.runMethod(InstrumentationTestCase.java:214)
C  777.579s Main  	at android.test.InstrumentationTestCase.runTest(InstrumentationTestCase.java:199)
C  777.579s Main  	at android.test.ActivityInstrumentationTestCase2.runTest(ActivityInstrumentationTestCase2.java:192)
C  777.580s Main  	at org.chromium.base.test.BaseTestResult.runParameterized(BaseTestResult.java:161)
C  777.580s Main  	at org.chromium.base.test.BaseTestResult.run(BaseTestResult.java:124)
C  777.580s Main  	at android.test.AndroidTestRunner.runTest(AndroidTestRunner.java:191)
C  777.580s Main  	at android.test.AndroidTestRunner.runTest(AndroidTestRunner.java:176)
C  777.580s Main  	at android.test.InstrumentationTestRunner.onStart(InstrumentationTestRunner.java:554)
C  777.580s Main  	at android.app.Instrumentation$InstrumentationThread.run(Instrumentation.java:1701)

Comment 2 by boliu@chromium.org, Apr 1 2016

Running this test in a loop exposed other problems. The test server is failing to shutdown :/
Cc: hush@chromium.org
I guess you mean ctstestserver. Do you have hush's fix? 

Comment 5 by boliu@chromium.org, Apr 1 2016

No. This is org.chromium.net.test.util.TestWebServer, but it originally was forked from the cts server.

afaict this is not ssl though, so doesn't look related. I'll file another bug for that.

Comment 6 by boliu@chromium.org, Apr 1 2016

Forked that to  crbug.com/600021 

Comment 8 by hush@chromium.org, May 9 2016

Status: Assigned (was: Untriaged)
Hi Bo,
These tests can be re-enabled now after you've fixed the shutdown deadlock

Comment 9 by boliu@chromium.org, May 9 2016

These tests has nothing to do with the test server shutdown issue. I ran one of these tests overnight to test the test server shutdown fix.

I think the flakes here are all from tests leaving storage data around after it's done, so repeating a single test isn't helping. Not sure.

Sign in to add a comment