Use socket isConnected() method rather than isBound()

[olivierayache]

Description

It is possible that the socket is already bound and not connected. In this case the connection to the server will never happened.

Related Issue

Fixes #962

Motivation and Context

I found this bug in Android context. In this context, I used setSocketFactory method on WebSocketClient with a socket factory which automatically bind created sockets to a defined network interface. In this case WebSocketClient will never connect to the server.

How Has This Been Tested?

A non regression test has been written in tests package (Issue962Test).
The test fails before the correction was applied.

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

• My code follows the code style of this project.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have added tests to cover my changes.
• All new and existing tests passed.

[marci4]

Hello @olivierayache,
thx for your pull request.

Best regards,
Marcel