commit ddb8afec79492750f16f98e5bd07d44f3dc485d4
author Vladimir Marko <vmarko@google.com> Tue Nov 22 14:38:31 2016 +0000
committer Vladimir Marko <vmarko@google.com> Tue Nov 22 14:42:49 2016 +0000
tree 002f092e054a3c53deccd55a972f53dfe58a1d0c
parent 0ba0c7d9b0328af39a385014652eb9af4e42d8b5

Do not test class unloading in 626-const-class-linking

Keep tests that classes were not unloaded but do not test
that they are actually unloaded. Both JIT and tracing can
hold onto those classes.

Test: m ART_USE_JIT=true test-art-host-run-test-626-const-class-linking
Test: m ART_TEST_TRACE=true test-art-host-run-test-626-const-class-linking
Bug: 30627598
Change-Id: If9d1da45d8691c5ee05b6cf500d7d132bcab611f

test/626-const-class-linking/src/Main.java

diff --git a/test/626-const-class-linking/src/Main.java b/test/626-const-class-linking/src/Main.java
index 44ea1c1..05a6bac 100644
--- a/test/626-const-class-linking/src/Main.java
+++ b/test/626-const-class-linking/src/Main.java
@@ -49,6 +49,7 @@
         }
         Runtime.getRuntime().gc();
         WeakReference<Class<?>> weak_test2 = wrapHelperGet(helper);
+        Runtime.getRuntime().gc();
 
         Class<?> test1 = weak_test1.get();
         if (test1 == null) {
@@ -62,20 +63,6 @@
             System.out.println("test1 != test2");
         }
 
-        test1 = null;
-        test2 = null;
-        if (!usingRI) {
-            clearResolvedTypes(helper);
-        }
-        helper = null;
-        delegating_loader = null;
-        Runtime.getRuntime().gc();
-        if (weak_test1.get() != null) {
-            System.out.println("weak_test1 still not null");
-        }
-        if (weak_test2.get() != null) {
-            System.out.println("weak_test2 still not null");
-        }
         System.out.println("testClearDexCache done");
     }
 
@@ -90,6 +77,8 @@
         Class<?> helper2 = delegating_loader.loadClass("Helper2");
         WeakReference<Class<?>> weak_test2 = wrapHelperGet(helper2);
 
+        Runtime.getRuntime().gc();
+
         Class<?> test1 = weak_test1.get();
         if (test1 == null) {
             System.out.println("test1 disappeared");
@@ -102,18 +91,6 @@
             System.out.println("test1 != test2");
         }
 
-        test1 = null;
-        test2 = null;
-        delegating_loader = null;
-        helper1 = null;
-        helper2 = null;
-        Runtime.getRuntime().gc();
-        if (weak_test1.get() != null) {
-            System.out.println("weak_test1 still not null");
-        }
-        if (weak_test2.get() != null) {
-            System.out.println("weak_test2 still not null");
-        }
         System.out.println("testMultiDex done");
     }
 
@@ -271,38 +248,38 @@
     }
 
     private static void testRacyMisbehavingLoader2() throws Exception {
-      final ClassLoader system_loader = ClassLoader.getSystemClassLoader();
+        final ClassLoader system_loader = ClassLoader.getSystemClassLoader();
 
-      final Thread[] threads = new Thread[4];
-      final Object[] results = new Object[threads.length];
+        final Thread[] threads = new Thread[4];
+        final Object[] results = new Object[threads.length];
 
-      final RacyMisbehavingLoader racy_loader =
-          new RacyMisbehavingLoader(system_loader, threads.length, true);
-      final Class<?> helper1 = racy_loader.loadClass("RacyMisbehavingHelper");
+        final RacyMisbehavingLoader racy_loader =
+            new RacyMisbehavingLoader(system_loader, threads.length, true);
+        final Class<?> helper1 = racy_loader.loadClass("RacyMisbehavingHelper");
 
-      for (int i = 0; i != threads.length; ++i) {
-          final int my_index = i;
-          Thread t = new Thread() {
-              public void run() {
-                  try {
-                      Method get = helper1.getDeclaredMethod("get");
-                      results[my_index] = get.invoke(null);
-                  } catch (InvocationTargetException ite) {
-                      results[my_index] = ite.getCause();
-                  } catch (Throwable t) {
-                      results[my_index] = t;
-                  }
-              }
-          };
-          t.start();
-          threads[i] = t;
-      }
-      for (Thread t : threads) {
-          t.join();
-      }
-      dumpResultStats(results);
-      System.out.println("testRacyMisbehavingLoader2 done");
-  }
+        for (int i = 0; i != threads.length; ++i) {
+            final int my_index = i;
+            Thread t = new Thread() {
+                public void run() {
+                    try {
+                        Method get = helper1.getDeclaredMethod("get");
+                        results[my_index] = get.invoke(null);
+                    } catch (InvocationTargetException ite) {
+                        results[my_index] = ite.getCause();
+                    } catch (Throwable t) {
+                        results[my_index] = t;
+                    }
+                }
+            };
+            t.start();
+            threads[i] = t;
+        }
+        for (Thread t : threads) {
+            t.join();
+        }
+        dumpResultStats(results);
+        System.out.println("testRacyMisbehavingLoader2 done");
+    }
 
     private static DelegatingLoader createDelegatingLoader() {
         ClassLoader system_loader = ClassLoader.getSystemClassLoader();