Fix FedCM command definition

This lets us re-enable the FedCM tests, which this change does we
well.

In addition, newer versions of Chrome look for a "login_url" field instead
of "signin_url", so this updates the fedcm.json file accordingly.

This test failure was pointed out here:
SeleniumHQ#12096 (comment)

common/src/web/fedcm/fedcm.json

@@ -2,5 +2,6 @@
   "accounts_endpoint": "accounts.json",
   "client_metadata_endpoint": "client_metadata.json",
   "id_assertion_endpoint": "id_assertion",
-  "signin_url": "/signin"
+  "signin_url": "/signin",
+  "login_url": "/signin"
 }

java/src/org/openqa/selenium/remote/codec/AbstractHttpCommandCodec.java

@@ -197,14 +197,15 @@ public AbstractHttpCommandCodec() {
     defineCommand(SET_USER_VERIFIED, post(webauthnId + "/uv"));
 
     // Federated Credential Management API
-    defineCommand(CANCEL_DIALOG, post("/fedcm/canceldialog"));
-    defineCommand(SELECT_ACCOUNT, post("/fedcm/selectaccount"));
-    defineCommand(CLICK_DIALOG, post("/fedcm/clickdialogbutton"));
-    defineCommand(GET_ACCOUNTS, get("/fedcm/accountlist"));
-    defineCommand(GET_FEDCM_TITLE, get("/fedcm/gettitle"));
-    defineCommand(GET_FEDCM_DIALOG_TYPE, get("/fedcm/getdialogtype"));
-    defineCommand(SET_DELAY_ENABLED, post("/fedcm/setdelayenabled"));
-    defineCommand(RESET_COOLDOWN, post("/fedcm/resetcooldown"));
+    String fedcm = sessionId + "/fedcm";
+    defineCommand(CANCEL_DIALOG, post(fedcm + "/canceldialog"));
+    defineCommand(SELECT_ACCOUNT, post(fedcm + "/selectaccount"));
+    defineCommand(CLICK_DIALOG, post(fedcm + "/clickdialogbutton"));
+    defineCommand(GET_ACCOUNTS, get(fedcm + "/accountlist"));
+    defineCommand(GET_FEDCM_TITLE, get(fedcm + "/gettitle"));
+    defineCommand(GET_FEDCM_DIALOG_TYPE, get(fedcm + "/getdialogtype"));
+    defineCommand(SET_DELAY_ENABLED, post(fedcm + "/setdelayenabled"));
+    defineCommand(RESET_COOLDOWN, post(fedcm + "/resetcooldown"));
 
     defineCommand(GET_DOWNLOADABLE_FILES, get(sessionId + "/se/files"));
     defineCommand(DOWNLOAD_FILE, post(sessionId + "/se/files"));

java/test/org/openqa/selenium/federatedcredentialmanagement/FederatedCredentialManagementTest.java

@@ -36,7 +36,6 @@
 import org.openqa.selenium.chrome.ChromeOptions;
 import org.openqa.selenium.support.ui.WebDriverWait;
 import org.openqa.selenium.testing.JupiterTestBase;
-import org.openqa.selenium.testing.NotYetImplemented;
 
 class FederatedCredentialManagementTest extends JupiterTestBase {
 
@@ -81,9 +80,6 @@ private int getSecurePort() {
   }
 
   @Test
-  @NotYetImplemented(
-      value = CHROME,
-      reason = "https://github.com/SeleniumHQ/selenium/pull/12096#issuecomment-2017760822")
   void testDismissDialog() {
     fedcmDriver.setDelayEnabled(false);
     assertNull(fedcmDriver.getFederatedCredentialManagementDialog());
@@ -116,9 +112,6 @@ void testDismissDialog() {
   }
 
   @Test
-  @NotYetImplemented(
-      value = CHROME,
-      reason = "https://github.com/SeleniumHQ/selenium/pull/12096#issuecomment-2017760822")
   void testSelectAccount() {
     fedcmDriver.setDelayEnabled(false);
     assertNull(fedcmDriver.getFederatedCredentialManagementDialog());