Skip to content

add scoped cdp connect, fix docs, add tests#2

Merged
fiws merged 4 commits intomainfrom
push-zlqvzpuwmtkx
Jan 15, 2026
Merged

add scoped cdp connect, fix docs, add tests#2
fiws merged 4 commits intomainfrom
push-zlqvzpuwmtkx

Conversation

@fiws
Copy link
Copy Markdown
Collaborator

@fiws fiws commented Jan 14, 2026

I assumed that calling close on the browser connected via cdp would close it but this is not the case.
see https://playwright.dev/docs/api/class-browser#browser-close

I discovered this while writing tests for CDP. So i fixed the docs, added a scoped cdp connect method.

I assumed that calling `close` on the browser connected via cdp would
close it but this is not the case. 
see https://playwright.dev/docs/api/class-browser#browser-close

I discovered this while writing tests for CDP. So i fixed the docs, added a scoped cdp connect method.
@fiws fiws self-assigned this Jan 14, 2026
Copy link
Copy Markdown
Contributor

@StefanWerW StefanWerW left a comment

Choose a reason for hiding this comment

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

Looks good!

@fiws fiws merged commit 198d45b into main Jan 15, 2026
1 check passed
@fiws fiws deleted the push-zlqvzpuwmtkx branch January 15, 2026 10:07
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.

2 participants