[OpenJDK 2D-Dev] [9] Review Request 8155753: Removing a monitor in the OS dispaly configuration causes assertion fails under Windows

[Vadim Pakhnushev]

Hi Semyon,

Generally seems reasonable, it seems that you should use screen instead 
of gdiScreen in the JNU_CallStaticMethodByName, otherwise the code won't 
compile.
Not sure how the rest of the code handles monitor removal, seems to me 
that there are no usages of this notifications anywhere, so maybe we 
don't need this code at all?

Thanks,
Vadim

On 12.09.2016 17:24, Semyon Sadetsky wrote:
> Hello,
>
> Please review fix for JDK9:
>
> bug: https://bugs.openjdk.java.net/browse/JDK-8155753
>
> webrev: http://cr.openjdk.java.net/~ssadetsky/8155753/webrev.00/
>
> The issue take place on Windows platform if Direct3d is on. The 
> notification routine about the monitor removal tries to get screen 
> number using the monitor handle which is obviously null at this 
> moment. As a fix the screen number is recorded in D3D context for 
> further possible notifications.
>
> --Semyon
>