<Swing Dev> [9] Review request for 8078614 WindowsClassicLookAndFeel : MetalComboBoxUI.getbaseLine fails with IllegalArgumentException

Sergey Bylokhov Sergey.Bylokhov at oracle.com
Tue Apr 28 14:27:10 UTC 2015


Hi, Victor.
A few notes:
  - import of com.sun.java.swing.plaf.windows.WindowsComboBoxUI is 
unnecessary and should be removed.
  - The test should use invokeAndWait, otherwise it can successfully 
complete, when the main method exits before exception will be generated 
on EDT.

On 27.04.15 18:44, Victor D'yakov wrote:
> Hello,
>
> Please review the fix for JDK9:
> bug: https://bugs.openjdk.java.net/browse/JDK-8078614
> webrev link: 
> http://cr.openjdk.java.net/~alexsch/victor-dyakov/8078614/webrev.00/
> Root Cause:
>     The case where the height can be less than insets top+bottom is 
> not checked.
> Solution:
>     Return 0 for case where the height is less than insets top+bottom.
> Testing:
>     Run test/javax/swing/JComboBox/6632953/bug6632953.java automated 
> test.
>
> -- 
> Victor


-- 
Best regards, Sergey.




More information about the swing-dev mailing list