baseUrl should be prepended after exploding variables. Fixes #4540

Author: barancev

java/client/src/com/thoughtworks/selenium/webdriven/commands/Open.java

@@ -39,9 +39,7 @@ public Open(String baseUrl) {
   @Override
   protected Void handleSeleneseCommand(final WebDriver driver, String url, String ignored) {
     try {
-      final String urlToOpen = url.indexOf("://") == -1 ?
-          new URL(baseUrl, url).toString() :
-          url;
+      final String urlToOpen = url.contains("://") ? url : new URL(baseUrl, url).toString();
 
       driver.get(urlToOpen);
     } catch (MalformedURLException e) {

java/client/src/com/thoughtworks/selenium/webdriven/commands/OpenWindow.java

@@ -42,9 +42,7 @@ public OpenWindow(String baseUrl, GetEval opener) {
   protected Void handleSeleneseCommand(final WebDriver driver, final String url,
       final String windowID) {
     try {
-      final String urlToOpen = url.indexOf("://") == -1 ?
-          new URL(baseUrl, url).toString() :
-          url;
+      final String urlToOpen = url.contains("://") ? url : new URL(baseUrl, url).toString();
 
       String[] args = {String.format("window.open('%s', '%s'); null;", urlToOpen, windowID)};
 

java/server/src/org/openqa/selenium/server/htmlrunner/CoreTestCase.java

@@ -134,15 +134,6 @@ private List<LoggableStep> findCommands(WebDriver driver, URL baseUrl) {
         throw new SeleniumException("Unknown command: " + step.get(0));
       }
       String value = step.get(1);
-      if (baseUrl != null && "open".equals(step.get(0)) || "openWindow".equals(step.get(0))) {
-        try {
-          value = value.indexOf("://") == -1 ?
-                  new URL(baseUrl, value).toString() :
-                  value;
-        } catch (MalformedURLException e) {
-          throw new SeleniumException("Unable to determine URL to open: " + value);
-        }
-      }
       steps.add(new LoggableStep(
         STEP_FACTORY.get(step.get(0)).create(value, step.get(2)),
         step.get(0),