[java] Fix spotbugs issues

Author: p0deje

java/src/org/openqa/selenium/chromium/ChromiumDriver.java

@@ -321,6 +321,10 @@ public void launchApp(String id) {
   @Override
   public Map<String, Object> executeCdpCommand(String commandName, Map<String, Object> parameters) {
     Require.nonNull("Command Name", commandName);
+    if (this.cdp == null) {
+      return java.util.Collections.emptyMap();
+    }
+
     return cdp.executeCdpCommand(commandName, parameters);
   }
 
@@ -358,36 +362,56 @@ public Optional<BiDi> maybeGetBiDi() {
 
   @Override
   public List<Map<String, String>> getCastSinks() {
+    if (this.casting == null) {
+      return java.util.Collections.emptyList();
+    }
+
     return casting.getCastSinks();
   }
 
   @Override
   public String getCastIssueMessage() {
+    if (this.casting == null) {
+      return "";
+    }
+
     return casting.getCastIssueMessage();
   }
 
   @Override
   public void selectCastSink(String deviceName) {
     Require.nonNull("Device Name", deviceName);
-    casting.selectCastSink(deviceName);
+
+    if (this.casting != null) {
+      casting.selectCastSink(deviceName);
+    }
   }
 
   @Override
   public void startDesktopMirroring(String deviceName) {
     Require.nonNull("Device Name", deviceName);
-    casting.startDesktopMirroring(deviceName);
+
+    if (this.casting != null) {
+      casting.startDesktopMirroring(deviceName);
+    }
   }
 
   @Override
   public void startTabMirroring(String deviceName) {
     Require.nonNull("Device Name", deviceName);
-    casting.startTabMirroring(deviceName);
+
+    if (this.casting != null) {
+      casting.startTabMirroring(deviceName);
+    }
   }
 
   @Override
   public void stopCasting(String deviceName) {
     Require.nonNull("Device Name", deviceName);
-    casting.stopCasting(deviceName);
+
+    if (this.casting != null) {
+      casting.stopCasting(deviceName);
+    }
   }
 
   @Override

java/src/org/openqa/selenium/firefox/ProfilesIni.java

@@ -136,10 +136,11 @@ protected File locateAppDataDirectory(Platform os) {
       appData =
           new File(
               MessageFormat.format(
-                  "{0}/Library/Application Support/Firefox", System.getenv("HOME")));
+                  "{0}/Library/Application Support/Firefox", System.getProperty("user.home")));
 
     } else {
-      appData = new File(MessageFormat.format("{0}/.mozilla/firefox", System.getenv("HOME")));
+      appData =
+          new File(MessageFormat.format("{0}/.mozilla/firefox", System.getProperty("user.home")));
     }
 
     if (!appData.exists()) {

java/src/org/openqa/selenium/remote/http/jdk/JdkHttpClient.java

@@ -523,17 +523,15 @@ public void close() {
       }
     }
 
-    if (this.client instanceof AutoCloseable) {
-      AutoCloseable closeable = (AutoCloseable) this.client;
-      executorService.submit(
-          () -> {
-            try {
-              closeable.close();
-            } catch (Exception e) {
-              LOG.log(Level.WARNING, "failed to close the http client: " + closeable, e);
-            }
-          });
-    }
+    AutoCloseable closeable = (AutoCloseable) this.client;
+    executorService.submit(
+        () -> {
+          try {
+            closeable.close();
+          } catch (Exception e) {
+            LOG.log(Level.WARNING, "failed to close the http client: " + closeable, e);
+          }
+        });
     this.client = null;
     executorService.shutdown();
   }

java/src/org/openqa/selenium/remote/tracing/opentelemetry/OpenTelemetryTracer.java

@@ -31,7 +31,7 @@
 public class OpenTelemetryTracer implements org.openqa.selenium.remote.tracing.Tracer {
 
   private static final Logger LOG = Logger.getLogger(OpenTelemetryTracer.class.getName());
-  private static boolean HTTP_LOGS;
+  private static volatile boolean HTTP_LOGS;
 
   // We obtain the underlying tracer instance from the singleton instance
   // that OpenTelemetry maintains. If we blindly grabbed the tracing provider