RFR: JDK-8353486 : Open source Swing Tests - Set 4 [v3]

Harshitha Onkar honkar at openjdk.org
Thu Apr 17 17:27:41 UTC 2025


On Thu, 17 Apr 2025 01:47:45 GMT, Alexander Zvegintsev <azvegint at openjdk.org> wrote:

>> Harshitha Onkar has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   review update
>
> test/jdk/javax/swing/JFileChooser/bug4943900.java line 43:
> 
>> 41:     private static final String INSTRUCTIONS = """
>> 42:         1. Click "Show FileChooser" button below, FileChooser dialog must
>> 43:            appear.
> 
> I do not see the "Show FileChooser" button, the file chooser showed automatically.
> 
> However, it is shown in an unobvious way (code-wise), via the `frame.add(fc);` call (and later the `setVisible(true)` call on the JFrame shows it).
> 
> Is this intentional?

Updated instructions.
I previously had "Show FileChooser" button on click of which jfc was shown but during the course of review got a suggestion to remove the extra button click and directly add it to the frame so that it doesn't cover the instruction UI.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/24456#discussion_r2049376985


More information about the client-libs-dev mailing list