[java] Include shutdown hook again to delete binary when stored in tmp folder

Author: bonigarcia

Diff for: java/src/org/openqa/selenium/manager/SeleniumManager.java

@@ -71,13 +71,31 @@ public class SeleniumManager {
   private final String managerPath = System.getenv("SE_MANAGER_PATH");
   private Path binary = managerPath == null ? null : Paths.get(managerPath);
   private String seleniumManagerVersion;
+  private boolean binaryInTemporalFolder = false;
 
   /** Wrapper for the Selenium Manager binary. */
   private SeleniumManager() {
     BuildInfo info = new BuildInfo();
     String releaseLabel = info.getReleaseLabel();
     int lastDot = releaseLabel.lastIndexOf(".");
     seleniumManagerVersion = BETA_PREFIX + releaseLabel.substring(0, lastDot);
+    if (managerPath == null) {
+      Runtime.getRuntime()
+          .addShutdownHook(
+              new Thread(
+                  () -> {
+                    if (binaryInTemporalFolder && binary != null && Files.exists(binary)) {
+                      try {
+                        Files.delete(binary);
+                      } catch (IOException e) {
+                        LOG.warning(
+                            String.format(
+                                "%s deleting temporal file: %s",
+                                e.getClass().getSimpleName(), e.getMessage()));
+                      }
+                    }
+                  }));
+    }
   }
 
   public static SeleniumManager getInstance() {
@@ -298,6 +316,7 @@ private Path getBinaryInCache(String binaryName) throws IOException {
     Path cacheParent = Paths.get(cachePath);
     if (!Files.isWritable(cacheParent)) {
       cacheParent = Files.createTempDirectory(SELENIUM_MANAGER);
+      binaryInTemporalFolder = true;
     }
 
     return Paths.get(