On Fri, 18 Apr 2025 18:04:54 GMT, Alexey Ivanov <aiva...@openjdk.org> wrote:
>> Harshitha Onkar has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> remove redudant frame
>
> test/jdk/javax/swing/JFileChooser/bug4943900.java line 65:
>
>> 63: } catch (Exception e) {
>> 64: throw new SkippedException("LaF not supported", e);
>> 65: }
>
> Is the intention to run in system L&F only rather than in all L&Fs?
The original bug was on GTK+linux only. I have extended it to
SystemLookAndFeel. Since there are slight variations on how the file filter
works on different LaF and the test instructions would be confusing if we were
to account for all variations of installed LaF. Hence limited it System LaF.
For example on macOS (Aqua LaF) when the txt filter is applied non-text files
are greyed out.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/24456#discussion_r2053117061
