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