inspector: removing checking of non existent field in lib/inspector.js

Keroosha · targos · commit 512ab1fddfd5 · 2019-05-31T08:46:53.000+02:00
Seems like sessionAttached doesn't exist and no more assigned anywhere PR-URL: #27919 Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Eugene Ostroukhov <eostroukhov@google.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
diff --git a/lib/inspector.js b/lib/inspector.js
@@ -37,12 +37,8 @@ class Session extends EventEmitter {
 connect() {
 if (this[connectionSymbol])
 throw new ERR_INSPECTOR_ALREADY_CONNECTED('The inspector session');
- const connection =
+ this[connectionSymbol] =
 new Connection((message) => this[onMessageSymbol](message));
- if (connection.sessionAttached) {
- throw new ERR_INSPECTOR_ALREADY_CONNECTED('Another inspector session');
- }
- this[connectionSymbol] = connection;
 }
 
 [onMessageSymbol](message) {

-Original file line number
+Diff line change
 connect(){
 if(this[connectionSymbol])
 thrownewERR_INSPECTOR_ALREADY_CONNECTED('The inspector session');
 -constconnection=
 +this[connectionSymbol]=
 newConnection((message)=>this[onMessageSymbol](message));
 -if(connection.sessionAttached){
 -thrownewERR_INSPECTOR_ALREADY_CONNECTED('Another inspector session');
 -}
 -this[connectionSymbol]=connection;
+}
 [onMessageSymbol](message){