package nl.hsac.fitnesse.junit.selenium;

import nl.hsac.fitnesse.fixture.util.SeleniumHelper;
import org.apache.commons.lang3.StringUtils;

/**
 * Base class for Selenium driver factory factories.
 */
public abstract class SeleniumDriverFactoryFactoryBase implements SeleniumDriverFactoryFactory {
    public final static String SELENIUM_GRID_URL = "seleniumGridUrl";
    public final static String SELENIUM_BROWSER = "seleniumBrowser";
    public final static String SELENIUM_DRIVER_CLASS = "seleniumDriverClass";
    public final static String SELENIUM_CAPABILITIES = "seleniumCapabilities";

    @Override
    public abstract boolean willOverride();

    @Override
    public abstract SeleniumHelper.DriverFactory getDriverFactory();

    protected boolean isPropertySet(String propertyName) {
        String value = getProperty(propertyName);
        return !StringUtils.isEmpty(value);
    }

    protected String getProperty(String propertyName) {
        String value = System.getProperty(propertyName);
        if (value != null && value.startsWith("\"") && value.endsWith("\"")) {
            value = value.substring(1, value.length() - 1);
        }
        return value;
    }
}

@@ -897,24 +897,21 @@
 %7D%0A%0A    p
-rotected
+ublic
  String 

Update to public because the SeleniumDriverSetup needs the getProperty method. I think this method should be moved somewhere else, so the SeleniumDriverSetup is not depending on a junit class.