Removing Firefox profile cleaning step in .NET bindings

Firefox used to require launching to clean up potential issues with the
anonymous temporary profile that we use with the driver. This is no longer
the case.

Author: jimevans

Diff for: dotnet/src/webdriver/Firefox/FirefoxBinary.cs

@@ -219,28 +219,6 @@ public void WaitForProcessExit()
             this.process.WaitForExit();
         }
 
-        /// <summary>
-        /// Initializes the binary with the specified profile.
-        /// </summary>
-        /// <param name="profile">The <see cref="FirefoxProfile"/> to use to initialize the binary.</param>
-        public void Clean(FirefoxProfile profile)
-        {
-            if (profile == null)
-            {
-                throw new ArgumentNullException("profile", "profile cannot be null");
-            }
-
-            this.StartProfile(profile, "-silent");
-            try
-            {
-                this.WaitForProcessExit();
-            }
-            catch (ThreadInterruptedException e)
-            {
-                throw new WebDriverException("Thread was interrupted", e);
-            }
-        }
-
         /// <summary>
         /// Stops the execution of this <see cref="FirefoxBinary"/>, terminating the process if necessary.
         /// </summary>

Diff for: dotnet/src/webdriver/Firefox/FirefoxDriverServer.cs

@@ -89,7 +89,6 @@ public void Start()
                     int portToUse = DetermineNextFreePort(this.host, this.profile.Port);
                     this.profile.Port = portToUse;
                     this.profile.WriteToDisk();
-                    this.process.Clean(this.profile);
                     this.process.StartProfile(this.profile, new string[] { "-foreground" });
 
                     this.SetAddress(portToUse);