<AWT 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 1 06:45:51 PDT 2012


   - 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 awt-dev mailing list