<AWT Dev> [9] Review request for 8051857: OperationTimedOut exception inside from XToolkit.syncNativeQueue call

[Sergey Bylokhov]

Hi, Alexander.
The fix looks good.

On 23.09.2014 17:59, Alexander Zvegintsev wrote:
> Hi Sergey,
>
> you are right,
> please see updated webrev here:
> http://cr.openjdk.java.net/~azvegint/jdk/9/8051857/01/
>
> Thanks,
>
> Alexander.
>
> On 09/23/2014 04:56 PM, Sergey Bylokhov wrote:
>> Hi, Alexander.
>> Why we need xawt_root_window in XlibWrapper? Since it is cached 
>> already in get_xawt_root_shell().
>>
>> On 23.09.2014 15:48, Alexander Zvegintsev wrote:
>>> Hello,
>>>
>>> please review the fix
>>> http://cr.openjdk.java.net/~azvegint/jdk/9/8051857/00/
>>> for the issue
>>> https://bugs.openjdk.java.net/browse/JDK-8051857
>>>
>>> We have XNextSecondaryLoopEvent() which blocks all events processing 
>>> except of this specific types:
>>> PropertyNotify, SelectionClear, SelectionNotify.
>>>
>>> Before the JDK-7033533 [1] fix, syncNativeQueue() used a 
>>> SelectionNotify, but now we are using ConfigureNotify.
>>> This leads to a situation when syncNativeQueue() called on 
>>> EventQueue thread waiting for ConfigureNotify from XAWT thread,
>>> and DataTransferer.convertData() called on XAWT thread waiting event 
>>> processing on the EventQueue thread.
>>> Here it is our OperationTimedOut exception.
>>>
>>> So the fix is simply to allow to process this ConfigureNotify event.
>>>
>>> JDK-7033533 and this fix should be backported to 8u40 to support 
>>> realSync() on OEL7.
>>>
>>> [1] https://bugs.openjdk.java.net/browse/JDK-7033533 realSync() 
>>> doesn't work with Xfce
>>>
>>
>>
>


-- 
Best regards, Sergey.