This looks good. Thank you, Vikrant!
Shura > On Jul 13, 2017, at 2:50 AM, Vikrant Agarwal <[email protected]> > 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/~vagarwal/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
