Commit 19843c06 authored by Yihua Huang's avatar Yihua Huang

Merge pull request #57 from d0ngw/master

Fix chrome driver can't quit
parents 0e98183f 4fc89485
package us.codecraft.webmagic.downloader.selenium; package us.codecraft.webmagic.downloader.selenium;
import org.apache.log4j.Logger;
import org.openqa.selenium.WebDriver; import org.openqa.selenium.WebDriver;
import org.openqa.selenium.chrome.ChromeDriver; import org.openqa.selenium.chrome.ChromeDriver;
...@@ -16,6 +17,7 @@ import java.util.concurrent.atomic.AtomicInteger; ...@@ -16,6 +17,7 @@ import java.util.concurrent.atomic.AtomicInteger;
* Time: 下午1:41 <br> * Time: 下午1:41 <br>
*/ */
class WebDriverPool { class WebDriverPool {
private Logger logger = Logger.getLogger(getClass());
private final static int DEFAULT_CAPACITY = 5; private final static int DEFAULT_CAPACITY = 5;
...@@ -81,8 +83,8 @@ class WebDriverPool { ...@@ -81,8 +83,8 @@ class WebDriverPool {
throw new IllegalStateException("Already closed!"); throw new IllegalStateException("Already closed!");
} }
for (WebDriver webDriver : webDriverList) { for (WebDriver webDriver : webDriverList) {
webDriver.close(); logger.info("Quit webDriver" + webDriver);
webDriver.quit();
} }
} }
} }
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment