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

Reply via email to