Skip to content

fix(streams): avoid leaking event listeners#27291

Merged
dgozman merged 1 commit intomicrosoft:mainfrom
dgozman:fix-max-listeners-stream
Sep 25, 2023
Merged

fix(streams): avoid leaking event listeners#27291
dgozman merged 1 commit intomicrosoft:mainfrom
dgozman:fix-max-listeners-stream

Conversation

@dgozman
Copy link
Copy Markdown
Collaborator

@dgozman dgozman commented Sep 25, 2023

Fixes #26998.

@github-actions
Copy link
Copy Markdown
Contributor

Test results for "tests 1"

7 flaky ⚠️ [chromium] › library/tracing.spec.ts:395:14 › should produce screencast frames fit
⚠️ [firefox] › library/proxy.spec.ts:179:3 › should exclude patterns
⚠️ [chromium] › library/tracing.spec.ts:239:5 › should not include trace resources from the previous chunks
⚠️ [chromium] › library/tracing.spec.ts:395:14 › should produce screencast frames fit
⚠️ [webkit] › library/browsercontext-reuse.spec.ts:50:1 › should reset serviceworker
⚠️ [webkit] › page/page-goto.spec.ts:266:3 › should fail when navigating to bad SSL
⚠️ [webkit] › page/page-goto.spec.ts:277:3 › should fail when navigating to bad SSL after redirects

25734 passed, 607 skipped
✔️✔️✔️

Merge workflow run.

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.

[BUG] Getting "MaxListenersExceededWarning: Possible EventEmitter memory leak detected" Error while using await browser.stopTracing();

2 participants