[java] Removing deprecated createServiceWithConfig in ChromeDriverService

diemol · diemol · commit fab49a5543f3 · 2023-07-25T21:39:30.000+02:00
diff --git a/java/src/org/openqa/selenium/chrome/ChromeDriver.java b/java/src/org/openqa/selenium/chrome/ChromeDriver.java
@@ -68,7 +68,7 @@ public ChromeDriver(ChromeDriverService service) {
    * @see #ChromeDriver(ChromeDriverService, ChromeOptions)
    */
   public ChromeDriver(ChromeOptions options) {
-    this(ChromeDriverService.createServiceWithConfig(options), options);
+    this(ChromeDriverService.createDefaultService(), options);
   }
 
   /**
diff --git a/java/src/org/openqa/selenium/chrome/ChromeDriverService.java b/java/src/org/openqa/selenium/chrome/ChromeDriverService.java
@@ -166,23 +166,6 @@ public static ChromeDriverService createDefaultService() {
     return new Builder().build();
   }
 
-  /**
-   * Configures and returns a new {@link ChromeDriverService} using the supplied configuration. In
-   * this configuration, the service will use the ChromeDriver executable identified by {@link
-   * org.openqa.selenium.remote.service.DriverFinder#getPath(DriverService, Capabilities)}. Each
-   * service created by this method will be configured to use a free port on the current system.
-   *
-   * @return A new ChromeDriverService using the supplied configuration from {@link ChromeOptions}.
-   * @deprecated Use {@link Builder#withLogLevel(ChromiumDriverLogLevel)} }
-   */
-  @Deprecated
-  public static ChromeDriverService createServiceWithConfig(ChromeOptions options) {
-    ChromeDriverLogLevel oldLevel = options.getLogLevel();
-    ChromiumDriverLogLevel level =
-        (oldLevel == null) ? null : ChromiumDriverLogLevel.fromString(oldLevel.toString());
-    return new Builder().withLogLevel(level).build();
-  }
-
   /** Builder used to configure new {@link ChromeDriverService} instances. */
   @AutoService(DriverService.Builder.class)
   public static class Builder

Original file line number	Diff line number	Diff line change
`@@ -68,7 +68,7 @@ public ChromeDriver(ChromeDriverService service) {`
`68`	`68`	`* @see #ChromeDriver(ChromeDriverService, ChromeOptions)`
`69`	`69`	`*/`
`70`	`70`	`public ChromeDriver(ChromeOptions options) {`
`71`		`- this(ChromeDriverService.createServiceWithConfig(options), options);`
	`71`	`+ this(ChromeDriverService.createDefaultService(), options);`
`72`	`72`	`}`
`73`	`73`
`74`	`74`	`/**`