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

feat(roll): roll Playwright 1.15.0-next-1630006646000 #871

Merged
merged 2 commits into from
Aug 27, 2021

Conversation

mxschmitt
Copy link
Member

@mxschmitt mxschmitt commented Aug 26, 2021

CI is broken because of pypa/setuptools-scm#608

@mxschmitt mxschmitt force-pushed the roll/1.15.0-next-1630006646000 branch from fc4e846 to 8384034 Compare August 26, 2021 22:45
@@ -3339,8 +3343,8 @@ async def is_hidden(
When true, the call requires selector to resolve to a single element. If given selector resolves to more then one
element, the call throws an exception.
timeout : Union[float, NoneType]
Maximum time in milliseconds, defaults to 30 seconds, pass `0` to disable timeout. The default value can be changed by
using the `browser_context.set_default_timeout()` or `page.set_default_timeout()` methods.
**DEPRECATED** This option is ignored. `frame.is_hidden()` does not wait for the element to become hidden and
Copy link
Contributor

Choose a reason for hiding this comment

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

DeprecationWarning should be emitted for this

@@ -6873,6 +6879,9 @@ async def route(
Once routing is enabled, every request matching the url pattern will stall unless it's continued, fulfilled or aborted.

> NOTE: The handler will only be called for the first url if the response is a redirect.
> NOTE: `page.route()` will not intercept requests intercepted by Service Worker. See
[this](https://github.com/microsoft/playwright/issues/1090) issue. We recommend disabling Service Workers when using
request interception. Via `await context.addInitScript(() => delete window.navigator.serviceWorker);`
Copy link
Contributor

@kumaraditya303 kumaraditya303 Aug 27, 2021

Choose a reason for hiding this comment

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

The snippet should be of python not js/ts

Copy link
Member Author

Choose a reason for hiding this comment

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

feel free to follow up upstream.

@mxschmitt mxschmitt force-pushed the roll/1.15.0-next-1630006646000 branch from 6cefe89 to d318c11 Compare August 27, 2021 12:57
@mxschmitt mxschmitt force-pushed the roll/1.15.0-next-1630006646000 branch from d318c11 to 78c22eb Compare August 27, 2021 13:30
@mxschmitt mxschmitt merged commit ab7502c into microsoft:master Aug 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants