You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@continuum.apache.org by ca...@apache.org on 2009/07/11 02:47:20 UTC
svn commit: r793158 -
/continuum/trunk/continuum-webapp-test/src/test/testng/org/apache/continuum/web/test/parent/AbstractSeleniumTest.java
Author: carlos
Date: Sat Jul 11 00:47:19 2009
New Revision: 793158
URL: http://svn.apache.org/viewvc?rev=793158&view=rev
Log:
Change the handling of thread local
Modified:
continuum/trunk/continuum-webapp-test/src/test/testng/org/apache/continuum/web/test/parent/AbstractSeleniumTest.java
Modified: continuum/trunk/continuum-webapp-test/src/test/testng/org/apache/continuum/web/test/parent/AbstractSeleniumTest.java
URL: http://svn.apache.org/viewvc/continuum/trunk/continuum-webapp-test/src/test/testng/org/apache/continuum/web/test/parent/AbstractSeleniumTest.java?rev=793158&r1=793157&r2=793158&view=diff
==============================================================================
--- continuum/trunk/continuum-webapp-test/src/test/testng/org/apache/continuum/web/test/parent/AbstractSeleniumTest.java (original)
+++ continuum/trunk/continuum-webapp-test/src/test/testng/org/apache/continuum/web/test/parent/AbstractSeleniumTest.java Sat Jul 11 00:47:19 2009
@@ -47,7 +47,7 @@
public static String maxWaitTimeInMs;
- private static ThreadLocal<Selenium> selenium;
+ private static ThreadLocal<Selenium> selenium = new ThreadLocal<Selenium>();
private static Properties p;
@@ -73,17 +73,10 @@
throws Exception
{
this.baseUrl = baseUrl;
+
if ( getSelenium() == null )
{
- final Selenium s = new DefaultSelenium( seleniumHost, seleniumPort, browser, baseUrl );
- selenium = new ThreadLocal<Selenium>()
- {
- @Override
- protected Selenium initialValue()
- {
- return s;
- }
- };
+ selenium.set( new DefaultSelenium( seleniumHost, seleniumPort, browser, baseUrl ) );
getSelenium().start();
}
}
@@ -132,6 +125,7 @@
if ( getSelenium() != null )
{
getSelenium().stop();
+ selenium.set( null );
}
}