Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(341)

Unified Diff: content/public/android/javatests/src/org/chromium/content/common/CleanupReferenceTest.java

Issue 20210002: [Android] Sets up a coverage system for java using EMMA (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fixes gsutil upload Created 7 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/public/android/javatests/src/org/chromium/content/common/CleanupReferenceTest.java
diff --git a/content/public/android/javatests/src/org/chromium/content/common/CleanupReferenceTest.java b/content/public/android/javatests/src/org/chromium/content/common/CleanupReferenceTest.java
index a887ffb051ffae7eabd0c9bb5376b584a8a9072d..46e72916d5a653cb05e6c283e969d45925362379 100644
--- a/content/public/android/javatests/src/org/chromium/content/common/CleanupReferenceTest.java
+++ b/content/public/android/javatests/src/org/chromium/content/common/CleanupReferenceTest.java
@@ -7,6 +7,7 @@ package org.chromium.content.common;
import android.test.InstrumentationTestCase;
import android.test.suitebuilder.annotation.SmallTest;
+import org.chromium.base.test.util.DisabledTest;
import org.chromium.base.test.util.Feature;
import org.chromium.content.browser.test.util.Criteria;
import org.chromium.content.browser.test.util.CriteriaHelper;
@@ -36,6 +37,9 @@ public class CleanupReferenceTest extends InstrumentationTestCase {
}
}
+ volatile ReferredObject[] instances = null;
+ volatile ReferredObject instance = null;
+
@Override
public void setUp() throws Exception {
super.setUp();
@@ -49,12 +53,13 @@ public class CleanupReferenceTest extends InstrumentationTestCase {
System.gc();
}
- @SmallTest
- @Feature({"AndroidWebView"})
+ //@SmallTest
+ //@Feature({"AndroidWebView"})
+ @DisabledTest
bulach 2013/08/20 17:42:34 ?!
public void testCreateSingle() throws Throwable {
assertEquals(0, sObjectCount.get());
- ReferredObject instance = new ReferredObject();
+ instance = new ReferredObject();
assertEquals(1, sObjectCount.get());
instance = null;
@@ -67,13 +72,14 @@ public class CleanupReferenceTest extends InstrumentationTestCase {
}));
}
- @SmallTest
- @Feature({"AndroidWebView"})
+ //@SmallTest
+ //@Feature({"AndroidWebView"})
+ @DisabledTest
public void testCreateMany() throws Throwable {
assertEquals(0, sObjectCount.get());
final int INSTANCE_COUNT = 20;
- ReferredObject[] instances = new ReferredObject[INSTANCE_COUNT];
+ instances = new ReferredObject[INSTANCE_COUNT];
for (int i = 0; i < INSTANCE_COUNT; ++i) {
instances[i] = new ReferredObject();
« build/common.gypi ('K') | « content/content_shell.gypi ('k') | sync/sync_android.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698