RFR: 8341982: Simplify JButton/bug4323121.java [v3]

Alexey Ivanov aivanov at openjdk.org
Fri Oct 18 11:44:19 UTC 2024


> The test `javax/swing/JButton/bug4323121.java` contains lots of unused methods.
> 
> I removed all the unused methods by extending `MouseAdapter`.
> 
> I use `CountDownLatch` to synchronise actions in the test.
> 
> The test still verifies `button.getModel().isArmed()` doesn't always return `true` for classes which extend `JButton`. I verified the updated test fails in 1.3.0 and passes in 1.4.0, so the test still reproduces the original problem.

Alexey Ivanov has updated the pull request incrementally with two additional commits since the last revision:

 - Amend indentation in the finally block
 - Inline mouseEntered listener for the button

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/21475/files
  - new: https://git.openjdk.org/jdk/pull/21475/files/128d7bef..f32b1f57

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=21475&range=02
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=21475&range=01-02

  Stats: 18 lines in 1 file changed: 8 ins; 7 del; 3 mod
  Patch: https://git.openjdk.org/jdk/pull/21475.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/21475/head:pull/21475

PR: https://git.openjdk.org/jdk/pull/21475


More information about the client-libs-dev mailing list