[java] minimize overhead of regex compilation

Author: joerg1985

java/src/org/openqa/selenium/By.java

@@ -22,6 +22,8 @@
 import java.util.List;
 import java.util.Map;
 import java.util.Objects;
+import java.util.regex.Pattern;
+
 import org.openqa.selenium.internal.Require;
 
 /**
@@ -414,6 +416,7 @@ protected final Map<String, Object> toJson() {
   }
 
   private abstract static class PreW3CLocator extends By implements Remotable {
+    private static final Pattern CSS_ESCAPE = Pattern.compile("([\\s'\"\\\\#.:;,!?+<>=~*^$|%&@`{}\\-\\/\\[\\]\\(\\)])");
     private final Parameters remoteParams;
     private final ByCssSelector fallback;
 
@@ -442,7 +445,7 @@ protected final Map<String, Object> toJson() {
     }
 
     private String cssEscape(String using) {
-      using = using.replaceAll("([\\s'\"\\\\#.:;,!?+<>=~*^$|%&@`{}\\-\\/\\[\\]\\(\\)])", "\\\\$1");
+      using = CSS_ESCAPE.matcher(using).replaceAll("\\\\$1");
       if (using.length() > 0 && Character.isDigit(using.charAt(0))) {
         using = "\\" + (30 + Integer.parseInt(using.substring(0, 1))) + " " + using.substring(1);
       }

java/src/org/openqa/selenium/remote/codec/w3c/W3CHttpCommandCodec.java

@@ -84,6 +84,7 @@
 import java.util.List;
 import java.util.Map;
 import java.util.concurrent.ConcurrentHashMap;
+import java.util.regex.Pattern;
 import java.util.stream.Collectors;
 import java.util.stream.Stream;
 import org.openqa.selenium.InvalidSelectorException;
@@ -99,6 +100,7 @@
 public class W3CHttpCommandCodec extends AbstractHttpCommandCodec {
 
   private static final ConcurrentHashMap<String, String> ATOM_SCRIPTS = new ConcurrentHashMap<>();
+  private static final Pattern CSS_ESCAPE = Pattern.compile("([\\s'\"\\\\#.:;,!?+<>=~*^$|%&@`{}\\-\\/\\[\\]\\(\\)])");
 
   public W3CHttpCommandCodec() {
     String sessionId = "/session/:sessionId";
@@ -376,7 +378,7 @@ private Map<String, String> asElement(Object id) {
   }
 
   private String cssEscape(String using) {
-    using = using.replaceAll("([\\s'\"\\\\#.:;,!?+<>=~*^$|%&@`{}\\-\\/\\[\\]\\(\\)])", "\\\\$1");
+    using = CSS_ESCAPE.matcher(using).replaceAll("\\\\$1");
     if (using.length() > 0 && Character.isDigit(using.charAt(0))) {
       using = "\\" + (30 + Integer.parseInt(using.substring(0, 1))) + " " + using.substring(1);
     }

java/src/org/openqa/selenium/remote/http/jdk/JdkHttpMessages.java

@@ -148,7 +148,12 @@ public URI getRawUri(HttpRequest req) {
         || uri.startsWith("https://")) {
       rawUrl = uri;
     } else {
-      rawUrl = baseUrl.toString().replaceAll("/$", "") + uri;
+      String base = baseUrl.toString();
+      if (base.endsWith("/")) {
+        rawUrl = base.substring(0, base.length() - 1) + uri;
+      } else {
+        rawUrl = base + uri;
+      }
     }
 
     return URI.create(rawUrl);

java/src/org/openqa/selenium/remote/http/netty/NettyMessages.java

@@ -138,7 +138,12 @@ private static String getRawUrl(URI baseUrl, String uri) {
     } else if (uri.startsWith("http://") || uri.startsWith("https://")) {
       rawUrl = uri;
     } else {
-      rawUrl = baseUrl.toString().replaceAll("/$", "") + uri;
+      String base = baseUrl.toString();
+      if (base.endsWith("/")) {
+        rawUrl = base.substring(0, base.length() - 1) + uri;
+      } else {
+        rawUrl = base + uri;
+      }
     }
     return rawUrl;
   }