specify nullability in package org.openqa.selenium.remote

* Mark everything as non-nullable by default, and
* Mark only the needed methods/parameters as nullable.

Partially implements #14291

Author: asolntsev

java/src/org/openqa/selenium/Platform.java

@@ -416,7 +416,7 @@ public static Platform getCurrent() {
    * @param osName the operating system name to determine the platform of
    * @return the most likely platform based on given operating system name
    */
-  public static Platform extractFromSysProperty(String osName) {
+  public static Platform extractFromSysProperty(@Nullable String osName) {
     return extractFromSysProperty(osName, System.getProperty("os.version", ""));
   }
 
@@ -429,7 +429,8 @@ public static Platform extractFromSysProperty(String osName) {
    * @param osVersion the operating system version to determine the platform of
    * @return the most likely platform based on given operating system name and version
    */
-  public static Platform extractFromSysProperty(String osName, String osVersion) {
+  public static Platform extractFromSysProperty(
+      @Nullable String osName, @Nullable String osVersion) {
     osName = requireNonNullElse(osName, "");
     osVersion = requireNonNullElse(osVersion, "");
     osName = osName.toLowerCase(Locale.ENGLISH);
@@ -516,6 +517,7 @@ public String[] getPartOfOsName() {
    * @return true if platforms are approximately similar, false otherwise
    */
   public boolean is(Platform compareWith) {
+    Platform family = this.family();
     return
     // Any platform is itself
     this == compareWith
@@ -524,7 +526,7 @@ public boolean is(Platform compareWith) {
         compareWith == ANY
         ||
         // And any Platform which is not a platform type belongs to the same family
-        (this.family() != null && this.family().is(compareWith));
+        (family != null && family.is(compareWith));
   }
 
   /**

java/src/org/openqa/selenium/chrome/ChromeDriverService.java

@@ -104,7 +104,7 @@ public class ChromeDriverService extends DriverService {
   public ChromeDriverService(
       @Nullable File executable,
       int port,
-      @Nullable Duration timeout,
+      Duration timeout,
       @Nullable List<String> args,
       @Nullable Map<String, String> environment)
       throws IOException {
@@ -328,7 +328,7 @@ protected List<String> createArgs() {
     protected ChromeDriverService createDriverService(
         @Nullable File exe,
         int port,
-        @Nullable Duration timeout,
+        Duration timeout,
         @Nullable List<String> args,
         @Nullable Map<String, String> environment) {
       try {

java/src/org/openqa/selenium/devtools/Connection.java

@@ -47,6 +47,7 @@
 import java.util.function.Consumer;
 import java.util.logging.Level;
 import java.util.logging.Logger;
+import org.jspecify.annotations.NullMarked;
 import org.jspecify.annotations.Nullable;
 import org.openqa.selenium.WebDriverException;
 import org.openqa.selenium.devtools.idealized.target.model.SessionID;
@@ -242,6 +243,7 @@ public void close() {
     this.isClosed.set(true);
   }
 
+  @NullMarked
   private class Listener implements WebSocket.Listener {
 
     @Override

java/src/org/openqa/selenium/edge/EdgeDriverService.java

@@ -100,7 +100,7 @@ public class EdgeDriverService extends DriverService {
   public EdgeDriverService(
       @Nullable File executable,
       int port,
-      @Nullable Duration timeout,
+      Duration timeout,
       @Nullable List<String> args,
       @Nullable Map<String, String> environment)
       throws IOException {
@@ -335,7 +335,7 @@ protected List<String> createArgs() {
     protected EdgeDriverService createDriverService(
         @Nullable File exe,
         int port,
-        @Nullable Duration timeout,
+        Duration timeout,
         @Nullable List<String> args,
         @Nullable Map<String, String> environment) {
       try {

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

@@ -38,7 +38,7 @@ public abstract class FirefoxDriverService extends DriverService {
   protected FirefoxDriverService(
       @Nullable File executable,
       int port,
-      @Nullable Duration timeout,
+      Duration timeout,
       @Nullable List<String> args,
       @Nullable Map<String, String> environment)
       throws IOException {

java/src/org/openqa/selenium/grid/web/ReverseProxyHandler.java

@@ -67,11 +67,10 @@ public HttpResponse execute(HttpRequest req) throws UncheckedIOException {
         toUpstream.setAttribute(attributeName, req.getAttribute(attributeName));
       }
 
-      for (String name : req.getQueryParameterNames()) {
-        for (String value : req.getQueryParameters(name)) {
-          toUpstream.addQueryParameter(name, value);
-        }
-      }
+      req.forEachQueryParameter(
+          (name, value) -> {
+            toUpstream.addQueryParameter(name, value);
+          });
 
       req.forEachHeader(
           (name, value) -> {

java/src/org/openqa/selenium/ie/InternetExplorerDriverService.java

@@ -79,7 +79,7 @@ public class InternetExplorerDriverService extends DriverService {
   public InternetExplorerDriverService(
       @Nullable File executable,
       int port,
-      @Nullable Duration timeout,
+      Duration timeout,
       @Nullable List<String> args,
       @Nullable Map<String, String> environment)
       throws IOException {
@@ -240,7 +240,7 @@ protected List<String> createArgs() {
     protected InternetExplorerDriverService createDriverService(
         @Nullable File exe,
         int port,
-        @Nullable Duration timeout,
+        Duration timeout,
         @Nullable List<String> args,
         @Nullable Map<String, String> environment) {
       try {

java/src/org/openqa/selenium/json/Json.java

@@ -26,6 +26,7 @@
 import java.lang.reflect.Type;
 import java.util.List;
 import java.util.Map;
+import org.jspecify.annotations.Nullable;
 import org.openqa.selenium.internal.Require;
 
 /**
@@ -116,7 +117,7 @@ public class Json {
    * @param toConvert the object to be serialized
    * @return JSON string representing the specified object
    */
-  public String toJson(Object toConvert) {
+  public String toJson(@Nullable Object toConvert) {
     return toJson(toConvert, JsonOutput.MAX_DEPTH);
   }
 
@@ -128,7 +129,7 @@ public String toJson(Object toConvert) {
    * @return JSON string representing the specified object
    * @throws JsonException if an I/O exception is encountered
    */
-  public String toJson(Object toConvert, int maxDepth) {
+  public String toJson(@Nullable Object toConvert, int maxDepth) {
     try (Writer writer = new StringWriter();
         JsonOutput jsonOutput = newOutput(writer)) {
       jsonOutput.write(toConvert, maxDepth);

java/src/org/openqa/selenium/remote/AbstractDriverOptions.java

@@ -17,6 +17,8 @@
 
 package org.openqa.selenium.remote;
 
+import static java.util.Collections.unmodifiableMap;
+import static java.util.Objects.requireNonNull;
 import static org.openqa.selenium.remote.CapabilityType.ACCEPT_INSECURE_CERTS;
 import static org.openqa.selenium.remote.CapabilityType.BROWSER_VERSION;
 import static org.openqa.selenium.remote.CapabilityType.ENABLE_DOWNLOADS;
@@ -34,7 +36,6 @@
 import java.util.Set;
 import java.util.TreeMap;
 import java.util.TreeSet;
-import org.jspecify.annotations.NonNull;
 import org.jspecify.annotations.Nullable;
 import org.openqa.selenium.MutableCapabilities;
 import org.openqa.selenium.PageLoadStrategy;
@@ -58,23 +59,23 @@ public DO setImplicitWaitTimeout(Duration timeout) {
     Map<String, Number> timeouts = getTimeouts();
     timeouts.put("implicit", timeout.toMillis());
 
-    setCapability(TIMEOUTS, Collections.unmodifiableMap(timeouts));
+    setCapability(TIMEOUTS, unmodifiableMap(timeouts));
     return self();
   }
 
   public DO setPageLoadTimeout(Duration timeout) {
     Map<String, Number> timeouts = getTimeouts();
     timeouts.put("pageLoad", timeout.toMillis());
 
-    setCapability(TIMEOUTS, Collections.unmodifiableMap(timeouts));
+    setCapability(TIMEOUTS, unmodifiableMap(timeouts));
     return self();
   }
 
   public DO setScriptTimeout(Duration timeout) {
     Map<String, Number> timeouts = getTimeouts();
     timeouts.put("script", timeout.toMillis());
 
-    setCapability(TIMEOUTS, Collections.unmodifiableMap(timeouts));
+    setCapability(TIMEOUTS, unmodifiableMap(timeouts));
     return self();
   }
 
@@ -109,7 +110,6 @@ public DO setEnableDownloads(boolean enableDownloads) {
     return self();
   }
 
-  @NonNull
   @SuppressWarnings("unchecked")
   private DO self() {
     return (DO) this;
@@ -125,7 +125,7 @@ public Set<String> getCapabilityNames() {
   protected abstract Set<String> getExtraCapabilityNames();
 
   @Override
-  public Object getCapability(String capabilityName) {
+  public @Nullable Object getCapability(String capabilityName) {
     Require.nonNull("Capability name", capabilityName);
 
     if (getExtraCapabilityNames().contains(capabilityName)) {
@@ -140,8 +140,9 @@ public Object getCapability(String capabilityName) {
   @Override
   public Map<String, Object> asMap() {
     Map<String, Object> toReturn = new TreeMap<>(super.asMap());
-    getExtraCapabilityNames().forEach(name -> toReturn.put(name, getCapability(name)));
-    return Collections.unmodifiableMap(toReturn);
+    getExtraCapabilityNames()
+        .forEach(name -> toReturn.put(name, requireNonNull(getCapability(name))));
+    return unmodifiableMap(toReturn);
   }
 
   private Map<String, Number> getTimeouts() {

java/src/org/openqa/selenium/remote/AddHasAuthentication.java

@@ -22,7 +22,6 @@
 import static org.openqa.selenium.remote.Browser.OPERA;
 
 import java.util.function.Predicate;
-import java.util.logging.Logger;
 import org.openqa.selenium.Capabilities;
 import org.openqa.selenium.HasAuthentication;
 import org.openqa.selenium.WebDriver;
@@ -32,7 +31,6 @@
 
 public class AddHasAuthentication implements AugmenterProvider<HasAuthentication> {
 
-  private static final Logger LOG = Logger.getLogger(AddHasAuthentication.class.getName());
   private static final Predicate<String> IS_CHROMIUM_BROWSER =
       name -> CHROME.is(name) || EDGE.is(name) || OPERA.is(name);