> 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 
> and SynthPasswordFieldUI, but the AquaTextPasswordFieldUI does NOT extend the 
> BasicPasswordFieldUI, so it wasn'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.

Jeremy Wood has updated the pull request incrementally with one additional 
commit since the last revision:

  Update 
test/jdk/javax/swing/plaf/basic/BasicPasswordFieldUI/PasswordSelectionWordTest.java
  
  Co-authored-by: Alexey Ivanov <alexey.iva...@oracle.com>

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/25443/files
  - new: https://git.openjdk.org/jdk/pull/25443/files/660028bc..0c0b417a

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=25443&range=07
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=25443&range=06-07

  Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 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

Reply via email to