inspector: check if connected before waiting

Fixes: #10093
PR-URL: #10094
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>

Author: Eugene Ostroukhov

src/inspector_agent.cc

@@ -515,10 +515,12 @@ bool AgentImpl::IsStarted() {
 }
 
 voidAgentImpl::WaitForDisconnect() {
- shutting_down_ = true;
-fprintf(stderr, "Waiting for the debugger to disconnect...\n");
-fflush(stderr);
- inspector_->runMessageLoopOnPause(0);
+if (state_ == State::kConnected) {
+ shutting_down_ = true;
+fprintf(stderr, "Waiting for the debugger to disconnect...\n");
+fflush(stderr);
+ inspector_->runMessageLoopOnPause(0);
+ }
 }
 
 #defineREADONLY_PROPERTY(obj, str, var) \

test/inspector/test-inspector-stops-no-file.js

@@ -0,0 +1,15 @@
+'use strict';
+require('../common');
+constspawn=require('child_process').spawn;
+
+constchild=spawn(process.execPath,
+['--inspect','no-such-script.js'],
+{'stdio': 'inherit'});
+
+functionsignalHandler(value){
+child.kill();
+process.exit(1);
+}
+
+process.on('SIGINT',signalHandler);
+process.on('SIGTERM',signalHandler);