RFR: 8354646: java.awt.TextField allows to identify the spaces in a password when double clicked at the starting and end of the text [v9]

[Jeremy Wood]

On Fri, 6 Jun 2025 07:12:52 GMT, Prasanta Sadhukhan <psadhukhan at openjdk.org> wrote:

>> 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..

Hmm. Sorry; I feel like I'm still not quite understanding the ask. Is this what you're asking for? https://github.com/openjdk/jdk/pull/25443/commits/39d19f2772cd980f0ee8b812de1b748de858ef56

IMO this feels weird:
A. We'd also need to update the instructions in that test, yes?
B. If we want that test to cover 8354646, then we can delete PasswordSelectionWordTest from this PR, right?

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

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