<Swing Dev> [12] RFR JDK-8213116:javax/swing/JComboBox/WindowsComboBoxSize/WindowsComboBoxSizeTest.java fails in Windows

Prasanta Sadhukhan prasanta.sadhukhan at oracle.com
Mon Nov 5 04:54:47 UTC 2018



On 04-Nov-18 12:02 AM, Philip Race wrote:
> What a sorry list of bugs.
Didn't quite understand this statement... are you saying the original 
combobox fix has spawned bad list of bugs?
> Can you
> 1) Add this evaluation to the bug report ?
> 2) Make sure - if you didn't already - that all related tests pass ?
> It probably isn't too many ..
>
FYI...I have run all JCombobox swing tests which has passed.

Regards
Prasanta
> -phil.
>
> On 11/1/18, 11:02 PM, Prasanta Sadhukhan wrote:
>> Hi Sergey,
>>
>>
>> On 01-Nov-18 12:25 AM, Sergey Bylokhov wrote:
>>> Hi, Prasanta.
>>> On 31/10/2018 11:40, Prasanta Sadhukhan wrote:
>>>> This is a regression ofJDK-8203281: 
>>>> <https://bugs.openjdk.java.net/browse/JDK-8203281>[Windows] 
>>>> JComboBox change in ui when editor.setBorder() is called where 
>>>> combobox arrow button height is reduced by 2
>>>
>>> It is reduced if the EmptyBorder is in use. Should the current fix 
>>> use the similar check?
>>>
>> We can use the similar check in the fix. Updated webrev
>> http://cr.openjdk.java.net/~psadhukhan/8213116/webrev.1/
>>
>> Regards
>> Prasanta
>>>> [http://hg.openjdk.java.net/jdk/client/file/20e47d686221/src/java.desktop/windows/classes/com/sun/java/swing/plaf/windows/WindowsComboBoxUI.java#l381] 
>>>>
>>>> but the minimum size of combobox is set to old values which was 
>>>> added during<https://bugs.openjdk.java.net/browse/JDK-8179027>
>>>> JDK-8179027: 
>>>> <https://bugs.openjdk.java.net/browse/JDK-8179027>JComboBox too 
>>>> small under Windows LAF
>>>> which was done to address regression caused by
>>>> JDK-6490753: <https://bugs.openjdk.java.net/browse/JDK-6490753> 
>>>> Vista:JComboBox doesn't looks as native combobox in different 
>>>> states of component.
>>>>
>>>> Now, JDK-8203281 <https://bugs.openjdk.java.net/browse/JDK-8203281> 
>>>> was also a regression caused by JDK-6490753 
>>>> <https://bugs.openjdk.java.net/browse/JDK-6490753> so the combobox 
>>>> size needs to be altered in the fix done for JDK-8179027 
>>>> <https://bugs.openjdk.java.net/browse/JDK-8179027> to make sure the 
>>>> values are compatible.
>>>>
>>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8213116
>>>> webrev: http://cr.openjdk.java.net/~psadhukhan/8213116/webrev.0/
>>>>
>>>> Regards
>>>> Prasanta
>>>
>>>
>>



More information about the swing-dev mailing list