RFR: 8213531: Test javax/swing/border/TestTitledBorderLeak.java fails [v2]

Prasanta Sadhukhan psadhukhan at openjdk.java.net
Thu Apr 28 13:34:31 UTC 2022


> Test was failing in linux citing `java.lang.RuntimeException: Expected Total TitledBorder to be freed : 10 Freed 9 `
> As per the fix done in JDK-8204963 http://hg.openjdk.java.net/jdk/jdk/rev/cd7d2f9154fd
> there was no platform specific code done for the fix and logs in `TitledBorder.installPropertyChangeListeners` shows it was called 10 times for the test but `CleanerFactory.cleaner().register` action was only called 9 times in linux causing it to fail.
> 
> Modified the test to not show the frame which cause the problem to go away and also it can still be used as 8204963 regression test as it still fails without the fix and pass with it. Modified test has passed in all platforms for several iterations.
> 
> Also removed the deprecated `System.runFinalization`

Prasanta Sadhukhan has updated the pull request incrementally with one additional commit since the last revision:

  Test fix

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

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/8450/files
  - new: https://git.openjdk.java.net/jdk/pull/8450/files/ea2c3e11..8bf50cca

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=8450&range=01
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=8450&range=00-01

  Stats: 1 line in 1 file changed: 1 ins; 0 del; 0 mod
  Patch: https://git.openjdk.java.net/jdk/pull/8450.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/8450/head:pull/8450

PR: https://git.openjdk.java.net/jdk/pull/8450



More information about the client-libs-dev mailing list