Skip to content

fix: emit responseCompleted at the right time#3835

Open
OrKoN wants to merge 2 commits intomainfrom
orkon/loading-failed
Open

fix: emit responseCompleted at the right time#3835
OrKoN wants to merge 2 commits intomainfrom
orkon/loading-failed

Conversation

@OrKoN
Copy link
Copy Markdown
Collaborator

@OrKoN OrKoN commented Oct 12, 2025

Closes #3574

@OrKoN OrKoN force-pushed the orkon/loading-failed branch from 68d3a9e to efac0d1 Compare October 12, 2025 10:37
@OrKoN OrKoN marked this pull request as ready for review October 12, 2025 10:51
@OrKoN OrKoN added puppeteer Run Puppeteer test when added to PR DO NOT MERGE Don't merge the marked PR labels Oct 12, 2025
@OrKoN
Copy link
Copy Markdown
Collaborator Author

OrKoN commented Oct 14, 2025

This PR cannot be landed because it would only emit responseCompleted for network requests that were consumed which might be according to the spec but currently fails some Puppeteer tests. Probably depends on fixing puppeteer/puppeteer#14315 first

Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

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

Benchmark

Details
Benchmark suite Current: d906cc1 Previous: 4184afc Ratio
ubuntu-latest-new-headless-node:test_performance_screenshot 156.1389446258545 ms 137.89057731628418 ms 1.13
macos-latest-new-headless-cd:test_performance_screenshot 2892.750597000122 ms 2487.3996257781982 ms 1.16
macos-latest-headful-node:test_performance_screenshot 2039.85481262207 ms 2778.6727905273438 ms 0.73
macos-latest-headful-cd:test_performance_screenshot 3262.034034729004 ms 1890.6847953796387 ms 1.73
ubuntu-latest-old-headless-node:test_performance_screenshot 268.94264221191406 ms 252.52380371093753 ms 1.07
macos-latest-old-headless-cd:test_performance_screenshot 236.11798286437988 ms 265.80657958984375 ms 0.89
macos-latest-old-headless-node:test_performance_screenshot 256.19964599609375 ms 226.79271697998047 ms 1.13
ubuntu-latest-new-headless-cd:test_performance_screenshot 178.29489707946777 ms 177.354097366333 ms 1.01
ubuntu-latest-old-headless-cd:test_performance_screenshot 268.0319309234619 ms 272.5314140319824 ms 0.98
ubuntu-latest-headful-node:test_performance_screenshot 169.14963722229004 ms

This comment was automatically generated by workflow using github-action-benchmark.

Copy link
Copy Markdown
Collaborator

@sadym-chromium sadym-chromium left a comment

Choose a reason for hiding this comment

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

Is the description accurate?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

DO NOT MERGE Don't merge the marked PR puppeteer Run Puppeteer test when added to PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

network.responseCompleted is emitted too early

3 participants