Fix WebSocket error handling

Author: cyrus-and

lib/chrome.js

@@ -253,7 +253,7 @@ class Chrome extends EventEmitter {
         // make sure to complete all the unresolved callbacks
         const err = new Error('WebSocket connection closed');
         for (const callback of Object.values(this._callbacks)) {
-            callback(true, err);
+            callback(err);
         }
         this._callbacks = {};
     }

test/send.js

@@ -81,15 +81,17 @@ describe('sending a command', () => {
     });
     it('should catch WebSocket errors after command send', (done) => {
         Chrome((chrome) => {
-            let result;
-            chrome.Page.enable((err, res)=>{
-                result = res;
-            });
-            chrome.close(()=>{
-                assert(result instanceof Error);
-                assert(result.message === 'WebSocket connection closed');
+            chrome.Runtime.evaluate({
+                expression: 'new Promise(_ => _)',
+                awaitPromise: true
+            }, (error, response) => {
+                assert(error instanceof Error);
+                assert(!error.request);
+                assert(!error.response);
+                assert(!response);
                 done();
             });
+            chrome.close();
         });
     });
     describe('without a callback', () => {