See new diffs here pls - http://cr.openjdk.java.net/~avstepan/8165207/webrev.02/
Sergey,
failure could be reproduced on any OS X - just comment the following lines in my fix:
if (!UIManager.getLookAndFeel()
.getName().toLowerCase().contains("os x"))
if (!UIManager.getLookAndFeel()
.getName().toLowerCase().contains("os x"))
old test does not work at all - sometimes second menu does not called by its mnemonic and then no single combobox is expanded - so test PASSes when it must fail. Now test is adopted to os x LaF.
19.12.2016, 21:58, "Sergey Bylokhov" <sergey.bylok...@oracle.com>:
> Hi, Alex.
> Can you please clarify on what versions of OS X and JDK the bug can be reproduced? Thanks.
>
>> Hello,
>> review my changes please.
>>
>> Fix for https://bugs.openjdk.java.net/browse/JDK-8165207:
>> Webrev:http://cr.openjdk.java.net/~avstepan/8165207/webrev.01/
>> Issue: https://bugs.openjdk.java.net/browse/JDK-8165207
>>
>> This is a test bug. On Aqua LaF JCombobox popup could appear both upper label and arrow and lower.
>> So - check for its coordinates does not make sense. Also fixed Ctrl-down shortcut. It is invalid - it is special shortcut for OS X. Down key is enough to expand combobox.
>> Tested on Mac OS X with different LaFs - Aqua, Nimbus, Metal. Works perfectly after fix.
>> I used jdk 9b144 for testing.
>>
>> --
>> Best regards,
>> Alexander.
--
С уважением, Александр.
