Hi, Anton.
The fix looks good.
On 07.11.2014 17:13, Anton V. Tarasov wrote:
Hello,
Please, review the fix.
https://bugs.openjdk.java.net/browse/JDK-8004148
http://cr.openjdk.java.net/~ant/JDK-8004148/webrev.0
There's no a test case for the original issue. However, there's
another test case which reproduces NPE with the same stack trace. The
problem was fixed in jdk8 in JDK-8001633.
However, I suggest to do a sanity check for null. I check for null
AppContext which covers the case for null Window as well.
Thanks,
Anton.
--
Best regards, Sergey.