RFR CODETOOLS-7902339: Handle different look and feels in JFileChooserOperator

[Alexandre (Shura) Iline]

Looks good.

Do you want me to push it for you?

Shura

> On Oct 29, 2018, at 1:49 AM, Muneer Kolarkunnu <abdul.kolarkunnu at oracle.com> wrote:
> 
> Hi Shura,
> 
> Thanks for review, please see inline comments, updated webrev: http://cr.openjdk.java.net/~akolarkunnu/7902339/webrev.01/
> 
> Regards,
> Muneer
> 
> -----Original Message-----
> From: Alexandre (Shura) Iline 
> Sent: Thursday, October 25, 2018 11:48 PM
> To: Muneer Kolarkunnu <abdul.kolarkunnu at oracle.com>
> Cc: jemmy-dev at openjdk.java.net
> Subject: Re: RFR CODETOOLS-7902339: Handle different look and feels in JFileChooserOperator
> 
> The JFileChooserOperator.java changes look great.
> 
> 1. You can use @BeforeMethod even with data provider, it seems. There a seem to be a way to pass parameters to @BeforeMethod. Have you tried that?
>>>> Done it, it really reduced lots of code duplication, done similar changes in JInternalFrameOperatorTest.java and JInternalFrameOperatorCloseTest.java tests also.
> 2. I am worried about hardcoding “Desktop” even in the tests. That would mean that Jemmy tests only works on English windows. Should we think of localization resource bundles for this case?
>>>> Changed this to use API FileSystemView.getFileSystemView().getHomeDirectory().
> 3. Please bump minor version
>>>> Done
> 
> Shura
> 
>> On Oct 23, 2018, at 11:18 PM, Muneer Kolarkunnu <abdul.kolarkunnu at oracle.com> wrote:
>> 
>> Hi All,
>> 
>> Please review fix for the enhancement: https://bugs.openjdk.java.net/browse/CODETOOLS-7902339 
>> 
>> Webrev : http://cr.openjdk.java.net/~akolarkunnu/7902339/webrev.00/ 
>> 
>> Description:
>> There are multiple issues in different L&Fs.  
>> In GTK and Motif L&F:  
>> There are two JLists, one is to list folders and second one one is to list files, but current JFileChooserOperator implementation didn't consider this, it expects only one JList that is to list files. 
>> 
>> Windows & Windows Classic L&F: 
>> There is no 'Go Home' button, but there is a toggle button to go desktop. In Windows platform 'Go Home' button usually(In metal and nimbus L&F) navigates to Desktop only.
>> 
>> Regards,
>> Muneer
>> 
>> 
>