[java] deprecate setting chrome driver service class logging values in Chrome options

titusfortner · titusfortner · commit c9bb6f40d1f3 · 2023-01-14T17:38:28.000-06:00
diff --git a/java/src/org/openqa/selenium/chrome/ChromeDriverService.java b/java/src/org/openqa/selenium/chrome/ChromeDriverService.java
@@ -153,7 +153,9 @@ public static ChromeDriverService createDefaultService() {
    * 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(ChromeDriverLogLevel)}  }
    */
+  @Deprecated
   public static ChromeDriverService createServiceWithConfig(ChromeOptions options) {
     return new Builder()
       .withLogLevel(options.getLogLevel())
diff --git a/java/src/org/openqa/selenium/chrome/ChromeOptions.java b/java/src/org/openqa/selenium/chrome/ChromeOptions.java
@@ -58,6 +58,10 @@ public ChromeOptions() {
     super(CapabilityType.BROWSER_NAME, CHROME.browserName(), CAPABILITY);
   }
 
+  /**
+   * @deprecated Use {@link ChromeDriverService.Builder#withLogLevel(ChromeDriverLogLevel)} to set log level.
+   */
+  @Deprecated
   public ChromeOptions setLogLevel(ChromeDriverLogLevel logLevel){
     this.logLevel = Require.nonNull("Log level", logLevel);
     return this;
@@ -75,6 +79,10 @@ public ChromeOptions merge(Capabilities extraCapabilities) {
     return newInstance;
   }
 
+  /**
+   * @deprecated Log Level only applies to {@link ChromeDriverService}.
+   */
+  @Deprecated
   public ChromeDriverLogLevel getLogLevel(){
     return logLevel;
   }

Original file line number	Diff line number	Diff line change
`@@ -58,6 +58,10 @@ public ChromeOptions() {`
`58`	`58`	`super(CapabilityType.BROWSER_NAME, CHROME.browserName(), CAPABILITY);`
`59`	`59`	`}`
`60`	`60`
	`61`	`+ /**`
	`62`	`+ * @deprecated Use {@link ChromeDriverService.Builder#withLogLevel(ChromeDriverLogLevel)} to set log level.`
	`63`	`+ */`
	`64`	`+ @Deprecated`
`61`	`65`	`public ChromeOptions setLogLevel(ChromeDriverLogLevel logLevel){`
`62`	`66`	`this.logLevel = Require.nonNull("Log level", logLevel);`
`63`	`67`	`return this;`
`@@ -75,6 +79,10 @@ public ChromeOptions merge(Capabilities extraCapabilities) {`
`75`	`79`	`return newInstance;`
`76`	`80`	`}`
`77`	`81`
	`82`	`+ /**`
	`83`	`+ * @deprecated Log Level only applies to {@link ChromeDriverService}.`
	`84`	`+ */`
	`85`	`+ @Deprecated`
`78`	`86`	`public ChromeDriverLogLevel getLogLevel(){`
`79`	`87`	`return logLevel;`
`80`	`88`	`}`