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"))
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" <[email protected]>:
> 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.
--
Best regards,
Alexander.
