[dotnet] Remove planned obsoleted members for 4.44

[nvborisenko]

💥 What does this PR do?

This pull request primarily removes deprecated APIs and updates access modifiers for several fields in the ChromiumDriver class. The main goal is to clean up obsolete code and improve encapsulation.

Removal of deprecated APIs

• Removed the obsolete ICommandServer interface from Remote/ICommandServer.cs, which is no longer supported and scheduled for removal.
• Removed the obsolete Start() method from DriverService.cs in favor of the asynchronous StartAsync(CancellationToken) method.

Code cleanup and encapsulation improvements

• Changed several static fields in ChromiumDriver.cs from public to protected, as planned, to restrict their visibility to derived classes. Also, removed the [Obsolete] attributes and related pragma warnings. [1] [2]

🔄 Types of changes

• Cleanup (formatting, renaming)
• Breaking change (fix or feature that would cause existing functionality to change)

[qodo-code-review]

Review Summary by Qodo

Remove planned obsolete members for .NET 4.44 release

🐞 Bug fix

Walkthroughs

Description

• Remove obsolete ICommandServer interface no longer supported
• Delete deprecated synchronous Start() method from DriverService
• Change 16 static fields in ChromiumDriver from public to protected
• Remove obsolete attributes and pragma warning directives

File Changes

1. dotnet/src/webdriver/Chromium/ChromiumDriver.cs Cleanup +15/-33

Hide obsolete chromium static fields with protected access

• Changed 16 static readonly fields from public to protected access modifier
• Removed [Obsolete] attributes from all static fields
• Removed pragma warning disable directive for obsolete members
• Cleaned up inline comments referencing the planned changes

dotnet/src/webdriver/Chromium/ChromiumDriver.cs

---

2. dotnet/src/webdriver/DriverService.cs Cleanup +0/-11

Remove deprecated synchronous Start method

• Removed deprecated synchronous Start() method
• Method was marked obsolete and delegated to StartAsync()
• Users should use StartAsync(CancellationToken) instead

dotnet/src/webdriver/DriverService.cs

---

3. dotnet/src/webdriver/Remote/ICommandServer.cs Cleanup +0/-32

Delete obsolete ICommandServer interface

• Deleted entire file containing obsolete ICommandServer interface
• Interface was marked as no longer supported
• No replacement interface provided as functionality is deprecated

dotnet/src/webdriver/Remote/ICommandServer.cs

---

[qodo-code-review]

Code Review by Qodo

🐞 Bugs (0)   📘 Rule violations (2)   📎 Requirement gaps (0)   🎨 UX Issues (0)

📘\ ≡ Correctness (2)

1. AcceptUntrustedCertificates no longer public 📘 ≡

Description

Static fields on ChromiumDriver were changed from public to protected, which is a breaking
change for external consumers referencing these members. This violates the requirement to preserve
backward-compatible public APIs for upgrades.

Code

dotnet/src/webdriver/Chromium/ChromiumDriver.cs[35]

+    protected static readonly bool AcceptUntrustedCertificates = true;

Evidence

PR Compliance ID 1 forbids breaking public API changes without a compatibility path. The diff
changes AcceptUntrustedCertificates (and similar command-name fields nearby) from `public static
readonly to protected static readonly`, removing them from the public surface area.

AGENTS.md
dotnet/src/webdriver/Chromium/ChromiumDriver.cs[35-35]

Agent prompt

The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

## Issue description
`ChromiumDriver` static fields were changed from `public` to `protected`, which breaks existing consumers referencing these members.

## Issue Context
If the intent is to tighten encapsulation, keep a compatibility surface (at least temporarily) so users can upgrade without changing code.

## Fix Focus Areas
- dotnet/src/webdriver/Chromium/ChromiumDriver.cs[35-105]

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools

---

2. DriverService.Start() removed 📘 ≡

Description

The synchronous DriverService.Start() method was removed, which breaks existing code calling it.
This violates the requirement to preserve backward-compatible public APIs for user-facing changes.

Code

dotnet/src/webdriver/DriverService.cs[L194-203]

-    /// <summary>
-    /// Starts the driver service if it is not already running.
-    /// </summary>
-    /// <exception cref="InvalidOperationException">If the driver service path is specified but the driver service executable name is not.</exception>
-    /// <exception cref="WebDriverException">If the service fails to initialize within the timeout period or exits unexpectedly.</exception>
-    [Obsolete("Use StartAsync(CancellationToken) instead. This method will be removed in a future release (4.43).")]
-    public void Start()
-    {
-        this.StartAsync().GetAwaiter().GetResult();
-    }

Evidence

PR Compliance ID 1 requires maintaining backward compatibility for public APIs. The diff deletes the
public void Start() method, removing it from the public API surface and causing compile-time
breaks for consumers.

AGENTS.md
dotnet/src/webdriver/DriverService.cs[194-203]

Agent prompt

The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

## Issue description
`DriverService.Start()` was deleted, breaking existing callers.

## Issue Context
If `StartAsync(CancellationToken)` is the preferred API, keep `Start()` as an `[Obsolete]` forwarding shim (calling `StartAsync().GetAwaiter().GetResult()`) until an appropriate major/breaking release window.

## Fix Focus Areas
- dotnet/src/webdriver/DriverService.cs[194-204]

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools

---

ⓘ The new review experience is currently in Beta. Learn more

[Copilot]

Pull request overview

This PR removes previously-obsoleted .NET WebDriver APIs and tightens encapsulation of ChromiumDriver command constants as part of the planned 4.44 cleanup.

Changes:

• Removed the obsolete ICommandServer interface from the Remote namespace.
• Removed the obsolete synchronous DriverService.Start() method in favor of StartAsync(...).
• Changed several ChromiumDriver static fields from public to protected and removed the related obsoletion/pragma scaffolding.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File	Description
dotnet/src/webdriver/Remote/ICommandServer.cs	Deletes the obsolete ICommandServer API surface.
dotnet/src/webdriver/DriverService.cs	Removes the obsolete sync Start() wrapper, leaving StartAsync(...) as the supported entry point.
dotnet/src/webdriver/Chromium/ChromiumDriver.cs	Makes command-name fields protected (usable by derived drivers) and removes obsolete/pragma noise.