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

[Alexander Zvegintsev]

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
>>
>
>