Skip to content

Commit

Permalink
[dotnet] Add CDP deprecation warning for Firefox
Browse files Browse the repository at this point in the history
  • Loading branch information
pujagani committed Nov 15, 2024
1 parent d55eff3 commit 0fc56db
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 0 deletions.
1 change: 1 addition & 0 deletions dotnet/src/webdriver/Firefox/FirefoxDriver.cs
Original file line number Diff line number Diff line change
Expand Up @@ -401,6 +401,7 @@ public DevToolsSession GetDevToolsSession(int devToolsProtocolVersion)
/// Creates a session to communicate with a browser using a Developer Tools debugging protocol.
/// </summary>
/// <returns>The active session to use to communicate with the Developer Tools debugging protocol.</returns>
[Obsolete("CDP support for Firefox is deprecated and will be removed in future versions. Please switch to WebDriver BiDi.")]
public DevToolsSession GetDevToolsSession(DevToolsOptions options)
{
if (this.devToolsSession == null)
Expand Down
10 changes: 10 additions & 0 deletions dotnet/src/webdriver/Remote/RemoteWebDriver.cs
Original file line number Diff line number Diff line change
Expand Up @@ -63,6 +63,8 @@ namespace OpenQA.Selenium.Remote
/// </example>
public class RemoteWebDriver : WebDriver, IDevTools, IHasDownloads
{
private readonly ILogger _logger = Log.GetLogger(typeof(RemoteWebDriver));

/// <summary>
/// The name of the Selenium grid remote DevTools end point capability.
/// </summary>
Expand Down Expand Up @@ -425,6 +427,14 @@ public ReadOnlyCollection<IWebElement> FindElementsByCssSelector(string cssSelec
/// <returns>The active session to use to communicate with the Developer Tools debugging protocol.</returns>
public DevToolsSession GetDevToolsSession()
{
if (this.Capabilities.GetCapability(BrowserName) == "firefox")
{
if (_logger.IsEnabled(LogEventLevel.Warn))
{
_logger.Warn("CDP support for Firefox is deprecated and will be removed in future versions. Please switch to WebDriver BiDi.");
}
}

return GetDevToolsSession(new DevToolsOptions() { ProtocolVersion = DevToolsSession.AutoDetectDevToolsProtocolVersion });
}

Expand Down

0 comments on commit 0fc56db

Please sign in to comment.