Thumbs up as long as the tests well cover the cases.
> 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.
> 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:
>>> 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?