do not specify chromeServiceBuilder

wechaty · Oct 24, 2016 · 09099fc · 09099fc
1 parent beadd7d
commit 09099fc
Showing 1 changed file with 8 additions and 2 deletions.
diff --git a/src/puppet-web/browser.ts b/src/puppet-web/browser.ts
@@ -223,13 +223,18 @@ export class Browser extends EventEmitter {
     /**
      * http://stackoverflow.com/a/27733960/1123955
      * issue #56
-     */
+     * only need under win32 with cygwin
+     * and will cause strange error:
+     * `The previously configured ChromeDriver service is still running. You must shut it down before you may adjust its configuration.`
+
     const chrome = require('selenium-webdriver/chrome')
     const path = require('chromedriver').path
 
     const service = new chrome.ServiceBuilder(path).build()
     chrome.setDefaultService(service)
 
+     */
+
     const options = {
       args: ['--no-sandbox']  // issue #26 for run inside docker
       // , binary: require('chromedriver').path
@@ -696,7 +701,8 @@ export class Browser extends EventEmitter {
   }
 
   public loadSession(): Promise<any> {
-    log.verbose('PuppetWebBrowser', `loadSession(${this.setting.sessionFile})`)
+    log.verbose('PuppetWebBrowser', 'loadSession() from %s', this.setting.sessionFile ? this.setting.sessionFile : '' )
+
     if (!this.setting.sessionFile) {
       return Promise.reject(new Error('loadSession() no sessionFile'))
     } else if (this.dead()) {