<Swing Dev> [9] Review request for 8031696 [macosx] TwentyThousandTest test failed with OOM

[Petr Pchelko]

Hello, Alexander.

I bet you do not need to remove the “filter != null” part, because Object.equals(null, null) is true and the code will be executed.

With best regards. Petr.

> On Nov 21, 2014, at 7:01 PM, Alexander Scherbatiy <alexandr.scherbatiy at oracle.com> wrote:
> 
> 
> Hello,
> 
> Could you review the fix:
>  bug: https://bugs.openjdk.java.net/browse/JDK-8031696
>  webrev: http://cr.openjdk.java.net/~alexsch/8031696/webrev.00
> 
>  Setting a new file filter in JFileChooser on Mac OS X starts LoadFilesThread from AquaFileSystemModel.
>  After the fix JDK-8022398 the file filter changed event is fired even the new file filter equals to the old one.
>  The fix does not fire the file filter changed event in case the file filter has not been changed.
> 
> Thanks,
> Alexandr.
>