[java] remove deprecated driver service constructors and methods

Author: titusfortner

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

@@ -89,26 +89,6 @@ public class ChromeDriverService extends DriverService {
   public static final String CHROME_DRIVER_DISABLE_BUILD_CHECK =
       "webdriver.chrome.disableBuildCheck";
 
-  /**
-   * @param executable The ChromeDriver executable.
-   * @param port Which port to start the ChromeDriver on.
-   * @param args The arguments to the launched server.
-   * @param environment The environment for the launched server.
-   * @throws IOException If an I/O error occurs.
-   * @deprecated use {@link ChromeDriverService#ChromeDriverService(File, int, Duration, List, Map)}
-   */
-  @Deprecated
-  public ChromeDriverService(
-      File executable, int port, List<String> args, Map<String, String> environment)
-      throws IOException {
-    this(
-        executable,
-        port,
-        DEFAULT_TIMEOUT,
-        unmodifiableList(new ArrayList<>(args)),
-        unmodifiableMap(new HashMap<>(environment)));
-  }
-
   /**
    * @param executable The ChromeDriver executable.
    * @param port Which port to start the ChromeDriver on.
@@ -249,20 +229,6 @@ public Builder withVerbose(boolean verbose) {
       return this;
     }
 
-    /**
-     * Configures the comma-separated list of remote IPv4 addresses which are allowed to connect to
-     * the driver server.
-     *
-     * @param allowedListIps Comma-separated list of remote IPv4 addresses.
-     * @return A self reference.
-     * @deprecated use {@link #withAllowedListIps(String)}
-     */
-    @Deprecated
-    public Builder withWhitelistedIps(String allowedListIps) {
-      this.allowedListIps = allowedListIps;
-      return this;
-    }
-
     /**
      * Configures the comma-separated list of remote IPv4 addresses which are allowed to connect to
      * the driver server.

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

@@ -71,26 +71,6 @@ public class GeckoDriverService extends FirefoxDriverService {
    */
   public static final String GECKO_DRIVER_PROFILE_ROOT = "webdriver.firefox.profileRoot";
 
-  /**
-   * @param executable The GeckoDriver executable.
-   * @param port Which port to start the GeckoDriver on.
-   * @param args The arguments to the launched server.
-   * @param environment The environment for the launched server.
-   * @throws IOException If an I/O error occurs.
-   * @deprecated use {@link GeckoDriverService#GeckoDriverService(File, int, Duration, List, Map)}
-   */
-  @Deprecated
-  public GeckoDriverService(
-      File executable, int port, List<String> args, Map<String, String> environment)
-      throws IOException {
-    super(
-        executable,
-        port,
-        DEFAULT_TIMEOUT,
-        unmodifiableList(new ArrayList<>(args)),
-        unmodifiableMap(new HashMap<>(environment)));
-  }
-
   /**
    * @param executable The GeckoDriver executable.
    * @param port Which port to start the GeckoDriver on.
@@ -262,12 +242,6 @@ protected List<String> createArgs() {
         args.add(profileRoot.getAbsolutePath());
       }
 
-      // deprecated
-      if (firefoxBinary != null) {
-        args.add("--binary");
-        args.add(firefoxBinary.getPath());
-      }
-
       if (allowHosts != null) {
         args.add("--allow-hosts");
         args.addAll(Arrays.asList(allowHosts.split(" ")));

java/src/org/openqa/selenium/safari/SafariDriverService.java

@@ -52,21 +52,6 @@ public class SafariDriverService extends DriverService {
 
   private static final File SAFARI_DRIVER_EXECUTABLE = new File("/usr/bin/safaridriver");
 
-  /**
-   * @param executable The SafariDriver executable.
-   * @param port Which port to start the SafariDriver on.
-   * @param args The arguments to the launched server.
-   * @param environment The environment for the launched server.
-   * @throws IOException If an I/O error occurs.
-   * @deprecated use {@link SafariDriverService#SafariDriverService(File, int, Duration, List, Map)}
-   */
-  @Deprecated
-  public SafariDriverService(
-      File executable, int port, List<String> args, Map<String, String> environment)
-      throws IOException {
-    this(executable, port, DEFAULT_TIMEOUT, args, environment);
-  }
-
   /**
    * @param executable The SafariDriver executable.
    * @param port Which port to start the SafariDriver on.

java/src/org/openqa/selenium/safari/SafariTechPreviewDriverService.java

@@ -53,22 +53,6 @@ public class SafariTechPreviewDriverService extends DriverService {
   private static final File TP_SAFARI_DRIVER_EXECUTABLE =
       new File("/Applications/Safari Technology Preview.app/Contents/MacOS/safaridriver");
 
-  /**
-   * @param executable The SafariDriver executable.
-   * @param port Which port to start the SafariDriver on.
-   * @param args The arguments to the launched server.
-   * @param environment The environment for the launched server.
-   * @throws IOException If an I/O error occurs.
-   * @deprecated use {@link SafariTechPreviewDriverService#SafariTechPreviewDriverService(File, int,
-   *     Duration, List, Map)}
-   */
-  @Deprecated
-  public SafariTechPreviewDriverService(
-      File executable, int port, List<String> args, Map<String, String> environment)
-      throws IOException {
-    this(executable, port, DEFAULT_TIMEOUT, args, environment);
-  }
-
   /**
    * @param executable The SafariDriver executable.
    * @param port Which port to start the SafariDriver on.