From e6fb3feeb6f73e6098aa0f7b3fc2077e1ca2bf9a Mon Sep 17 00:00:00 2001
From: titusfortner <titusfortner@gmail.com>
Date: Fri, 24 Sep 2021 17:45:05 -0500
Subject: [PATCH] [dotnet] mark deprecated chromedriver commands as obsolete
 and implement new preferred command

---
 dotnet/src/webdriver/Chrome/ChromeDriver.cs   |  1 +
 .../src/webdriver/Chromium/ChromiumDriver.cs  | 34 +++++++++++--------
 dotnet/src/webdriver/Edge/EdgeDriver.cs       |  1 +
 3 files changed, 22 insertions(+), 14 deletions(-)

diff --git a/dotnet/src/webdriver/Chrome/ChromeDriver.cs b/dotnet/src/webdriver/Chrome/ChromeDriver.cs
index 7d1d6ba0ffae1..9f9e96c2c3948 100644
--- a/dotnet/src/webdriver/Chrome/ChromeDriver.cs
+++ b/dotnet/src/webdriver/Chrome/ChromeDriver.cs
@@ -138,6 +138,7 @@ public ChromeDriver(ChromeDriverService service, ChromeOptions options)
         public ChromeDriver(ChromeDriverService service, ChromeOptions options, TimeSpan commandTimeout)
             : base(service, options, commandTimeout)
         {
+            this.AddCustomChromeCommand(ExecuteCdp, HttpCommandInfo.PostCommand, "/session/{sessionId}/goog/cdp/execute");
         }
 
     }
diff --git a/dotnet/src/webdriver/Chromium/ChromiumDriver.cs b/dotnet/src/webdriver/Chromium/ChromiumDriver.cs
index 5a89bce81dfe5..957bc94dcfe47 100644
--- a/dotnet/src/webdriver/Chromium/ChromiumDriver.cs
+++ b/dotnet/src/webdriver/Chromium/ChromiumDriver.cs
@@ -43,6 +43,7 @@ public abstract class ChromiumDriver : WebDriver, ISupportsLogs, IDevTools
         private const string DeleteNetworkConditionsCommand = "deleteNetworkConditions";
         private const string SendChromeCommand = "sendChromeCommand";
         private const string SendChromeCommandWithResult = "sendChromeCommandWithResult";
+        protected const string ExecuteCdp = "executeCdpCommand";
 
         private readonly string optionsCapabilityName;
         private DevToolsSession devToolsSession;
@@ -128,11 +129,12 @@ public ChromiumNetworkConditions NetworkConditions
         }
 
         /// <summary>
-        /// Executes a custom Chrome command.
+        /// Executes a custom Chrome Dev Tools Protocol Command.
         /// </summary>
         /// <param name="commandName">Name of the command to execute.</param>
         /// <param name="commandParameters">Parameters of the command to execute.</param>
-        public void ExecuteChromeCommand(string commandName, Dictionary<string, object> commandParameters)
+        /// <returns>An object representing the result of the command, if applicable.</returns>
+        public object ExecuteCdpCommand(string commandName, Dictionary<string, object> commandParameters)
         {
             if (commandName == null)
             {
@@ -142,7 +144,19 @@ public void ExecuteChromeCommand(string commandName, Dictionary<string, object>
             Dictionary<string, object> parameters = new Dictionary<string, object>();
             parameters["cmd"] = commandName;
             parameters["params"] = commandParameters;
-            this.Execute(SendChromeCommand, parameters);
+            Response response = this.Execute(ExecuteCdp, parameters);
+            return response.Value;
+        }
+
+        /// <summary>
+        /// Executes a custom Chrome command.
+        /// </summary>
+        /// <param name="commandName">Name of the command to execute.</param>
+        /// <param name="commandParameters">Parameters of the command to execute.</param>
+        [Obsolete("ExecuteChromeCommand is deprecated in favor of ExecuteCdpCommand.")]
+        public void ExecuteChromeCommand(string commandName, Dictionary<string, object> commandParameters)
+        {
+            ExecuteCdpCommand(commandName, commandParameters);
         }
 
         /// <summary>
@@ -151,18 +165,10 @@ public void ExecuteChromeCommand(string commandName, Dictionary<string, object>
         /// <param name="commandName">Name of the command to execute.</param>
         /// <param name="commandParameters">Parameters of the command to execute.</param>
         /// <returns>An object representing the result of the command.</returns>
+        [Obsolete("ExecuteChromeCommandWithResult is deprecated in favor of ExecuteCdpCommand.")]
         public object ExecuteChromeCommandWithResult(string commandName, Dictionary<string, object> commandParameters)
         {
-            if (commandName == null)
-            {
-                throw new ArgumentNullException("commandName", "commandName must not be null");
-            }
-
-            Dictionary<string, object> parameters = new Dictionary<string, object>();
-            parameters["cmd"] = commandName;
-            parameters["params"] = commandParameters;
-            Response response = this.Execute(SendChromeCommandWithResult, parameters);
-            return response.Value;
+            return ExecuteCdpCommand(commandName, commandParameters);
         }
 
         /// <summary>
@@ -248,7 +254,7 @@ private static ICapabilities ConvertOptionsToCapabilities(ChromiumOptions option
             return options.ToCapabilities();
         }
 
-        private void AddCustomChromeCommand(string commandName, string method, string resourcePath)
+        protected void AddCustomChromeCommand(string commandName, string method, string resourcePath)
         {
             HttpCommandInfo commandInfoToAdd = new HttpCommandInfo(method, resourcePath);
             this.CommandExecutor.TryAddCommand(commandName, commandInfoToAdd);
diff --git a/dotnet/src/webdriver/Edge/EdgeDriver.cs b/dotnet/src/webdriver/Edge/EdgeDriver.cs
index 1cb91dc407c70..4e95fd96bfb01 100644
--- a/dotnet/src/webdriver/Edge/EdgeDriver.cs
+++ b/dotnet/src/webdriver/Edge/EdgeDriver.cs
@@ -106,6 +106,7 @@ public EdgeDriver(EdgeDriverService service, EdgeOptions options)
         public EdgeDriver(EdgeDriverService service, EdgeOptions options, TimeSpan commandTimeout)
             : base(service, options, commandTimeout)
         {
+            this.AddCustomChromeCommand(ExecuteCdp, HttpCommandInfo.PostCommand, "/session/{sessionId}/ms/cdp/execute");
         }
     }
 }