Clean up Chrome support in FallbackConfig (#2026)

Author: basil

src/main/java/org/jenkinsci/test/acceptance/FallbackConfig.java

@@ -12,15 +12,12 @@
 import java.io.File;
 import java.io.IOException;
 import java.net.InetAddress;
-import java.net.MalformedURLException;
-import java.net.URL;
 import java.net.UnknownHostException;
 import java.nio.file.Files;
 import java.nio.file.Path;
 import java.nio.file.Paths;
 import java.nio.file.StandardCopyOption;
 import java.time.Duration;
-import java.util.HashMap;
 import java.util.Locale;
 import java.util.Map;
 import java.util.logging.Logger;
@@ -66,7 +63,6 @@
 import org.openqa.selenium.firefox.FirefoxDriver;
 import org.openqa.selenium.firefox.FirefoxOptions;
 import org.openqa.selenium.firefox.GeckoDriverService;
-import org.openqa.selenium.remote.Augmenter;
 import org.openqa.selenium.remote.CapabilityType;
 import org.openqa.selenium.remote.LocalFileDetector;
 import org.openqa.selenium.remote.RemoteWebDriver;
@@ -116,19 +112,10 @@ private WebDriver createWebDriver(TestCleaner cleaner, TestName testName) throws
                 return createContainerWebDriver(
                         cleaner, "selenium/standalone-firefox:4.32.0", buildFirefoxOptions(testName));
             case "chrome-container":
-                return createContainerWebDriver(cleaner, "selenium/standalone-chrome:4.32.0", new ChromeOptions());
+                return createContainerWebDriver(
+                        cleaner, "selenium/standalone-chrome:4.32.0", buildChromeOptions(testName));
             case "chrome":
-                Map<String, String> prefs = new HashMap<>();
-                prefs.put(LANGUAGE_SELECTOR, "en");
-                ChromeOptions options = new ChromeOptions();
-                options.setExperimentalOption("prefs", prefs);
-                if (HarRecorder.isCaptureHarEnabled()) {
-                    options.setAcceptInsecureCerts(true);
-                    options.setProxy(createSeleniumProxy(testName.get()));
-                }
-
-                setDriverPropertyIfMissing("chromedriver", ChromeDriverService.CHROME_DRIVER_EXE_PROPERTY);
-                return new ChromeDriver(options);
+                return new ChromeDriver(buildChromeOptions(testName));
             case "safari":
                 return new SafariDriver();
             case "saucelabs":
@@ -157,16 +144,16 @@ private WebDriver createWebDriver(TestCleaner cleaner, TestName testName) throws
         }
     }
 
-    private WebDriver buildRemoteWebDriver(Capabilities options) throws MalformedURLException {
+    private WebDriver buildRemoteWebDriver(Capabilities options) {
         String u = System.getenv("REMOTE_WEBDRIVER_URL");
         if (StringUtils.isBlank(u)) {
             throw new Error("remote-webdriver type browsers require REMOTE_WEBDRIVER_URL to be set");
         }
-        RemoteWebDriver driver = new RemoteWebDriver(
-                new URL(u), // http://192.168.99.100:4444/wd/hub
-                options);
-        driver.setFileDetector(new LocalFileDetector());
-        return new Augmenter().augment(driver);
+        // http://192.168.99.100:4444/wd/hub
+        WebDriver driver =
+                RemoteWebDriver.builder().address(u).addAlternative(options).build();
+        ((RemoteWebDriver) driver).setFileDetector(new LocalFileDetector());
+        return driver;
     }
 
     private String getBrowser() {
@@ -202,9 +189,15 @@ private FirefoxOptions buildFirefoxOptions(TestName testName) throws IOException
 
     private ChromeOptions buildChromeOptions(TestName testName) throws IOException {
         ChromeOptions chromeOptions = new ChromeOptions();
+        chromeOptions.setExperimentalOption("prefs", Map.of(LANGUAGE_SELECTOR, "en"));
+        chromeOptions.enableBiDi();
         if (HarRecorder.isCaptureHarEnabled()) {
             chromeOptions.setProxy(createSeleniumProxy(testName.get()));
         }
+        setDriverPropertyIfMissing("chromedriver", ChromeDriverService.CHROME_DRIVER_EXE_PROPERTY);
+        chromeOptions.setCapability("se:name", testName.get());
+        chromeOptions.setCapability(
+                "se:recordVideo", TestRecorderRule.isRecorderEnabled() && System.getenv("VIDEO_FOLDER") != null);
         return chromeOptions;
     }
 
@@ -266,10 +259,12 @@ public String toString() {
             Thread.sleep(3000); // Give the container and selenium some time to spawn
 
             try {
-                RemoteWebDriver remoteWebDriver =
-                        new RemoteWebDriver(new URL("http://127.0.0.1:" + controlPort + "/wd/hub"), capabilities);
+                WebDriver driver = RemoteWebDriver.builder()
+                        .address("http://127.0.0.1:" + controlPort + "/wd/hub")
+                        .addAlternative(capabilities)
+                        .build();
                 cleaner.addTask(cleanContainer);
-                return new Augmenter().augment(remoteWebDriver);
+                return driver;
             } catch (RuntimeException e) {
                 cleanContainer.close();
                 throw e;