Re: [9] Review request for 8159432: [PIT][macosx] StackOverflow in closed/java/awt/Dialog/DialogDeadlock/DialogDeadlockTest

2016-11-02 Thread 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.

Re: [9] Review request for 8159432: [PIT][macosx] StackOverflow in closed/java/awt/Dialog/DialogDeadlock/DialogDeadlockTest

2016-11-01 Thread Sergey Bylokhov
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:

Re: [9] Review request for 8159432: [PIT][macosx] StackOverflow in closed/java/awt/Dialog/DialogDeadlock/DialogDeadlockTest

2016-10-05 Thread Semyon Sadetsky
On 05.10.2016 03:04, Sergey Bylokhov wrote: On 24.08.16 22:18, Semyon Sadetsky wrote: The issue is connected to the restoring focus to the previously focused window when there are a lot of focus restore requests are coming very often (for example series of windows showing and hiding quickly).

Re: [9] Review request for 8159432: [PIT][macosx] StackOverflow in closed/java/awt/Dialog/DialogDeadlock/DialogDeadlockTest

2016-10-04 Thread Sergey Bylokhov
On 24.08.16 22:18, Semyon Sadetsky wrote: The issue is connected to the restoring focus to the previously focused window when there are a lot of focus restore requests are coming very often (for example series of windows showing and hiding quickly). In this case waiting for asynchronous focus

Re: [9] Review request for 8159432: [PIT][macosx] StackOverflow in closed/java/awt/Dialog/DialogDeadlock/DialogDeadlockTest

2016-10-03 Thread Alexander Zvegintsev
looks good to me. -- Thanks, Alexander. On 24.08.2016 22:18, Semyon Sadetsky wrote: Hello, Please review fix for JDK9: bug: https://bugs.openjdk.java.net/browse/JDK-8159432 webrev: http://cr.openjdk.java.net/~ssadetsky/8159432/webrev.00/ The issue is connected to the restoring focus to the