Hi, Mandy I agree that the new permission type is overkill. The changes look good to me.
-Brent On 3/15/17 12:42 PM, Mandy Chung wrote:
StackWalker::getInstance is currently specified to check for StackFramePermission("retainClassReference“) due to an early review feedback . Given it has only one target, it’s overkill to define a specific permission type for stack walking use. This patch proposes to replace the StackWalker permission check with RuntimePermission("getStackWalkerWithClassReference”) and remove StackFramePermission class. http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8176815/webrev.00/ Mandy