<AWT Dev> [9] Review request for 8159432: [PIT][macosx] StackOverflow in closed/java/awt/Dialog/DialogDeadlock/DialogDeadlockTest

[Alexander Zvegintsev]

+1

Thanks,
Alexander.

On 11/1/16 10:59 PM, Sergey Bylokhov wrote:
> Looks fine.
>
> On 05.10.16 11:45, Semyon Sadetsky wrote:
>>>> To avoid this the 8139218 solution is revisited to manage the focus
>>>> restore synchronously only if it is possible and send a single
>>>> asynchronous focus request otherwise.
>>>
>>> Can you please clarify what is the difference between:
>>> tempLost.requestFocusInWindow() which was updated in the fix, and
>>> toFocus.requestFocusInWindow() which was not updated. As far as I
>>> understand the second case(when the toFocus==restoreFocusTo) is a
>>> component to which we should send ROLLBACK.
>> If you look on the comment above the toFocus.requestFocusInWindow() line
>> you get the case when it is called.
>> Actually tempLost.requestFocusInWindow() is the right place to set
>> ROLLBACK. Focus is requested with ROLLBACK cause only when
>> tempLost==toFocus, but it is hard prove, so I'll better to add the
>> condition:
>>
>> http://cr.openjdk.java.net/~ssadetsky/8159432/webrev.01/
>
>
>
>