Skip to content

[dotnet] [bidi] Support DestinationFolder optional arg in StartScreencast - #18042

Merged
nvborisenko merged 1 commit into
SeleniumHQ:trunkfrom
nvborisenko:bidi-screencast-destination-folder
Sep 16, 2026
Merged

nvborisenko merged 1 commit into
SeleniumHQ:trunkfrom
nvborisenko:bidi-screencast-destination-folder

Conversation

@nvborisenko

Copy link
Copy Markdown
Member

https://w3c.github.io/webdriver-bidi/#command-browsingContext-startScreencast

💥 What does this PR do?

This pull request updates the screencast functionality in the BrowsingContext module to support specifying a destination folder for screencast output. The main changes involve updating the parameters and options for starting a screencast.

Screencast destination folder support:

  • Added a new DestinationFolder property to the StartScreencastOptions class, allowing users to specify where screencast files should be saved.
  • Updated the StartScreencastParameters record to include the DestinationFolder parameter, and modified the StartScreencastAsync method to pass this value when starting a screencast. [1] [2]

🤖 AI assistance

  • No substantial AI assistance used
  • AI assisted (complete below)
    • Tool(s):
    • What was generated:
    • I reviewed all AI output and can explain the change

🔄 Types of changes

  • New feature (non-breaking change)

@qodo-code-review

Copy link
Copy Markdown
Contributor

Qodo reviews are paused for this user.

Troubleshooting steps vary by plan Learn more →

On a Teams plan?
Reviews resume once this user has a paid seat and their Git account is linked in Qodo.
Link Git account →

Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center?
These require an Enterprise plan - Contact us
Contact us →

@selenium-ci selenium-ci added the C-dotnet .NET Bindings label Sep 16, 2026
@nvborisenko
nvborisenko merged commit 03002c9 into SeleniumHQ:trunk Sep 16, 2026
26 checks passed
@nvborisenko
nvborisenko deleted the bidi-screencast-destination-folder branch September 16, 2026 19:46
This was referenced Oct 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C-dotnet .NET Bindings

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants