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

[Avik Niyogi]

Hi Alexander,
The following is my input for this webrev:

In case the LAF is Aqua, the check for combobox alignment is skipped. 
Instead, if default LAF is Aqua, the LAF should be changed to cross platform LAF to check if it works on other LAF or not.
Either run this in a loop with checks for all LAF versions available on OS, or check for default LAF at the beginning and change it to Cross-Platform LAF to check for alignment.
The current changes makes the test pass but does seem to give false positives by process of elimination for Aqua LAF.

Also, found the following minor issues which could be looked into if possible:
variables done and error are unused.
numData, dayData and mnDayData could be declared as final.
@override annotations are missing for methods used in ComboPopupCheckListener, PopupHandler.
panel is redeclared within scope instead of reusing available variable in method createContentPane().
wildcard * is used in import statements which according to guidelines should be replace to use only the classes required.

With Regards,
Avik Niyogi
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/swing-dev/attachments/20170111/9819d87d/attachment.html>