Merge pull request #562 from mousetraps/i466

#466 Debugger does not start reliably running Node.js console app

Author: mousetraps

Nodejs/Product/Nodejs/Debugger/Communication/DebuggerConnection.cs

@@ -122,7 +122,7 @@ public void Connect(Uri uri) {
             lock (_networkClientLock) {
                 int connection_attempts = 0;
                 const int MAX_ATTEMPTS = 5;
-                while (!_isClosed) {
+                while (true) {
                     connection_attempts++;
                     try {
                         // TODO: This currently results in a call to the synchronous TcpClient
@@ -147,7 +147,7 @@ public void Connect(Uri uri) {
                             throw;
                         }
                         LiveLogger.WriteLine("Connection attempt {0} failed with: {1}", connection_attempts, ex);
-                        if (connection_attempts >= MAX_ATTEMPTS) {
+                        if (connection_attempts >= MAX_ATTEMPTS && !_isClosed) {
                             throw;
                         }
                         else {