You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by rf...@apache.org on 2013/06/20 21:52:07 UTC
svn commit: r1495151 -
/maven/sandbox/trunk/dist-tools/dist-tool-plugin/src/main/java/org/apache/maven/dist/tools/DistCheckSiteMojo.java
Author: rfscholte
Date: Thu Jun 20 19:52:07 2013
New Revision: 1495151
URL: http://svn.apache.org/r1495151
Log:
Be sure driver is closed
Modified:
maven/sandbox/trunk/dist-tools/dist-tool-plugin/src/main/java/org/apache/maven/dist/tools/DistCheckSiteMojo.java
Modified: maven/sandbox/trunk/dist-tools/dist-tool-plugin/src/main/java/org/apache/maven/dist/tools/DistCheckSiteMojo.java
URL: http://svn.apache.org/viewvc/maven/sandbox/trunk/dist-tools/dist-tool-plugin/src/main/java/org/apache/maven/dist/tools/DistCheckSiteMojo.java?rev=1495151&r1=1495150&r2=1495151&view=diff
==============================================================================
--- maven/sandbox/trunk/dist-tools/dist-tool-plugin/src/main/java/org/apache/maven/dist/tools/DistCheckSiteMojo.java (original)
+++ maven/sandbox/trunk/dist-tools/dist-tool-plugin/src/main/java/org/apache/maven/dist/tools/DistCheckSiteMojo.java Thu Jun 20 19:52:07 2013
@@ -470,17 +470,22 @@ public class DistCheckSiteMojo extends A
@Override
public void execute() throws MojoExecutionException
{
- //resolve only to what we set
- if ( screenShot )
+ try
{
- // create driver once reduce time to complete mojo
- driver = new FirefoxDriver();
+ //resolve only to what we set
+ if ( screenShot )
+ {
+ // create driver once reduce time to complete mojo
+ driver = new FirefoxDriver();
+ }
+ super.execute();
}
-
- super.execute();
- if ( screenShot )
+ finally
{
- driver.close();
+ if ( screenShot )
+ {
+ driver.close();
+ }
}
}
}