<Swing Dev> [8] Review request for 7124209 [macosx] SpringLayout issue. BASELINE is not in the range: [NORTH, SOUTH]
Konstantin Shefov
konstantin.shefov at oracle.com
Thu Nov 22 11:31:35 UTC 2012
Please review a fix for this issue:
http://cr.openjdk.java.net/~kshefov/7124209/webrev.02
On 09-Nov-12 12:47, Alexander Scherbatiy wrote:
> On 11/8/2012 4:25 PM, Konstantin Shefov wrote:
>> http://cr.openjdk.java.net/~kshefov/7124209/webrev.02/
>> <http://cr.openjdk.java.net/%7Ekshefov/7124209/webrev.02/>
>
> The fix looks good for me.
>
> Thanks,
> Alexandr.
>
>>
>> On 08-Nov-12 16:02, Alexander Scherbatiy wrote:
>>> 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