<AWT Dev> <Awt Dev> [9] Review Request for 8131670: [TEST_BUG]Test java/awt/EventQueue/6980209/bug6980209.java fails on Linux
Alexander Scherbatiy
alexandr.scherbatiy at oracle.com
Fri Sep 4 14:46:47 UTC 2015
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
>>>>
>>>>
>>>
>>
>
>
More information about the awt-dev
mailing list