On Thu, May 23, 2019 at 9:23 AM Sean Mullan <[email protected]> wrote:
> Looks fine although you should update the 8224635 title to match the > changeset. > Done > > Also, I assume you will file a followon bug for the original issue > (JDK-8224256). I think a new bug must be filed instead of re-opening > 8224256 because a changeset has already been pushed for it. > Yes I will file another bug to get this back in, this time making sure that it passes on mach5 tier2 before submitting. > > --Sean > > On 5/23/19 12:08 PM, Arthur Eubanks wrote: > > bug: https://bugs.openjdk.java.net/browse/JDK-8224635 > > webrev: > http://cr.openjdk.java.net/~aeubanks/8224635/webrev.00/index.html > > > > Test java/security/SecureClassLoader/DefineClass.java is failing after > > JDK-8224256 > > This change reverts the changes in JDK-8224256 and adds back in the test. >
