Looks fine.
On 25/04/2019 13:31, Philip Race wrote:
Sorry, meant to include core-libs on this !
-phil.
On 4/25/19, 1:12 PM, Phil Race wrote:
Bug: https://bugs.openjdk.java.net/browse/JDK-8130266
CSR: https://bugs.openjdk.java.net/browse/JDK-
Webrev: http://cr.openjdk.java.net/~prr/8130266/
Please review this fix + the CSR which eliminates the java.awt.graphicenv
System property.
To replace it we have platform-specific classes which can then more easily embed
platform-specific logic. For example, logic in java.awt.GraphicsEnvironment was
then moved down into this class. Also isInAquaSession used on Mac was moved
into that class from the Mac Toolkit class.
A reference to the property in J2D Bench - just used to log / document the
testing environment
was removed along with similar references to two other properties removed in
earlier fixes.
This has been built + tested on Mac, Linux + Windows in headful mode and also
specifying headless via -Djava.awt.headless and by letting it default to
headless
in an ssh session on Mac + by unsetting the DISPLAY variable on Linux.
-phil.
--
Best regards, Sergey.
