<Swing Dev> [8] Review request for 7147408 [macosx] Add autodelay to fix a regression test

Sergey Bylokhov Sergey.Bylokhov at oracle.com
Fri Nov 9 18:48:28 UTC 2012


Hi, Konstantin.
Fix looks good.

09.11.2012 17:26, Konstantin Shefov wrote:
> http://cr.openjdk.java.net/~kshefov/7147408/webrev.03/
>
> On 09-Nov-12 16:30, Sergey Bylokhov wrote:
>> 09.11.2012 12:59, Konstantin Shefov wrote:
>>> Could you please look at this test fix?
>>> http://cr.openjdk.java.net/~kshefov/7147408/webrev.02/
>> Where finished and timer is used?
>>
>>   44     private Timer timer;
>>   46     private boolean finished = false;
>>
>> passed used in destroy method w/o synchronization.
>>>
>>> On 09-Nov-12 12:46, Alexander Scherbatiy wrote:
>>>> On 11/8/2012 5:20 PM, Konstantin Shefov wrote:
>>>>> http://cr.openjdk.java.net/~kshefov/7147408/webrev.02/ 
>>>>> <http://cr.openjdk.java.net/%7Ekshefov/7147408/webrev.02/>
>>>>
>>>>     The fix looks good for me.
>>>>
>>>>    Thanks,
>>>>    Alexandr.
>>>>
>>>>>
>>>>> On 08-Nov-12 16:44, Alexander Scherbatiy wrote:
>>>>>> On 11/7/2012 6:59 PM, Konstantin Shefov wrote:
>>>>>>> I made modifications you suggested.
>>>>>>> http://cr.openjdk.java.net/~kshefov/7147408/webrev.01/ 
>>>>>>> <http://cr.openjdk.java.net/%7Ekshefov/7147408/webrev.01/>
>>>>>>
>>>>>>     Could you also fix the issue with the catched RuntimeException?
>>>>>>     All others looks good for me.
>>>>>>
>>>>>>     Thanks,
>>>>>>     Alexandr.
>>>>>>
>>>>>>>
>>>>>>> Konstantin
>>>>>>>
>>>>>>> On 01-Nov-12 18:04, Alexander Scherbatiy wrote:
>>>>>>>>
>>>>>>>> - The test needs to create and use Swing components only on EDT.
>>>>>>>>    The same is for JApplet (see 
>>>>>>>> http://docs.oracle.com/javase/tutorial/uiswing/components/applet.html). 
>>>>>>>>
>>>>>>>>    In the same way the following variable initialization is not 
>>>>>>>> correct because the JEditorPane is not created on the EDT:
>>>>>>>>     private JEditorPane jep = new JEditorPane();
>>>>>>>>
>>>>>>>> - The test fails with the compilation error: cannot find symbol 
>>>>>>>> Util.blockTillDisplayed(jep);
>>>>>>>>   Use the toolkit realSync()  method instead:
>>>>>>>>   SunToolkit toolkit = (SunToolkit) Toolkit.getDefaultToolkit();
>>>>>>>>   ...
>>>>>>>>   toolkit.realSync();
>>>>>>>>
>>>>>>>>   Thanks,
>>>>>>>>   Alexandr.
>>>>>>>>
>>>>>>>>
>>>>>>>> On 10/31/2012 1:58 PM, Konstantin Shefov wrote:
>>>>>>>>> On 30-Oct-12 16:43, Anthony Petrov wrote:
>>>>>>>>>> Looks fine to me.
>>>>>>>>>>
>>>>>>>>>> -- 
>>>>>>>>>> best regards,
>>>>>>>>>> Anthony
>>>>>>>>>>
>>>>>>>>>> On 10/30/2012 3:51 PM, Konstantin Shefov wrote:
>>>>>>>>>>> Hello,
>>>>>>>>>>>
>>>>>>>>>>> Please review a fix for the issue:
>>>>>>>>>>>
>>>>>>>>>>> 7147408 [macosx] Add autodelay to fix a regression test
>>>>>>>>>>>
>>>>>>>>>>> The webrev is 
>>>>>>>>>>> http://cr.openjdk.java.net/~kshefov/7147408/webrev/
>>>>>>>>>>>
>>>>>>>>>>> It is suggested to move the test to open jdk.
>>>>>>>>>>>
>>>>>>>>>>> Thanks,
>>>>>>>>>>> Konstantin
>>>>>>>>>>>
>>>>>>>>
>>>>>>
>>>>
>>
>>


-- 
Best regards, Sergey.




More information about the swing-dev mailing list