RFR: 8354646: java.awt.TextField allows to identify the spaces in a password when double clicked at the starting and end of the text
Jeremy Wood
duke at openjdk.org
Mon May 26 08:39:25 UTC 2025
Make sure AquaTextPasswordFieldUI can't use selectWordAction.
The core problem here was we could call selectWordAction in the Aqua LAF on a JPasswordField. This problem was already solved in the BasicPasswordFieldUI, but the AquaTextPasswordFieldUI (and SynthPasswordFieldUI) do NOT extend the BasicPasswordFieldUI, so they weren't inheriting this solution.
So the problem is partially about multiple inheritance.
My first response at solving the problem is just to move the existing solution to the parent BasicTextUI class and use an instanceof to make sure it is only applied to JPasswordFields.
There may be many different philosophies/recommendations on how to resolve this; I'm open to suggestions.
-------------
Commit messages:
- 8354646: Make sure AquaTextPasswordFieldUI can't use selectWordAction
- Merge branch 'master' of https://github.com/mickleness/jdk
- Merge branch 'master' of https://github.com/mickleness/jdk
- Merge branch 'master' of https://github.com/mickleness/jdk
- Merge branch 'master' of https://github.com/mickleness/jdk
- Merge branch 'master' of https://github.com/mickleness/jdk
- Merge branch 'openjdk:master' into master
- Merge branch 'openjdk:master' into master
- Merge branch 'openjdk:master' into master
- Merge pull request #5 from openjdk/master
- ... and 4 more: https://git.openjdk.org/jdk/compare/4a4209ff...ccf24d73
Changes: https://git.openjdk.org/jdk/pull/25443/files
Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=25443&range=00
Issue: https://bugs.openjdk.org/browse/JDK-8354646
Stats: 100 lines in 3 files changed: 81 ins; 19 del; 0 mod
Patch: https://git.openjdk.org/jdk/pull/25443.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/25443/head:pull/25443
PR: https://git.openjdk.org/jdk/pull/25443
More information about the client-libs-dev
mailing list