<Swing Dev> Review request for 7124218: Space should select cell in the JTable

[Alexander Scherbatiy]

   - May be isMac() method can only check L&F because a L&F which 
contains Mac description usually implies that it is L&F designed for Mac 
OS X
   - isMac(osName, lookAndFeel) check is used to do different actions 
but the only essential difference is in the provided key code.
      it can be changed to keyTap(isMac(...) ? KeyEvent.VK_ENTER : 
KeyEvent.SPACE)
   - The exception handling in the tryLookAndFeel() method is skipped.
     What should be the test behavior in case if it is not possible to 
set a L&F?
   - runTestCase() method should access JTable methods on EDT.

  Thanks,
  Alexandr.

On 11/3/2015 12:20 PM, Avik Niyogi wrote:
> Hi All,
>
> Kindly review the fix for JDK9.
> *Bug*: https://bugs.openjdk.java.net/browse/JDK-7124218
>
> *Webrev*: http://cr.openjdk.java.net/~rchamyal/avik/7124218/webrev.00/ 
> <http://cr.openjdk.java.net/%7Erchamyal/avik/7124218/webrev.00/>
>
> *Issue*: "Space should select cell in the JTable" is being expected 
> for Look and Feels (and OS)
> where key-bindings are different.
>
> *Cause*: Keybindings expected in the test case are not taking native 
> key-bindings into account.
>
> *Fix*: Recreated test case with appropriate cases for all required OS 
> to account
> for supported native, non-native and cross-platform Look and Feels.
>
> With Regards,
> Avik Niyogi