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

[Alexander Scherbatiy]

   The fix looks good to me.

   Thanks,
   Alexandr.

On 9/23/2014 7:44 PM, Sergey Bylokhov wrote:
> 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
>>>>
>>>
>>>
>>
>
>