<Swing Dev> [8] Review request for 7124209 [macosx] SpringLayout issue. BASELINE is not in the range: [NORTH, SOUTH]

Alexander Scherbatiy alexandr.scherbatiy at oracle.com
Thu Nov 8 12:02:31 UTC 2012


On 11/7/2012 7:00 PM, Konstantin Shefov wrote:
> Please, look at modified fix:
> http://cr.openjdk.java.net/~kshefov/7124209/webrev.01/ 
> <http://cr.openjdk.java.net/%7Ekshefov/7124209/webrev.01/>

    Try/catch block in the main method catches RuntimeException as well. 
So the jtreg passes the test even it really fails.
    It is also a good idea to fail the test if the 
SwingUtilities.invokeAndWait() throws an exception because it is also a 
wrong situation.

    Thanks,
    Alexandr.

>
> On 01-Nov-12 17:45, Alexander Scherbatiy wrote:
>>
>>   - Please, remove comments and unnecessary System.out from the fix
>>   - Create and check swing components on EDT. Avoiding this rule can 
>> leads to unpredictable test failures.
>>   - Swing tests usually have bugBugID.java or some meaningful name.
>>   - It is better to use  the swing-dev at openjdk.java.net alias to 
>> review the swing tests.
>>
>>   Thanks,
>>   Alexandr.
>>
>> On 10/31/2012 7:06 PM, Konstantin Shefov wrote:
>>> Hello,
>>>
>>> Please review a fix for the issue:
>>>
>>> 7124209 [macosx] SpringLayout issue. BASELINE is not in the range: 
>>> [NORTH, SOUTH]
>>>
>>> The webrev is http://cr.openjdk.java.net/~kshefov/7124209/webrev/
>>>
>>> It is suggested to move the test to open jdk.
>>>
>>> Thanks,
>>> Konstantin
>>>
>>




More information about the swing-dev mailing list