On Tue, 11 Jul 2023 10:50:13 GMT, Tejesh R <[email protected]> wrote:

> The bug mentions about transparency issue on Linux which actually got 
> resolved with [JDK-8006421](https://bugs.openjdk.org/browse/JDK-8006421) fix. 
> Now there is another problem related to the bug, which is screen selection 
> going wrong during checking for new screen when the test is run. The problem 
> is that (As seen in the pictures attached in bug) the transparency is lost 
> for windows which are in screen 0 (default screen) too, which is not supposed 
> to happen where windows on screen 1 should have lost there transparency. The 
> main reason being the call to `toGlobal()` when window bounds are passed to 
> `checkIfOnNewScreen` method. I didn't get the actual reason for `toGlobal` 
> being used here (actually not required to check monitor number), but it 
> actually doubled the X position of the window [here]( 
> https://github.com/openjdk/jdk/blob/4b1403d06b99b91ddd89ad6e54669b0595f1f8e5/src/java.desktop/unix/classes/sun/awt/X11/XBaseWindow.java#L776).
>  Removing `toGlobal` solve the issue and neithe
 r didn't cause any regression (existing test + the reason `toGlobal()` was 
added [JDK-8143295](https://bugs.openjdk.org/browse/JDK-8143295)).
> 
> The automated test fails if GC is changed for windows within screen 0 
> (default screen). CI testing is green.

This pull request has now been integrated.

Changeset: 0a3c6d6b
Author:    Tejesh R <[email protected]>
URL:       
https://git.openjdk.org/jdk/commit/0a3c6d6bd010231d02e92016037149e85fb1db3f
Stats:     151 lines in 2 files changed: 148 ins; 0 del; 3 mod

8280482: Window transparency bug on Linux

Reviewed-by: dnguyen, azvegint

-------------

PR: https://git.openjdk.org/jdk/pull/14825

Reply via email to