RFR 8162876: [TEST_BUG] sun/net/www/protocol/http/HttpInputStream.java fails intermittently

[Svetlana Nikandrova]

Pavel, Chris,

thank you for you replays. I believe I addressed all Pavel's comments 
except the one mentioned by Chris below. I saw tests with a really long 
bug list in @bug so I think it's a good practice to avoid testbug 
numbers. Hope it is not critical.
I also switched back from Writer to OutputStream to reproduce original 
test scenario.

http://cr.openjdk.java.net/~snikandrova/8162876/webrev.01/ 
<http://cr.openjdk.java.net/%7Esnikandrova/8162876/webrev.01/>

Thank you,
Svetlana

On 01.08.2016 22:31, Chris Hegarty wrote:
>> On 1 Aug 2016, at 12:03, Pavel Rappo <pavel.rappo at oracle.com> wrote:
>> ...    return len;
>>
>> 5. I believe we should add 8162876 to @bug
> Strictly speaking, we should not. The @bug should capture the bug numbers
> that the test exercises the changes for. It is not necessary to include the bug
> number for a bug to the test itself.
>
>>   25  * @bug 4937598
>>
>> Otherwise I find this version to be more robust and readable.
> +1
>
> -Chris.