Simplify thread safety now that we have a wait in place

Author: RenderMichael

dotnet/src/webdriver/DriverService.cs

@@ -240,46 +240,37 @@ public async Task StartAsync()
         {
             if (this.driverServiceProcess == null)
             {
-                lock (this.driverServiceProcessLock)
-                {
+                this.driverServiceProcess = new Process();
 
-                    if (this.driverServiceProcess == null)
+                try
+                {
+                    if (this.DriverServicePath != null)
                     {
-                        var driverServiceProcess = new Process();
-
-                        try
+                        if (this.DriverServiceExecutableName is null)
                         {
-                            if (this.DriverServicePath != null)
-                            {
-                                if (this.DriverServiceExecutableName is null)
-                                {
-                                    throw new InvalidOperationException("If the driver service path is specified, the driver service executable name must be as well");
-                                }
-
-                                driverServiceProcess.StartInfo.FileName = Path.Combine(this.DriverServicePath, this.DriverServiceExecutableName);
-                            }
-                            else
-                            {
-                                driverServiceProcess.StartInfo.FileName = new DriverFinder(this.GetDefaultDriverOptions()).GetDriverPath();
-                            }
+                            throw new InvalidOperationException("If the driver service path is specified, the driver service executable name must be as well");
+                        }
 
-                            driverServiceProcess.StartInfo.Arguments = this.CommandLineArguments;
-                            driverServiceProcess.StartInfo.UseShellExecute = false;
-                            driverServiceProcess.StartInfo.CreateNoWindow = this.HideCommandPromptWindow;
+                        this.driverServiceProcess.StartInfo.FileName = Path.Combine(this.DriverServicePath, this.DriverServiceExecutableName);
+                    }
+                    else
+                    {
+                        this.driverServiceProcess.StartInfo.FileName = new DriverFinder(this.GetDefaultDriverOptions()).GetDriverPath();
+                    }
 
-                            DriverProcessStartingEventArgs eventArgs = new DriverProcessStartingEventArgs(driverServiceProcess.StartInfo);
-                            this.OnDriverProcessStarting(eventArgs);
+                    this.driverServiceProcess.StartInfo.Arguments = this.CommandLineArguments;
+                    this.driverServiceProcess.StartInfo.UseShellExecute = false;
+                    this.driverServiceProcess.StartInfo.CreateNoWindow = this.HideCommandPromptWindow;
 
-                            driverServiceProcess.Start();
-                        }
-                        catch
-                        {
-                            driverServiceProcess.Dispose();
-                            throw;
-                        }
+                    DriverProcessStartingEventArgs eventArgs = new DriverProcessStartingEventArgs(this.driverServiceProcess.StartInfo);
+                    this.OnDriverProcessStarting(eventArgs);
 
-                        this.driverServiceProcess = driverServiceProcess;
-                    }
+                    this.driverServiceProcess.Start();
+                }
+                catch
+                {
+                    this.driverServiceProcess.Dispose();
+                    throw;
                 }
             }