Fix looks fine. For test, is it required to explicitly set the l&f, I guess we should check for installed l&f for a particular platform and not unnecessarily check for Aqua & Gtk in windows?

Regards
Prasanta
On 1/3/2018 3:16 AM, Sergey Bylokhov wrote:
Hello.
Please review the fix for jdk10. The fix will be pushed to:
http://hg.openjdk.java.net/jdk/jdk10

Bug: https://bugs.openjdk.java.net/browse/JDK-8193673
Webrev can be found at: http://cr.openjdk.java.net/~serb/8193673/webrev.00

The fix for JDK-8189656[1] missed an update of Gensrc-java.desktop.gmk which compiles the property files for windows L&F.

[1] https://bugs.openjdk.java.net/browse/JDK-8189656


Reply via email to