Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[js] Add windowTypes option support for ChromiumDriver #7897

Merged
merged 6 commits into from
Apr 20, 2021

Conversation

txava
Copy link
Contributor

@txava txava commented Dec 29, 2019

Description

Adds support for setting windowTypes option for ChromeDriver, which
allows getting an extra set of window type handles like webview handles.

Motivation and Context

In order to be able to interact with a webview, one needs to get a handle for the webview (which is a window handle).
ChromeDriver won't return handles for webviews unless it is explicitly specified via windowTypes option see: https://chromedriver.chromium.org/capabilities.

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.

@CLAassistant
Copy link

CLA assistant check
All committers have signed the CLA.

Adds support for setting windowTypes option for ChromeDriver, which
allows getting an extra set of window type handles like webview handles.
For more info, see:  https://chromedriver.chromium.org/capabilities
@txava txava force-pushed the support-windowTypes branch from eaa342d to 1423547 Compare March 8, 2020 20:17
@txava txava changed the title [js] Add windowTypes option support for ChromeDriver [js] Add windowTypes option support for ChromiumDriver Mar 8, 2020
@diemol diemol closed this Jul 12, 2020
@diemol diemol reopened this Jul 12, 2020
@diemol diemol changed the base branch from master to trunk July 12, 2020 19:53
@corevo
Copy link
Member

corevo commented Aug 3, 2020

Please add a test where you add a windowType and assert that the array contains that windowType.

@corevo corevo self-requested a review August 3, 2020 12:28
@diemol
Copy link
Member

diemol commented Dec 22, 2020

@txava, would you like to continue with this PR? Seems there are pending comments to resolve.

@txava
Copy link
Contributor Author

txava commented Apr 10, 2021

@diemol, @corevo

Please add a test where you add a windowType and assert that the array contains that windowType.

Added commit with a test for this request.

Copy link
Member

@harsha509 harsha509 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @txava !

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@harsha509 harsha509 merged commit 18c9454 into SeleniumHQ:trunk Apr 20, 2021
pujagani pushed a commit to pujagani/selenium that referenced this pull request Apr 21, 2021
* [js] windowTypes option support for ChromiumDriver

Adds support for setting windowTypes option for ChromeDriver, which
allows getting an extra set of window type handles like webview handles.
For more info, see:  https://chromedriver.chromium.org/capabilities

* [js] Add test for windowTypes option support for ChromiumDriver

* [js] Replacing deepEqual(deprecated) with deepStrictEqual

Co-authored-by: Sri Harsha <[email protected]>
Co-authored-by: Sri Harsha <[email protected]>
@txava txava deleted the support-windowTypes branch May 8, 2021 14:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants