New issue
Advanced search Search tips

Issue 596233 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Apr 2016
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

CleanupReferenceTest fails all the time on Marshmallow 64 bit Tester

Project Member Reported by hush@chromium.org, Mar 19 2016

Issue description

C  289.647s Main  [FAIL] org.chromium.content.common.CleanupReferenceTest#testCreateMany:
C  289.647s Main  junit.framework.AssertionFailedError: Values did not match. Expected: 0, actual: 20
C  289.647s Main  	at org.chromium.content.browser.test.util.CriteriaHelper.pollInstrumentationThread(CriteriaHelper.java:74)
C  289.647s Main  	at org.chromium.content.browser.test.util.CriteriaHelper.pollInstrumentationThread(CriteriaHelper.java:89)
C  289.647s Main  	at org.chromium.content.common.CleanupReferenceTest.testCreateMany(CleanupReferenceTest.java:99)
C  289.647s Main  	at android.test.InstrumentationTestCase.runMethod(InstrumentationTestCase.java:214)
C  289.647s Main  	at android.test.InstrumentationTestCase.runTest(InstrumentationTestCase.java:199)
C  289.647s Main  	at org.chromium.base.test.BaseTestResult.run(BaseTestResult.java:129)
C  289.647s Main  	at android.test.AndroidTestRunner.runTest(AndroidTestRunner.java:191)
C  289.647s Main  	at android.test.AndroidTestRunner.runTest(AndroidTestRunner.java:176)
C  289.647s Main  	at android.test.InstrumentationTestRunner.onStart(InstrumentationTestRunner.java:555)
C  289.647s Main  	at android.app.Instrumentation$InstrumentationThread.run(Instrumentation.java:1879)
C  289.647s Main  
C  289.647s Main  [FAIL] org.chromium.content.common.CleanupReferenceTest#testCreateSingle:
C  289.648s Main  junit.framework.AssertionFailedError: Values did not match. Expected: 0, actual: 1
C  289.648s Main  	at org.chromium.content.browser.test.util.CriteriaHelper.pollInstrumentationThread(CriteriaHelper.java:74)
C  289.648s Main  	at org.chromium.content.browser.test.util.CriteriaHelper.pollInstrumentationThread(CriteriaHelper.java:89)
C  289.648s Main  	at org.chromium.content.common.CleanupReferenceTest.testCreateSingle(CleanupReferenceTest.java:70)
C  289.648s Main  	at android.test.InstrumentationTestCase.runMethod(InstrumentationTestCase.java:214)
C  289.648s Main  	at android.test.InstrumentationTestCase.runTest(InstrumentationTestCase.java:199)
C  289.648s Main  	at org.chromium.base.test.BaseTestResult.run(BaseTestResult.java:129)
C  289.648s Main  	at android.test.AndroidTestRunner.runTest(AndroidTestRunner.java:191)
C  289.648s Main  	at android.test.AndroidTestRunner.runTest(AndroidTestRunner.java:176)
C  289.648s Main  	at android.test.InstrumentationTestRunner.onStart(InstrumentationTestRunner.java:555)
C  289.648s Main  	at android.app.Instrumentation$InstrumentationThread.run(Instrumentation.java:1879)
C  289.648s Main  ********************************************************************************
C  289.648s Main  Summary
C  289.648s Main  ********************************************************************************


Hello agrieve: you are the last person who attempted to fix it (https://codereview.chromium.org/1423713013/), so I'm cc-ing you. 
 
Owner: boliu@chromium.org
This test just wasn't meant to be I think. Time to delete it?

Comment 3 by boliu@chromium.org, Mar 19 2016

I'm ok to disable for now.

There is no reason why this test fails/flakes when the webview gc tests are running just fine..
Tests are failing for the same reason on Marshmallow Tablet Tester, e.g.:
https://build.chromium.org/p/chromium.android/builders/Marshmallow%20Tablet%20Tester/builds/3303

Disable test?

Comment 5 by boliu@chromium.org, Mar 22 2016

Someone go ahead and disable it. I don't have time to look at this right now
Project Member

Comment 6 by bugdroid1@chromium.org, Apr 12 2016

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

commit 400351d906e94e6c4117d7a946af6bde109f368f
Author: boliu <boliu@chromium.org>
Date: Tue Apr 12 15:36:34 2016

Fix CleanupReferenceTest on Android M

Because android

BUG= 596233 

Review URL: https://codereview.chromium.org/1880793002

Cr-Commit-Position: refs/heads/master@{#386693}

[modify] https://crrev.com/400351d906e94e6c4117d7a946af6bde109f368f/content/public/android/javatests/src/org/chromium/content/common/CleanupReferenceTest.java

Comment 7 by boliu@chromium.org, Apr 12 2016

Status: Fixed (was: Available)

Sign in to add a comment