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(webdriver): support Page.deleteCookie()
for WebDriver BiDi
#12031
Conversation
ba6b22d
to
64b7469
Compare
Let's add a test to also understand the cdp behavior better. |
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.
Probably need to enable some tests via TestExpectations
2d454b9
to
6b46913
Compare
// TODO: delete cookie before setting them. | ||
// await this.deleteCookie(bidiCookie); | ||
|
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.
The scope of set and deleted cookie can be different, so deleting can cause some unwilling side effects. Example: a cookie is set for a specific path, but removed from the whole domain.
4c2d036
to
0304512
Compare
31ef7c7
to
7244abc
Compare
This reverts commit 201e8a6.
7244abc
to
e33ee32
Compare
Page.deleteCookie()
Page.deleteCookie()
for WebDriver BiDi
36c75c1
to
fdac75a
Compare
fdac75a
to
4fdddff
Compare
Implement
Page.deleteCookie()
via BiDistorage.deleteCookies
.