Review request for 8029378: com/sun/jdi/BadHandshakeTest.java failed with java.util.concurrent.TimeoutException

[shanliang]

David Holmes wrote:
> On 16/01/2014 5:37 PM, shanliang wrote:
>> Hi,
>>
>> Please review this simple fix, the test needs  more time to wait
>> Phaser.awaitAdvanceInterruptibly(...).
>
> Integer.MAX_VALUE? There's no point using a timed form at all.
>
> David
Yes Phaser.awaitAdvanceInterruptibly(int phase) could be used here, but 
the call of this method is wrapped in:
    jdk.testlibrary.ProcessTools.startProcess(...)

So we have to add a new method ProcessTools.startProcess(...) which has 
no timeout parameter. I did not do this because I thought to have a 
simple fix only within the test.

If this is useful, here is the new web:
http://cr.openjdk.java.net/~sjiang/JDK-8029378/01/

Thanks,
Shanliang

>> webrev: http://cr.openjdk.java.net/~sjiang/JDK-8029378/00/
>> bug: https://bugs.openjdk.java.net/browse/JDK-8029378
>>
>> Thanks,
>> Shanliang