[dotnet] have url property setter call navigation method instead of t…

…he other way around
SeleniumHQ · May 17, 2024 · 2ddc605 · 2ddc605
1 parent 7f67d6e
commit 2ddc605
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 13 deletions.
diff --git a/dotnet/src/webdriver/Navigator.cs b/dotnet/src/webdriver/Navigator.cs
@@ -17,6 +17,7 @@
 // </copyright>
 
 using System;
+using System.Collections.Generic;
 
 namespace OpenQA.Selenium
 {
@@ -58,7 +59,17 @@ public void Forward()
         /// <param name="url">String of where you want the browser to go to</param>
         public void GoToUrl(string url)
         {
-            this.driver.Url = url;
+            if (url == null)
+            {
+                throw new ArgumentNullException(nameof(url), "URL cannot be null.");
+            }
+
+            Dictionary<string, object> parameters = new Dictionary<string, object>
+            {
+                { "url", url }
+            };
+            this.driver.InternalExecute(DriverCommand.Get, parameters);
+
         }
 
         /// <summary>
@@ -72,7 +83,7 @@ public void GoToUrl(Uri url)
                 throw new ArgumentNullException(nameof(url), "URL cannot be null.");
             }
 
-            this.driver.Url = url.ToString();
+            this.GoToUrl(url.ToString());
         }
 
         /// <summary>

diff --git a/dotnet/src/webdriver/WebDriver.cs b/dotnet/src/webdriver/WebDriver.cs
@@ -109,17 +109,7 @@ public string Url
                 return commandResponse.Value.ToString();
             }
 
-            set
-            {
-                if (value == null)
-                {
-                    throw new ArgumentNullException(nameof(value), "Argument 'url' cannot be null.");
-                }
-
-                Dictionary<string, object> parameters = new Dictionary<string, object>();
-                parameters.Add("url", value);
-                this.Execute(DriverCommand.Get, parameters);
-            }
+            set => new Navigator(this).GoToUrl(value);
         }
 
         /// <summary>