Re: RFR: 8212700: Change the mechanism by which JDK loads the platform-specific AWT Toolkit

2019-05-08 Thread Sergey Bylokhov
OK, looks fine then. On 5/7/19, 7:47 PM, Sergey Bylokhov wrote: Hi, Phil. Looks fine, but probably it will be worth to rename PlatformGraphicsInfo to PlatformInfo? I don't want (or intend) to do that. I already knew I was going to re-use this class for this purpose when I created it and

Re: RFR: 8212700: Change the mechanism by which JDK loads the platform-specific AWT Toolkit

2019-05-07 Thread Philip Race
On 5/7/19, 7:47 PM, Sergey Bylokhov wrote: Hi, Phil. Looks fine, but probably it will be worth to rename PlatformGraphicsInfo to PlatformInfo? I don't want (or intend) to do that. I already knew I was going to re-use this class for this purpose when I created it and adjusted the classname

Re: RFR: 8212700: Change the mechanism by which JDK loads the platform-specific AWT Toolkit

2019-05-07 Thread Sergey Bylokhov
Hi, Phil. Looks fine, but probably it will be worth to rename PlatformGraphicsInfo to PlatformInfo? On 06/05/2019 15:11, Phil Race wrote: Bug: https://bugs.openjdk.java.net/browse/JDK-8212700 CSR: https://bugs.openjdk.java.net/browse/JDK-8223417 Webrev:

Re: RFR: 8212700: Change the mechanism by which JDK loads the platform-specific AWT Toolkit

2019-05-07 Thread Roger Riggs
Hi Phil, The changes look fine. Thanks for the cleanup, Roger On 05/07/2019 12:01 AM, Philip Race wrote: On 5/6/19, 3:11 PM, Phil Race wrote: Bug: https://bugs.openjdk.java.net/browse/JDK-8212700 CSR: https://bugs.openjdk.java.net/browse/JDK-8223417 Webrev:

Re: RFR: 8212700: Change the mechanism by which JDK loads the platform-specific AWT Toolkit

2019-05-06 Thread Philip Race
On 5/6/19, 3:11 PM, Phil Race wrote: Bug: https://bugs.openjdk.java.net/browse/JDK-8212700 CSR: https://bugs.openjdk.java.net/browse/JDK-8223417 Webrev: https://cr.openjdk.java.net/~prr/8212700 Please review this bug + CSR to remove the awt.headless system property. That would be

RFR: 8212700: Change the mechanism by which JDK loads the platform-specific AWT Toolkit

2019-05-06 Thread Phil Race
Bug: https://bugs.openjdk.java.net/browse/JDK-8212700 CSR: https://bugs.openjdk.java.net/browse/JDK-8223417 Webrev: https://cr.openjdk.java.net/~prr/8212700 Please review this bug + CSR to remove the awt.headless system property. This should be the last of the fixes to remove use of system