[grid] Ignoring case when parsing browser name

This is useful when a user types in the
`config.yaml`, "Firefox" instead of "firefox".

Before this change, if the browser name did not
exactly match the one in the code, configs were
ignored.

Author: diemol

java/client/src/org/openqa/selenium/chrome/ChromeDriverInfo.java

@@ -18,6 +18,7 @@
 package org.openqa.selenium.chrome;
 
 import com.google.auto.service.AutoService;
+
 import org.openqa.selenium.Capabilities;
 import org.openqa.selenium.ImmutableCapabilities;
 import org.openqa.selenium.SessionNotCreatedException;
@@ -45,9 +46,9 @@ public Capabilities getCanonicalCapabilities() {
 
   @Override
   public boolean isSupporting(Capabilities capabilities) {
-    return BrowserType.CHROME.equals(capabilities.getBrowserName()) ||
-      capabilities.getCapability("chromeOptions") != null ||
-      capabilities.getCapability("goog:chromeOptions") != null;
+    return BrowserType.CHROME.equalsIgnoreCase(capabilities.getBrowserName()) ||
+           capabilities.getCapability("chromeOptions") != null ||
+           capabilities.getCapability("goog:chromeOptions") != null;
   }
 
   @Override

java/client/src/org/openqa/selenium/edge/EdgeDriverInfo.java

@@ -47,12 +47,12 @@ public Capabilities getCanonicalCapabilities() {
 
   @Override
   public boolean isSupporting(Capabilities capabilities) {
-    return (BrowserType.EDGE.equals(capabilities.getBrowserName())
-      || capabilities.getCapability("ms:edgeOptions") != null
-      || capabilities.getCapability("edgeOptions") != null)
-      &&
-      (capabilities.getCapability(EdgeOptions.USE_CHROMIUM) == null
-        || Objects.equals(capabilities.getCapability(EdgeOptions.USE_CHROMIUM), true));
+    return (BrowserType.EDGE.equalsIgnoreCase(capabilities.getBrowserName())
+            || capabilities.getCapability("ms:edgeOptions") != null
+            || capabilities.getCapability("edgeOptions") != null)
+           &&
+           (capabilities.getCapability(EdgeOptions.USE_CHROMIUM) == null
+            || Objects.equals(capabilities.getCapability(EdgeOptions.USE_CHROMIUM), true));
   }
 
   @Override

java/client/src/org/openqa/selenium/firefox/GeckoDriverInfo.java

@@ -51,7 +51,7 @@ public boolean isSupporting(Capabilities capabilities) {
       return false;
     }
 
-    if (BrowserType.FIREFOX.equals(capabilities.getBrowserName())) {
+    if (BrowserType.FIREFOX.equalsIgnoreCase(capabilities.getBrowserName())) {
       return true;
     }
 

java/client/src/org/openqa/selenium/ie/InternetExplorerDriverInfo.java

@@ -18,6 +18,7 @@
 package org.openqa.selenium.ie;
 
 import com.google.auto.service.AutoService;
+
 import org.openqa.selenium.Capabilities;
 import org.openqa.selenium.ImmutableCapabilities;
 import org.openqa.selenium.SessionNotCreatedException;
@@ -45,8 +46,8 @@ public Capabilities getCanonicalCapabilities() {
 
   @Override
   public boolean isSupporting(Capabilities capabilities) {
-    return BrowserType.IE.equals(capabilities.getBrowserName()) ||
-      capabilities.getCapability("se:ieOptions") != null;
+    return BrowserType.IE.equalsIgnoreCase(capabilities.getBrowserName()) ||
+           capabilities.getCapability("se:ieOptions") != null;
   }
 
   @Override

java/client/src/org/openqa/selenium/opera/OperaDriverInfo.java

@@ -46,8 +46,8 @@ public Capabilities getCanonicalCapabilities() {
 
   @Override
   public boolean isSupporting(Capabilities capabilities) {
-    return BrowserType.OPERA_BLINK.equals(capabilities.getBrowserName()) ||
-      BrowserType.OPERA.equals(capabilities.getBrowserName());
+    return BrowserType.OPERA_BLINK.equalsIgnoreCase(capabilities.getBrowserName()) ||
+           BrowserType.OPERA.equalsIgnoreCase(capabilities.getBrowserName());
   }
 
   @Override

java/client/src/org/openqa/selenium/safari/SafariDriverInfo.java

@@ -18,6 +18,7 @@
 package org.openqa.selenium.safari;
 
 import com.google.auto.service.AutoService;
+
 import org.openqa.selenium.Capabilities;
 import org.openqa.selenium.ImmutableCapabilities;
 import org.openqa.selenium.SessionNotCreatedException;
@@ -45,7 +46,7 @@ public Capabilities getCanonicalCapabilities() {
 
   @Override
   public boolean isSupporting(Capabilities capabilities) {
-    if (BrowserType.SAFARI.equals(capabilities.getBrowserName())) {
+    if (BrowserType.SAFARI.equalsIgnoreCase(capabilities.getBrowserName())) {
       return true;
     }
 

java/client/src/org/openqa/selenium/safari/SafariTechPreviewDriverInfo.java

@@ -18,6 +18,7 @@
 package org.openqa.selenium.safari;
 
 import com.google.auto.service.AutoService;
+
 import org.openqa.selenium.Capabilities;
 import org.openqa.selenium.ImmutableCapabilities;
 import org.openqa.selenium.SessionNotCreatedException;
@@ -44,7 +45,7 @@ public Capabilities getCanonicalCapabilities() {
 
   @Override
   public boolean isSupporting(Capabilities capabilities) {
-    if (SafariOptions.SAFARI_TECH_PREVIEW.equals(capabilities.getBrowserName())) {
+    if (SafariOptions.SAFARI_TECH_PREVIEW.equalsIgnoreCase(capabilities.getBrowserName())) {
       return true;
     }