Merge pull request #502 from GoogleChrome/fixforheadless

Fallback to reuse existing tab
GoogleChrome · Jul 14, 2016 · 16199bd · 16199bd
2 parents 4a66309 + 27d3ed8
commit 16199bd
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/lighthouse-core/driver/drivers/cri.js b/lighthouse-core/driver/drivers/cri.js
@@ -38,7 +38,7 @@ class CriDriver extends Driver {
       /* eslint-disable new-cap */
       chromeRemoteInterface.New((err, tab) => {
         if (err) {
-          return reject(err);
+          log.warn('CRI driver', 'cannot create new tab, will reuse tab.', err);
         }
 
         chromeRemoteInterface({port: port, chooseTab: tab}, chrome => {
@@ -57,6 +57,7 @@ class CriDriver extends Driver {
   disconnect() {
     return new Promise((resolve, reject) => {
       if (!this._tab) {
+        this._chrome.close();
         return resolve();
       }