[dotnet] Remove deprecated DevTools capabilities from Firefox options

[nvborisenko]

User description

💥 What does this PR do?

As I remember Firefox doesn't support CDP.

🔧 Implementation Notes

Just remove any mentions about DevTools in Firefox Options class.

💡 Additional Considerations

🔄 Types of changes

• Cleanup (formatting, renaming)
• Bug fix (backwards compatible)
• New feature (non-breaking change which adds functionality and tests!)
• Breaking change (fix or feature that would cause existing functionality to change)

---

PR Type

Other

---

Description

• Remove deprecated DevTools capabilities from Firefox options

• Clean up Firefox driver configuration classes

• Remove EnableDevToolsProtocol property and related constants

---

Changes diagram

flowchart LR
  A["FirefoxOptions class"] -- "remove" --> B["EnableDevToolsProtocol property"]
  A -- "remove" --> C["DevTools capability constant"]
  A -- "remove" --> D["DevTools capability registration"]
  E["Test configuration classes"] -- "remove" --> F["EnableDevToolsProtocol usage"]

Loading

---

Changes walkthrough 📝

Relevant files

Cleanup

FirefoxOptions.cs Remove DevTools protocol support from Firefox options        dotnet/src/webdriver/Firefox/FirefoxOptions.cs Remove FirefoxEnableDevToolsProtocolCapability constant Remove EnableDevToolsProtocol property and its documentation Remove DevTools capability registration in constructor Remove DevTools capability setting in ToCapabilities() method +0/-11    NightlyChannelFirefoxDriver.cs Clean up nightly Firefox driver configuration                        dotnet/test/common/CustomDriverConfigs/NightlyChannelFirefoxDriver.cs Remove EnableDevToolsProtocol = true from default options +1/-1      StableChannelFirefoxDriver.cs Clean up stable Firefox driver configuration                          dotnet/test/common/CustomDriverConfigs/StableChannelFirefoxDriver.cs Remove EnableDevToolsProtocol = true from default options +1/-1

---

Need help?

Type /help how to ... in the comments thread for any questions about Qodo Merge usage.

Check out the documentation for more information.

[qodo-merge-pro]

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪

🧪 No relevant tests

🔒 No security concerns identified

⚡ Recommended focus areas for review Breaking Change Removing the EnableDevToolsProtocol property is a breaking change that could affect existing code using this property. Consider deprecation warnings or version compatibility documentation. public string? BrowserExecutableLocation { get => this.BinaryLocation; set => this.BinaryLocation = value; } /// <summary> /// Gets or sets the logging level of the Firefox driver. /// </summary> public FirefoxDriverLogLevel LogLevel { get; set; } = FirefoxDriverLogLevel.Default; /// <summary> /// Gets or sets the options for automating Firefox on Android. /// </summary> public FirefoxAndroidOptions? AndroidOptions { get; set; }

[qodo-merge-pro]

PR Code Suggestions ✨

No code suggestions found for the PR.

[diemol]

Yes, this was already removed from other bindings, I believe.

[nvborisenko]

Let's move on with standard deprecation policy.

[nvborisenko]

Strange, I found moz:debuggerAddress mentions in py/js/java/rb. So everybody missed this deprecation or I understand this capability incorrectly.

[nvborisenko]

It is planned: https://bugzilla.mozilla.org/show_bug.cgi?id=1967916