Please review the updated fix: http://cr.openjdk.java.net/~pchelko/8011142/webrev.01/
It appears that we do not have a regression test for the AppContext fix. So it was decided to change the test instead of removing it. Thank you. With best regards. Petr. On 26.11.2013, at 17:19, Anthony Petrov <[email protected]> wrote: > +1. > > -- > best regards, > Anthony > > On 11/26/2013 04:16 PM, Petr Pchelko wrote: >> Hello, AWT Team. >> >> Please review the fix for the issue: >> https://bugs.openjdk.java.net/browse/JDK-8011142 >> The fix is available at: >> http://cr.openjdk.java.net/~pchelko/8011142/webrev/ >> >> The MainAppContext test verified that the mainAppContext is created when >> there's already another AppContext created. >> After the AppContext fix this is incorrect, so the test should be removed, >> as it's expecting the wrong behavior. The second >> test mentioned in the bug was already fixed. >> >> We cannot modify the test, because it's testing exclusively the behavior >> which is wrong now. >> >> Thank you. >> With best regards. Petr. >>
