[jbosstools-commits] JBoss Tools SVN: r43069 - trunk/tests/plugins/org.jboss.tools.ui.bot.ext/src/org/jboss/tools/ui/bot/ext/condition.

[jbosstools-commits at lists.jboss.org]

Author: rhopp
Date: 2012-08-16 05:00:22 -0400 (Thu, 16 Aug 2012)
New Revision: 43069

Modified:
   trunk/tests/plugins/org.jboss.tools.ui.bot.ext/src/org/jboss/tools/ui/bot/ext/condition/BrowserIsLoaded.java
Log:
Fixed Documentation section test on Windows slaves.

Modified: trunk/tests/plugins/org.jboss.tools.ui.bot.ext/src/org/jboss/tools/ui/bot/ext/condition/BrowserIsLoaded.java
===================================================================
--- trunk/tests/plugins/org.jboss.tools.ui.bot.ext/src/org/jboss/tools/ui/bot/ext/condition/BrowserIsLoaded.java	2012-08-16 08:31:14 UTC (rev 43068)
+++ trunk/tests/plugins/org.jboss.tools.ui.bot.ext/src/org/jboss/tools/ui/bot/ext/condition/BrowserIsLoaded.java	2012-08-16 09:00:22 UTC (rev 43069)
@@ -20,10 +20,15 @@
 
 	SWTBot bot;
 	private SWTBotBrowserExt browser;
+	private boolean loadedAlreadyOnStart = false; 
 	
 	public BrowserIsLoaded(SWTBotBrowserExt browser){
-		browser.addProgressListenerToBrowser();
-		this.browser = browser;
+		if (!browser.getText().isEmpty()){
+			loadedAlreadyOnStart = true; //no need for listener
+		}else{
+			browser.addProgressListenerToBrowser();
+			this.browser = browser;
+		}
 	}
 	
 	@Override
@@ -35,7 +40,7 @@
 		}catch(WidgetNotFoundException wnfe){
 		}
 		
-		return browser.isPageLoaded();
+		return loadedAlreadyOnStart ? true : browser.isPageLoaded();
 	}
 
 	@Override