RFR: 8297976: Remove sun.net.ProgressMonitor and related classes [v2]

[Daniel Jeliński]

On Tue, 6 Dec 2022 12:43:44 GMT, Daniel Fuchs <dfuchs at openjdk.org> wrote:

>> test/jdk/sun/net/www/http/KeepAliveStream/KeepAliveStreamFinalizer.java line 175:
>> 
>>> 173:         public InputStream getInputStream() throws IOException {
>>> 174:             if (finalized) {
>>> 175:                 System.out.println(failureReason = "getInputStream called after finalize");
>> 
>> For ease of debugging, perhaps use `System.err.println` instead of `System.out`? That way the `Thread.dumpStack()` which is done on the next line will appear in the same jtreg section as this message.
>> 
>> Same comment in few other places where we have this construct.
>
> I agree with Jaikiran that since we're not using testng here using `System.err` consistently is probably better.

Good point. Replaced `System.out` with `System.err` everywhere.

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

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