#54109: test: reduce protocol throughput timing noise
Merged
Description of Change
Follow-up to #54028 (1e4b7cb) and #53206.
The re-enabled test fails on Linux CI. Separate timing phases can bias the ratio when host load changes. Compare paired samples in alternating order and time them in the renderer. Keep the 3x limit and verify each request's path.
Local Linux build: all 179 protocol tests passed. Lint, formatting, and spec typecheck passed. The full suite and other platforms were not run. Azure validation is pending.
Checklist
- I have built and tested this change
- I have filled out the PR description
- I have reviewed and verified the changes
-
npm testpasses - tests are changed or added
- PR release notes describe the change in a way relevant to app developers, and are capitalized, punctuated, past tense, and no longer than 80 characters.
Release Notes
Notes: none
Backports
Semver Impact
Major
Breaking changes
Minor
New features
Patch
Bug fixes
None
Docs, tests, etc.
Semantic Versioning helps users understand the impact of updates:
- Major (X.y.z): Breaking changes that may require code modifications
- Minor (x.Y.z): New features that maintain backward compatibility
- Patch (x.y.Z): Bug fixes that don't change the API
- None: Changes that don't affect using facing parts of Electron