On Mon, 9 Jun 2025 06:18:46 GMT, Jeremy Wood <d...@openjdk.org> wrote:

>> test/jdk/javax/swing/plaf/basic/BasicPasswordFieldUI/PasswordSelectionWordTest.java
>>  line 45:
>> 
>>> 43: public class PasswordSelectionWordTest {
>>> 44:     public static void main(String[] args) throws Exception {
>>> 45:         for (UIManager.LookAndFeelInfo laf :
>> 
>> Now since the test is for all the look and feel, should it be inside 
>> `test/jdk/javax/swing/JPasswordField` folder?
>> Other than this it looks good to me.
>
> Okay; this is moved.

I don't agree. The test belongs in `BasicPasswordFieldUI`, or rather under 
`BasicTextUI` where the main change is made.

The fix is in UI class, not in the `JPasswordField`, moreover the subject of 
the issue refers to an AWT component `java.awt.TextField`.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/25443#discussion_r2136478249

Reply via email to