[8u-Backport] Review request for 8185634, 8187803 : Java Fx-Swing dialogs appearing behind main stage

Kevin Rushforth kevin.rushforth at oracle.com
Wed Jan 17 21:43:11 UTC 2018


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
>


More information about the openjfx-dev mailing list