[🚀 Feature]: Do not enable CDP for Firefox

[thc202]

Feature and motivation

Selenium now (4.27+) warns when Firefox CDP is enabled:

selenium/java/src/org/openqa/selenium/firefox/FirefoxDriver.java

Lines 168 to 172 in 19fc217

	if (reportedUri.isPresent()) {
	LOG.warning(
	"CDP support for Firefox is deprecated and will be removed in future versions. "
	+ "Please switch to WebDriver BiDi.");
	}

but at the same time it is enabling it:

selenium/java/src/org/openqa/selenium/firefox/FirefoxOptions.java

Lines 66 to 69 in 9d68245

	// Firefox 129 onwards the CDP protocol will not be enabled by default. Setting this preference
	// will enable it.
	// https://fxdx.dev/deprecating-cdp-support-in-firefox-embracing-the-future-with-webdriver-bidi/.
	addPreference("remote.active-protocols", 3);

requiring the user to set the Firefox preference to just enable BiDi and suppress the deprecation warning.

Since CDP is deprecated I think Selenium should stop enabling that protocol and remove the preference.

[github-actions]

@thc202, thank you for creating this issue. We will troubleshoot it as soon as we can.

---

Info for maintainers

Triage this issue by using labels.

If information is missing, add a helpful comment and then I-issue-template label.

If the issue is a question, add the I-question label.

If the issue is valid but there is no time to troubleshoot it, consider adding the help wanted label.

If the issue requires changes or fixes from an external project (e.g., ChromeDriver, GeckoDriver, MSEdgeDriver, W3C), add the applicable G-* label, and it will provide the correct link and auto-close the issue.

After troubleshooting the issue, please add the R-awaiting answer label.

Thank you!

[diemol]

Our policy is to add the deprecation and then wait for two releases to remove the feature. This should be removed around February. However, some coordination is needed with the Firefox team because they are also removing CDP from Firefox.

As it is, there is nothing to do for the team for now.

[thc202]

It would have been better that the deprecation warn was only enabled if the user was actually using CDP rather than warn all users even the ones that are not using CDP at all (as is the case now since BiDi is what's being used internally), but fair if it's the policy.

[diemol]

That is a good point. We should improve that for the next time.

[github-actions]

This issue has been automatically locked since there has not been any recent activity since it was closed. Please open a new issue for related bugs.