Thumbs up as long as the tests well cover the cases. Mandy
> On Oct 13, 2016, at 2:19 PM, Xueming Shen <xueming.s...@oracle.com> wrote: > > Yes, there are couple test cases in the cs unit/regression tests running > under security manager > when testing. > > sherman > > On 10/13/2016 02:13 PM, Mandy Chung wrote: >>> On Oct 13, 2016, at 2:03 PM, Xueming Shen<xueming.s...@oracle.com> wrote: >>> >>> Hi >>> >>> Please help review the change for JDK-8167957 >>> >>> issue: https://bugs.openjdk.java.net/browse/JDK-8167957 >>> webrev: http://cr.openjdk.java.net/~sherman/8167957/webrev >> The change looks fine. Is there any existing test running with security >> manager to verify this? >> >> Mandy >> >