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

Konstantin Shefov konstantin.shefov at oracle.com
Fri Nov 9 13:26:43 UTC 2012


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
>>>>>>>>>>
>>>>>>>
>>>>>
>>>
>
>



More information about the swing-dev mailing list