-
Notifications
You must be signed in to change notification settings - Fork 1k
chore: roll to 1.53.1 #2892
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
chore: roll to 1.53.1 #2892
Conversation
playwright/_impl/_browser_type.py
Outdated
params["clientCertificates"] = await to_client_certificates_protocol( | ||
params["clientCertificates"] | ||
) | ||
params["selectorEngines"] = self._playwright.selectors._selectorEngines |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is copied from _browser.py
, with the changes being this line and below.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd like a follow up with an explicit timeout in each method, instead of a global "timeout calculator". It feels like that would be less magic and not so bad in terms of code.
Originally mirrored microsoft/playwright-dotnet#3184, but due to the selector bug (microsoft/playwright#36227) causing half the Python tests to fail, the roll had to be made to include this change. Thus the roll is the entire roll to
1.53.1
.Implements changes from microsoft/playwright#35888, microsoft/playwright#35969, and microsoft/playwright#35988, among others.
References #2818