fix(browser): don't add already active socket again on reconnect

[j0tunn]

Sometimes one socket can send few register messages, and every time this socket is being added to browser's active sockets as new. And on disconnect we get crash with error message like "TypeError: Cannot read property '1234567' of null".

In logs I can see messages like:

DEBUG [karma]: A browser has connected on socket __5YycZnaOXfrHk8kQFP
INFO [IE 8.0.0 (Windows 7)]: Connected on socket __5YycZnaOXfrHk8kQFP with id 35636946
DEBUG [launcher]: internet explorer via Remote WebDriver (id 35636946) captured in 3.71 secs
DEBUG [IE 8.0.0 (Windows 7)]: New connection __5YycZnaOXfrHk8kQFP (already have __5YycZnaOXfrHk8kQFP)
DEBUG [IE 8.0.0 (Windows 7)]: New connection __5YycZnaOXfrHk8kQFP (already have __5YycZnaOXfrHk8kQFP, __5YycZnaOXfrHk8kQFP)

This is the cause of problem described here: karma-runner/karma-junit-reporter#21

[j0tunn]

@vojtajina Take a look please

[maksimr]

@j0tunn Thanks!

Add a test for this behavior.

[j0tunn]

@maksimr Done

[j0tunn]

@maksimr ping

[maksimr]

#merge

[karmarunnerbot]

Sent to presubmit as presubmit-canary-pr-1248

[j0tunn]

thanks!