RFR: 8255124: KeepAliveStreamCleaner may crash with java.lang.IllegalMonit…

[Alan Bateman]

On Wed, 21 Oct 2020 13:12:03 GMT, Jan Lahoda <jlahoda at openjdk.org> wrote:

> …orStateException: current thread is not owner
> 
> The KeepAliveStreamCleaner is calling Condition.wait (i.e. the Object.wait variant), should presumably be calling Condition.await. (NetBeans is producing warnings on the line as well.)

Marked as reviewed by alanb (Reviewer).

test/jdk/sun/net/www/http/KeepAliveCache/B5045306.java line 143:

> 141:         }
> 142:         if (!uncaught.isEmpty()) {
> 143:             throw new AssertionError("Unhandled exception:", uncaught.get(0));

The existing test throws RuntimeException when it fails, maybe this case should do the same.

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

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