<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