Inform users of better alternatives to long-used DesiredCapabilities …

…methods
SeleniumHQ · Nov 22, 2017 · a4ac624 · a4ac624
1 parent 8356068
commit a4ac624
Showing 1 changed file with 8 additions and 0 deletions.
diff --git a/java/client/src/org/openqa/selenium/remote/DesiredCapabilities.java b/java/client/src/org/openqa/selenium/remote/DesiredCapabilities.java
@@ -28,9 +28,12 @@
 import org.openqa.selenium.Platform;
 
 import java.util.Map;
+import java.util.logging.Logger;
 
 public class DesiredCapabilities extends MutableCapabilities {
 
+  private static final Logger LOG = Logger.getLogger(Capabilities.class.getName());
+
   public DesiredCapabilities(String browser, String version, Platform platform) {
     setCapability(BROWSER_NAME, browser);
     setCapability(VERSION, version);
@@ -110,10 +113,12 @@ public static DesiredCapabilities android() {
   }
 
   public static DesiredCapabilities chrome() {
+    LOG.info("Using `new ChromeOptions()` is preferred to `DesiredCapabilities.chrome()`");
     return new DesiredCapabilities(BrowserType.CHROME, "", Platform.ANY);
   }
 
   public static DesiredCapabilities firefox() {
+    LOG.info("Using `new FirefoxOptions()` is preferred to `DesiredCapabilities.firefox()`");
     DesiredCapabilities capabilities = new DesiredCapabilities(
         BrowserType.FIREFOX,
         "",
@@ -128,6 +133,7 @@ public static DesiredCapabilities htmlUnit() {
   }
 
   public static DesiredCapabilities edge() {
+    LOG.info("Using `new EdgeOptions()` is preferred to `DesiredCapabilities.edge()`");
     return new DesiredCapabilities(BrowserType.EDGE, "", Platform.WINDOWS);
   }
   public static DesiredCapabilities internetExplorer() {
@@ -155,10 +161,12 @@ public static DesiredCapabilities opera() {
   }
 
   public static DesiredCapabilities operaBlink() {
+    LOG.info("Using `new OperaOptions()` is preferred to `DesiredCapabilities.operaBlink()`");
     return new DesiredCapabilities(BrowserType.OPERA_BLINK, "", Platform.ANY);
   }
 
   public static DesiredCapabilities safari() {
+    LOG.info("Using `new SafariOptions()` is preferred to `DesiredCapabilities.safari()`");
     return new DesiredCapabilities(BrowserType.SAFARI, "", Platform.MAC);
   }