Are those changes came from jemmy 2 repo?
How about the README file in the jemmy/src folder that states the next:
------
This src folder contains a copy of Jemmy 2 library sources from
https://jemmy.java.net/.
Do NOT modify files in it.
------
--Semyon
On 07/13/2017 03:03 AM, Prasanta Sadhukhan wrote:
looks good to me.
Regards
Prasanta
On 7/13/2017 3:20 PM, Vikrant Agarwal wrote:
Hi All,
Please review the following:
Bug : JDK-8184272 <https://bugs.openjdk.java.net/browse/JDK-8184272>
JDK 10 Webrev:
http://cr.openjdk.java.net/~vagarwal/8184272/webrev.00/
<http://cr.openjdk.java.net/%7Evagarwal/8184272/webrev.00/>
Summary:
Jemmy JFileChooserOperator.enterSubDir does not navigate to sub
directory and instead goes to user's default directory
The present code for JFileChooserOperator.enterSubDir(..) first
selects the directory, then tries to navigate to the selected
directory by getting the path of selected directory from
getSelectedFile().
JFileChooserOperator.getSelectedFile() calls
JFileChooser.getSelectedFile(), which returns the path for the
selected file, but if the selection is a directory, then it returns
Null if the fileSelectionMode is not set to select directory.
This when passed to setCurrentDirectory() sets the diectory as user's
default directory.
Doing setFileSelectionMode(JFileChooser.FILES_AND_DIRECTORIES)
ensures that the getSelectedFile() returns the correct path even in
case it is a directory.
Best Regards,
Vikrant
