8206330: Revisit com/sun/jdi/RedefineCrossEvent.java

Reviewed-by: cjplummer, sspitsyn, amenkov
openjdk · Oct 23, 2018 · 4268737 · 4268737
1 parent f853aeb
commit 4268737
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 13 deletions.
diff --git a/test/jdk/ProblemList.txt b/test/jdk/ProblemList.txt
@@ -874,14 +874,6 @@ javax/rmi/ssl/SSLSocketParametersTest.sh                        8162906 generic-
 
 ############################################################################
 
-###########################################################################
-#
-# Java EE Module Removal
-#
-com/sun/jdi/RedefineCrossEvent.java                             8194308    generic-all  Java EE Module Removal
-
-############################################################################
-
 # jdk_jfr
 
 jdk/jfr/event/io/TestInstrumentation.java                       8202142    generic-all

diff --git a/test/jdk/com/sun/jdi/RedefineCrossEvent.java b/test/jdk/com/sun/jdi/RedefineCrossEvent.java
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2002, 2015, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2002, 2018, Oracle and/or its affiliates. All rights reserved.
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
  *
  * This code is free software; you can redistribute it and/or modify it
@@ -27,9 +27,6 @@
  * @summary Test class redefinition at each event cross tested with other tests
  * @author Robert Field
  *
- * @modules java.corba
- *          jdk.jdi
- *
  * @run build TestScaffold VMConnection TargetAdapter TargetListener
  * @run compile -g AccessSpecifierTest.java
  * @run compile -g AfterThreadDeathTest.java

diff --git a/test/jdk/com/sun/jdi/TestScaffold.java b/test/jdk/com/sun/jdi/TestScaffold.java
@@ -542,7 +542,9 @@ public void eventReceived(Event event) {
                                 redefine(mainStartClass);
                             }
                         } else {
-                            redefine(rt);
+                            if (!name.startsWith("jdk.")) {
+                                redefine(rt);
+                            }
                         }
                     }
                 }