[java] Making sure browser path is not null

Closes #12450
SeleniumHQ · Jul 31, 2023 · aa68cb1 · aa68cb1
1 parent e69feee
commit aa68cb1
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 3 deletions.
diff --git a/java/src/org/openqa/selenium/chrome/ChromeDriver.java b/java/src/org/openqa/selenium/chrome/ChromeDriver.java
@@ -98,7 +98,9 @@ private static ChromeDriverCommandExecutor generateExecutor(
  if (service.getExecutable() == null) {
  Result result = DriverFinder.getPath(service, options);
  service.setExecutable(result.getDriverPath());
- options.setBinary(result.getBrowserPath());
+ if (result.getBrowserPath() != null) {
+ options.setBinary(result.getBrowserPath());
+ }
  }
  return new ChromeDriverCommandExecutor(service, clientConfig);
  }

diff --git a/java/src/org/openqa/selenium/edge/EdgeDriver.java b/java/src/org/openqa/selenium/edge/EdgeDriver.java
@@ -70,7 +70,9 @@ private static EdgeDriverCommandExecutor generateExecutor(
  if (service.getExecutable() == null) {
  Result result = DriverFinder.getPath(service, options);
  service.setExecutable(result.getDriverPath());
- options.setBinary(result.getBrowserPath());
+ if (result.getBrowserPath() != null) {
+ options.setBinary(result.getBrowserPath());
+ }
  }
  return new EdgeDriverCommandExecutor(service, clientConfig);
  }

diff --git a/java/src/org/openqa/selenium/firefox/FirefoxDriver.java b/java/src/org/openqa/selenium/firefox/FirefoxDriver.java
@@ -140,7 +140,9 @@ private static FirefoxDriverCommandExecutor generateExecutor(
  if (service.getExecutable() == null) {
  Result result = DriverFinder.getPath(service, options);
  service.setExecutable(result.getDriverPath());
- options.setBinary(result.getBrowserPath());
+ if (result.getBrowserPath() != null) {
+ options.setBinary(result.getBrowserPath());
+ }
  }
  return new FirefoxDriverCommandExecutor(service, clientConfig);
  }