On 10/4/2016 4:05 PM, Sergey Bylokhov wrote:

Hi, Semyon.
  - It seems that DisplayMode contain the data which were not upscaled to the 
units instead of pixels. It seems incorrect that it is necessary to convert 
them here to set the correct size for the window.
Not sure that it is true. Display mode is a device attribute. It should be the same as the one presented in the native system configuration.
  - Is it possible to eliminate check for Unity? I guess that if the size of 
mode and currentwindow are different then the window should be reshaped 
unconditionally is unity used or not.
It is not necessary for other Linux OSes because this transitional size updates happen only on Unity. Since this WM behavior is undocumented I prefer to have an accurately addressed fix.

----- [email protected] wrote:

Looks fine.

--
Thanks,
Alexander.

On 10/03/2016 07:38 PM, Semyon Sadetsky wrote:
The fix is updated
http://cr.openjdk.java.net/~ssadetsky/8141528/webrev.01/

The root case of the issue is that in Unity the screen window size
and
location are get updated several times after display mode is
changed,
and bounds which are set in line 459 of X11GraphicsDevice class are
overwritten. Since those window updates caused by display mode
change
are not predictable the solution is changed to set the correct
full-screen size in the ConfigureNotify handler.

--Semyon


On 12/7/2015 2:11 PM, Semyon Sadetsky wrote:
Hello,

Please review fix for JDK9:

bug: https://bugs.openjdk.java.net/browse/JDK-8141528
webrev: http://cr.openjdk.java.net/~ssadetsky/8141528/webrev.00/

Compiz WM requires override redirect for full-screen window to
preserve the full size upon display mode switching.

--Semyon


Reply via email to