<AWT Dev> [13] Review Request: 8222789 Replace the usage of "Sun's Xinerama API" by the "XFree86's Xinerama API" on Solaris

[Phil Race]

Looks OK.

-phil.

On 4/21/19 11:48 PM, Sergey Bylokhov wrote:
> Hello.
> Please review the fix for JDK 13.
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8222789
> Fix: http://cr.openjdk.java.net/~serb/8222789/webrev.00
>
> Currently, AWT on Solaris uses "Sun's Xinerama API" which is 
> non-standard API implemented in the libXext, we use it from 2001, but 
> the standard version of "Xinerama API" was added Solaris in nv_62:
> https://blogs.oracle.com/solaris/x11-apis-for-querying-multi-head-screen-layout-v2 
>
>
> I have checked that libXinerama.so.1 library exists on Solaris 11 
> Sparc and x64.
>
> I was able to prove that Java2D reports the correct number of the 
> screens(the same as before the fix):
>  - in Solaris Sparc when the X redirected to the dual-head Linux
>  - in Solaris x64 in a VirtualBox when two screens are emulated
>
> After the fix we will use the same code for Linux/Solaris/AIX.
>