[dotnet] Add dispose on constructor failure to ensure driver closes (#13673)

* Forced Dispose of driver on chromedriver exception

* Changed try-catch to only envelop StartSession and moved driver Quit command to own function

* Remove extra spaces

* Undo DriverDispose changes and instead call quit

---------

Co-authored-by: Nikolay Borisenko <[email protected]>

Author: MASACR99

dotnet/src/webdriver/WebDriver.cs

@@ -54,7 +54,18 @@ public class WebDriver : IWebDriver, ISearchContext, IJavaScriptExecutor, IFinds
         protected WebDriver(ICommandExecutor executor, ICapabilities capabilities)
         {
             this.executor = executor;
-            this.StartSession(capabilities);
+
+            try
+            {
+                this.StartSession(capabilities);
+            }
+            catch (Exception)
+            {
+                // Failed to start driver session, disposing of driver
+                this.Quit();
+                throw;
+            }
+
             this.elementFactory = new WebElementFactory(this);
             this.network = new NetworkManager(this);
             this.registeredCommands.AddRange(DriverCommand.KnownCommands);
@@ -692,7 +703,6 @@ protected virtual void Dispose(bool disposing)
             {
                 this.sessionId = null;
             }
-
             this.executor.Dispose();
         }