<Swing Dev> Fwd: [9] Review request for 8165207: [macosx] Test javax/swing/Popup/TaskbarPositionTest.java fails on Mac 10.12

Alexander Popov ox38popugaev at yandex.ru
Mon Dec 26 14:01:40 UTC 2016


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" <sergey.bylokhov at 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.

--
Best regards,
Alexander.



More information about the swing-dev mailing list