On Fri, 6 Jun 2025 06:39:22 GMT, Jeremy Wood <d...@openjdk.org> wrote:
>> test/jdk/javax/swing/plaf/basic/BasicPasswordFieldUI/PasswordSelectionWordTest.java >> line 27: >> >>> 25: * @test >>> 26: * @key headful >>> 27: * @bug 4231444 8354646 >> >> This bugid needs to be added to >> java/awt/TextField/SetEchoCharWordOpsTest.java also as that is the original >> test which raises this issue. > > I'm confused by this ask. Are you asking me to update > SetEchoCharWordOpsTest.java to include this line: > > ` * @bug 6191897 4231444 8354646` > > I understand topically these tickets are all related, but I ran > SetEchoCharWordOpsTest in Aqua using master and this branch: I didn't > perceive any difference. > > (For context: the git history doesn't really show me if any changes > accompanied SetEchoCharWordOpsTest , but I think it passes because of code > like this: `"PasswordField.focusInputMap", > aquaKeyBindings.getPasswordFieldInputMap()` ) I am asking to add "this bugid" 8354646 in SetEchoCharWordOpsTest.java... I see if we double click at beginning and/or at the end it will identify "One" and "Three" separately which is not desirable and it will fail the test checkpoint "if you are able to move the caret word-by-word press Fail" as caret can be placed in between those words before the fix.. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/25443#discussion_r2131645722