Add Nullable annotations to parameters in EdgeDriverService and DriverService constructors

Author: iampopovich

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

@@ -97,11 +97,11 @@ public class EdgeDriverService extends DriverService {
    * @throws IOException If an I/O error occurs.
    */
   public EdgeDriverService(
-      File executable,
+      @Nullable File executable,
       int port,
-      Duration timeout,
-      List<String> args,
-      Map<String, String> environment)
+      @Nullable Duration timeout,
+      @Nullable List<String> args,
+      @Nullable Map<String, String> environment)
       throws IOException {
     super(executable, port, timeout, List.copyOf(args), Map.copyOf(environment));
   }
@@ -197,7 +197,7 @@ public Builder withBuildCheckDisabled(boolean noBuildCheck) {
      * @param logLevel {@link ChromiumDriverLogLevel} for desired log level output.
      * @return A self reference.
      */
-    public Builder withLoglevel(ChromiumDriverLogLevel logLevel) {
+    public Builder withLoglevel(@Nullable ChromiumDriverLogLevel logLevel) {
       this.logLevel = logLevel;
       this.silent = false;
       this.verbose = false;
@@ -239,7 +239,7 @@ public Builder withVerbose(boolean verbose) {
      * @param allowedListIps Comma-separated list of remote IPv4 addresses.
      * @return A self reference.
      */
-    public Builder withAllowedListIps(String allowedListIps) {
+    public Builder withAllowedListIps(@Nullable String allowedListIps) {
       this.allowedListIps = allowedListIps;
       return this;
     }
@@ -250,7 +250,7 @@ public Builder withAllowedListIps(String allowedListIps) {
      * @param readableTimestamp Whether the timestamp of the log is readable.
      * @return A self reference.
      */
-    public Builder withReadableTimestamp(Boolean readableTimestamp) {
+    public Builder withReadableTimestamp(@Nullable Boolean readableTimestamp) {
       this.readableTimestamp = readableTimestamp;
       return this;
     }
@@ -322,7 +322,7 @@ protected List<String> createArgs() {
 
     @Override
     protected EdgeDriverService createDriverService(
-        File exe, int port, Duration timeout, List<String> args, Map<String, String> environment) {
+        @Nullable File exe, int port, @Nullable Duration timeout, @Nullable List<String> args, @Nullable Map<String, String> environment) {
       try {
         return new EdgeDriverService(exe, port, timeout, args, environment);
       } catch (IOException e) {

java/src/org/openqa/selenium/remote/service/DriverService.java

@@ -106,11 +106,11 @@ public class DriverService implements Closeable {
    * @throws IOException If an I/O error occurs.
    */
   protected DriverService(
-      File executable,
+      @Nullable File executable,
       int port,
-      Duration timeout,
-      List<String> args,
-      Map<String, String> environment)
+      @Nullable Duration timeout,
+      @Nullable List<String> args,
+      @Nullable Map<String, String> environment)
       throws IOException {
     if (executable != null) {
       this.executable = executable.getCanonicalPath();