The fix looks good to me.
Just format the code on line 79 before the push.
Thanks,
Alexandr.
On 9/3/2015 6:29 PM, Sergey Bylokhov wrote:
Looks fine.
On 03.09.15 17:57, Semyon Sadetsky wrote:
Guys, just a reminder.
On 8/4/2015 3:34 PM, Semyon Sadetsky wrote:
Hi Sergey,
It was not me. webrev diff to the last repository version.
the updated fix:
http://cr.openjdk.java.net/~ssadetsky/8131670/webrev.01/
--Semyon
http://cr.openjdk.java.net/~ssadetsky/8131670/webrev.01/
On 7/22/2015 12:26 AM, Sergey Bylokhov wrote:
Hi, Semyon.
Why you adds back the private logging to the test(it was removed
recently)? Why Logger cannot be used? I suggest to set
setAutoDelay+setAutoWaitForIdle in the main method also.
On 17.07.15 16:37, Semyon Sadetsky wrote:
Hello,
Please review fix for JDK9:
bug: https://bugs.openjdk.java.net/browse/JDK-8131670
webrev: http://cr.openjdk.java.net/~ssadetsky/8131670/webrev.00/
1ms auto-delay was set to the robot to improve stability on Linux
platforms.
Also additional focus request procedure added to improve focus gain
on OEL7.
--Semyon
