Skip to content

[JS][bidi] added captureScreenshot command - #12510

Merged
pujagani merged 9 commits into
SeleniumHQ:trunkfrom
TamsilAmani:js-bidi-capture-screenshot
Oct 17, 2023
Merged

[JS][bidi] added captureScreenshot command#12510
pujagani merged 9 commits into
SeleniumHQ:trunkfrom
TamsilAmani:js-bidi-capture-screenshot

Conversation

@TamsilAmani

@TamsilAmani TamsilAmani commented Aug 7, 2023

Copy link
Copy Markdown
Contributor

Description

Added captureScreenshot methods for different options (no clip, box, element) along with tests.

Motivation and Context

To implement BiDi command https://w3c.github.io/webdriver-bidi/#command-browsingContext-captureScreenshot
Add feature #12317

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist

  • I have read the contributing document.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@TamsilAmani
TamsilAmani requested a review from harsha509 August 7, 2023 10:44
Comment thread javascript/node/selenium-webdriver/bidi/browsingContext.js Outdated
Comment thread javascript/node/selenium-webdriver/bidi/browsingContext.js Outdated
Comment thread javascript/node/selenium-webdriver/bidi/browsingContext.js
@isaulv isaulv added the C-nodejs JavaScript Bindings label Aug 15, 2023
@pujagani pujagani added the B-devtools Includes everything BiDi or Chrome DevTools related label Sep 26, 2023
Comment thread javascript/node/selenium-webdriver/bidi/browsingContext.js Outdated
Comment thread javascript/node/selenium-webdriver/bidi/browsingContext.js Outdated
Comment thread javascript/node/selenium-webdriver/test/bidi/bidi_test.js
@pujagani

Copy link
Copy Markdown
Contributor

Fixed/resolved all comments. The failing test is not associated with the changes. Can we merge this?

@pujagani
pujagani merged commit bb12405 into SeleniumHQ:trunk Oct 17, 2023
aguspe pushed a commit to aguspe/selenium that referenced this pull request Oct 22, 2023
Co-authored-by: Sri Harsha <12621691+harsha509@users.noreply.github.com>
Co-authored-by: Puja Jagani <puja.jagani93@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

B-devtools Includes everything BiDi or Chrome DevTools related C-nodejs JavaScript Bindings

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants