[dotnet] remove deprecated DriverService constructors with options pa…

…rameter
SeleniumHQ · Jan 19, 2024 · 5eb3caa · 5eb3caa
1 parent 4562685
commit 5eb3caa
Show file tree

Hide file tree

Showing 5 changed files with 0 additions and 60 deletions.
diff --git a/dotnet/src/webdriver/Chrome/ChromeDriverService.cs b/dotnet/src/webdriver/Chrome/ChromeDriverService.cs
@@ -56,18 +56,6 @@ public static ChromeDriverService CreateDefaultService()
             return new ChromeDriverService(null, null, PortUtilities.FindFreePort());
         }
 
-        /// <summary>
-        /// Creates a default instance of the ChromeDriverService.
-        /// </summary>
-        /// /// <param name="options">Browser options used to find the correct ChromeDriver binary.</param>
-        /// <returns>A ChromeDriverService that implements default settings.</returns>
-        [Obsolete("CreateDefaultService() now evaluates options in Driver constructor")]
-        public static ChromeDriverService CreateDefaultService(ChromeOptions options)
-        {
-            string fullServicePath = DriverFinder.FullPath(options);
-            return CreateDefaultService(Path.GetDirectoryName(fullServicePath), Path.GetFileName(fullServicePath));
-        }
-
         /// <summary>
         /// Creates a default instance of the ChromeDriverService using a specified path to the ChromeDriver executable.
         /// </summary>

diff --git a/dotnet/src/webdriver/Edge/EdgeDriverService.cs b/dotnet/src/webdriver/Edge/EdgeDriverService.cs
@@ -65,18 +65,6 @@ public static EdgeDriverService CreateDefaultService()
             return new EdgeDriverService(null, null, PortUtilities.FindFreePort());
         }
 
-        /// <summary>
-        /// Creates a default instance of the EdgeDriverService.
-        /// </summary>
-        /// <param name="options">Browser options used to find the correct MSEdgeDriver binary.</param>
-        /// <returns>A EdgeDriverService that implements default settings.</returns>
-        [Obsolete("CreateDefaultService() now evaluates options in Driver constructor")]
-        public static EdgeDriverService CreateDefaultService(EdgeOptions options)
-        {
-            string fullServicePath = DriverFinder.FullPath(options);
-            return CreateDefaultService(Path.GetDirectoryName(fullServicePath), Path.GetFileName(fullServicePath));
-        }
-
         /// <summary>
         /// Creates a default instance of the EdgeDriverService using a specified path to the EdgeDriver executable.
         /// </summary>

diff --git a/dotnet/src/webdriver/Firefox/FirefoxDriverService.cs b/dotnet/src/webdriver/Firefox/FirefoxDriverService.cs
@@ -218,18 +218,6 @@ public static FirefoxDriverService CreateDefaultService()
         }
 
 
-        /// <summary>
-        /// Creates a default instance of the FirefoxDriverService.
-        /// </summary>
-        /// <param name="options">Browser options used to find the correct GeckoDriver binary.</param>
-        /// <returns>A FirefoxDriverService that implements default settings.</returns>
-        [Obsolete("CreateDefaultService() now evaluates options in Driver constructor")]
-        public static FirefoxDriverService CreateDefaultService(FirefoxOptions options)
-        {
-            string fullServicePath = DriverFinder.FullPath(options);
-            return CreateDefaultService(Path.GetDirectoryName(fullServicePath), Path.GetFileName(fullServicePath));
-        }
-
         /// <summary>
         /// Creates a default instance of the FirefoxDriverService using a specified path to the Firefox driver executable.
         /// </summary>

diff --git a/dotnet/src/webdriver/IE/InternetExplorerDriverService.cs b/dotnet/src/webdriver/IE/InternetExplorerDriverService.cs
@@ -158,18 +158,6 @@ public static InternetExplorerDriverService CreateDefaultService()
             return new InternetExplorerDriverService(null, null, PortUtilities.FindFreePort());
         }
 
-        /// <summary>
-        /// Creates a default instance of the InternetExplorerDriverService.
-        /// </summary>
-        /// <param name="options">Browser options used to find the correct IEDriver binary.</param>
-        /// <returns>A InternetExplorerDriverService that implements default settings.</returns>
-        [Obsolete("CreateDefaultService() now evaluates options in Driver constructor")]
-        public static InternetExplorerDriverService CreateDefaultService(InternetExplorerOptions options)
-        {
-            string fullServicePath = DriverFinder.FullPath(options);
-            return CreateDefaultService(Path.GetDirectoryName(fullServicePath), Path.GetFileName(fullServicePath));
-        }
-
         /// <summary>
         /// Creates a default instance of the InternetExplorerDriverService using a specified path to the IEDriverServer executable.
         /// </summary>

diff --git a/dotnet/src/webdriver/Safari/SafariDriverService.cs b/dotnet/src/webdriver/Safari/SafariDriverService.cs
@@ -171,18 +171,6 @@ public static SafariDriverService CreateDefaultService()
             return new SafariDriverService(null, null, PortUtilities.FindFreePort());
         }
 
-        /// <summary>
-        /// Creates a default instance of the SafariDriverService.
-        /// </summary>
-        /// <param name="options">Browser options used to find the correct GeckoDriver binary.</param>
-        /// <returns>A SafariDriverService that implements default settings.</returns>
-        [Obsolete("CreateDefaultService() now evaluates options in Driver constructor")]
-        public static SafariDriverService CreateDefaultService(SafariOptions options)
-        {
-            string fullServicePath = DriverFinder.FullPath(options);
-            return CreateDefaultService(Path.GetDirectoryName(fullServicePath), Path.GetFileName(fullServicePath));
-        }
-
         /// <summary>
         /// Creates a default instance of the SafariDriverService using a specified path to the SafariDriver executable.
         /// </summary>