Re: [9] Review request for 8152677 [macosx] All files filter can't be selected in JFileChooser

2016-04-19 Thread Alexander Scherbatiy
Hello, Could you review the updated fix: http://cr.openjdk.java.net/~alexsch/8152677/webrev.01 - file chooser file filter is used as an initial value for the old file filter in the AquaFileChooserUI.FilterComboBoxModel - AcceptAllFileFilter class and filed are made static The

Re: [9] Review request for 8152677 [macosx] All files filter can't be selected in JFileChooser

2016-04-15 Thread Sergey Bylokhov
On 15.04.16 19:22, Alexander Scherbatiy wrote: On 15/04/16 18:27, Sergey Bylokhov wrote: When a JFileChooser is created it always adds Accept All file filter in the constructor because useAcceptAllFileFilter filed is true by default. Because of this the AquaFileChooserUI.FilterComboBoxModel

Re: [9] Review request for 8152677 [macosx] All files filter can't be selected in JFileChooser

2016-04-15 Thread Alexander Scherbatiy
On 15/04/16 18:27, Sergey Bylokhov wrote: Is it possible that default selection of FileChooser will be different from "acceptAllFileFilter"? When a JFileChooser is created it always adds Accept All file filter in the constructor because useAcceptAllFileFilter filed is true by default.

Re: [9] Review request for 8152677 [macosx] All files filter can't be selected in JFileChooser

2016-04-15 Thread Sergey Bylokhov
Is it possible that default selection of FileChooser will be different from "acceptAllFileFilter"? On 15.04.16 16:46, Alexander Scherbatiy wrote: Hello, Could you review the fix: bug: https://bugs.openjdk.java.net/browse/JDK-8152677 webrev:

[9] Review request for 8152677 [macosx] All files filter can't be selected in JFileChooser

2016-04-15 Thread Alexander Scherbatiy
Hello, Could you review the fix: bug: https://bugs.openjdk.java.net/browse/JDK-8152677 webrev: http://cr.openjdk.java.net/~alexsch/8152677/webrev.00 FilterComboBoxModel from AquaFileChooserUI overrides getSelectedItem() method to always return the selected filter from the file chooser.