Hi Dipak,
To underscore the fact that the AWT part of the change has to be pushed
under a separate bug ID I have modified the subject line.
The FX half of the implementation changes look fine. I tested it on
Linux and it works as expected. There are two problems with the unit tests:
1. The package name used in each of the three test files doesn't match
the location in the directory tree.
[This one must be fixed]
2. When used with the default boot JDK of 8u40, the unit tests will
fail. This may not be a problem we care about, since they are robot
tests and won't get run without explicitly enabling robot tests.
[Feel free to ignore this point]
As for the AWT half of the changes, you will need to get two (?) AWT
Reviewers to approve, and then also get permission to backport this to
jdk8u by sending email to jdk8u-dev requesting such permission.
-- Kevin
Dipak Kumar wrote:
Hi All,
Please review the below patch (for 8u-backport) --
Webrev : http://cr.openjdk.java.net/~dkumar/8185634/webrev.00/
<http://cr.openjdk.java.net/%7Edkumar/8185634/webrev.00/>
JBS - https://bugs.openjdk.java.net/browse/JDK-8185634
Thanks,
Dipak
