Re: <AWT Dev> [8u-Backport] Review request for 8185634, 8187803 : Java Fx-Swing dialogs appearing behind main stage
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
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.
One thing I notice is that in XWindowPeer.java the calculation for the
value of transientForWindow
is different .. because you don't have the fix for
"setAlwaysOnTop doesn't behave correctly in Linux/Solaris under certain
Do you first need to backport that too ? Maybe Semyon (who authored that
fix) can say whether
it matters ..