Looks good to me.

Thanks,

Alexander.

On 10/06/2015 09:15 AM, Semyon Sadetsky wrote:
Sorry. I meant it is not guaranteed to be sent by every WM.

On 10/6/2015 9:03 AM, Semyon Sadetsky wrote:
In is extended event. In does not guaranteed to be sent by any WM.

On 10/5/2015 6:12 PM, Sergey Bylokhov wrote:
Why we cannot treat such a XA_NET_FRAME_EXTENTS as a ConfigureNotify in which only insets are changed, and the window bounds/insets should be revalidated?

On 05.10.15 14:56, Semyon Sadetsky wrote:
Hello,

Please review fix for JDK9:

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

The test scenario attached to the jira contains potential errors because Componet.getLocation() won't return the location at any moment of time.
Anyway a window location issue exists in Unity. The root cause is that
the real insets sent with the XA_NET_FRAME_EXTENTS event can be
overridden by the "guessed" insets which are zero on Unity. So the
returned location is increased by real insets while the real window
location is correct.
Yet another issue I found in Unity is a window size issue which is also
caused by the frame insets detection. The Unity WM doesn't provide the
frame insets immediately and XA_NET_FRAME_EXTENTS event may come after
the ConfigureNotify event for the frame.
The solution proposes an adaptation of the existing frame insets request
algorithm to the Unity WM so that makes it more stable.

--Semyon







Reply via email to