looks ok to me (apart from some indentation issue in the test)
Regards Prasanta On 27-Jun-19 2:01 AM, Phil Race wrote:
Bug: https://bugs.openjdk.java.net/browse/JDK-8226783 Webrev: http://cr.openjdk.java.net/~prr/8226783/ This fix is for JDK 13 to fix a regression in a recent build of JDK 13. We use the presence of an environment variable GNOME_DESKTOP_SESSION_ID to determine if GTK should be the System L&F on Linux & Solaris Due to a typo when relocating this from native code to Java code JDK 13 fails to find it and defaults to Metal as the system L&F - this is what we do on a KDE desktop for example. I've added a test that would have helped spot this outage earlier and should help in a similar future situation. -phil.
