8272064: test/jdk/jdk/jfr/api/consumer/TestHiddenMethod.java needs update for JEP 416

Reviewed-by: jbachorik

Author: egahlin

test/jdk/jdk/jfr/api/consumer/TestHiddenMethod.java

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2014, 2018, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2014, 2021, 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
@@ -51,10 +51,10 @@
  */
 public final class TestHiddenMethod {
 
-    public static void main(String[] args) throws Throwable {
-        try (Recording recording = new Recording()) {
-            recording.enable(MyEvent.class).withThreshold(Duration.ofMillis(0));
-            recording.start();
+    // Must call in separate thread because JTREG uses reflection
+    // to invoke main method, which uses hidden methods.
+    public static class TestThread extends Thread {
+        public void run() {
             // doPrivileged calls a method that has the @Hidden
             // annotation
             AccessController.doPrivileged(new PrivilegedAction<Void>() {
@@ -65,8 +65,19 @@ public Void run() {
                     return null;
                 }
             });
+
             MyEvent event = new MyEvent();
             event.commit();
+        }
+    }
+
+    public static void main(String[] args) throws Throwable {
+        try (Recording recording = new Recording()) {
+            recording.enable(MyEvent.class).withThreshold(Duration.ofMillis(0));
+            recording.start();
+            Thread t = new TestThread();
+            t.start();
+            t.join();
             recording.stop();
 
             List<RecordedEvent> events = Events.fromRecording(recording);
@@ -78,9 +89,7 @@ public Void run() {
             System.out.println("visibleEvent:" + visibleEvent);
 
             assertTrue(hasHiddenStackFrame(hiddenEvent), "No hidden frame in hidden event: " + hiddenEvent);
-
-            // Temporary disable this test until JDK-8272064 is resolved.
-            // assertFalse(hasHiddenStackFrame(visibleEvent), "Hidden frame in visible event: " + visibleEvent);
+            assertFalse(hasHiddenStackFrame(visibleEvent), "Hidden frame in visible event: " + visibleEvent);
         }
     }