perf(headers): if the guard is immutable (forEach)

[tsctx]

This relates to...

Rationale

See #2424

Changes

Features

Bug Fixes

Breaking Changes and Deprecations

Status

• I have read and agreed to the Developer's Certificate of Origin
• Tested
• Benchmarked (optional)
• Documented
• Review ready
• In review
• Merge ready

[tsctx]

cc @KhafraDev

[codecov-commenter]

Codecov Report

Attention: 29 lines in your changes are missing coverage. Please review.

Comparison is base (e39a632) 85.54% compared to head (51be95b) 85.77%.
Report is 88 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2427      +/-   ##
==========================================
+ Coverage   85.54%   85.77%   +0.22%     
==========================================
  Files          76       76              
  Lines        6858     6925      +67     
==========================================
+ Hits         5867     5940      +73     
+ Misses        991      985       -6     

Files	Coverage Δ
index-fetch.js	100.00% <100.00%> (ø)
lib/api/api-stream.js	100.00% <100.00%> (ø)
lib/core/connect.js	81.25% <100.00%> (ø)
lib/fetch/body.js	97.68% <100.00%> (ø)
lib/fetch/constants.js	97.22% <100.00%> (+0.67%)	⬆️
lib/fetch/file.js	95.45% <100.00%> (+0.05%)	⬆️
lib/fetch/global.js	100.00% <ø> (ø)
lib/fetch/response.js	97.04% <100.00%> (+1.20%)	⬆️
lib/fetch/util.js	86.53% <100.00%> (+0.71%)	⬆️
lib/fetch/webidl.js	94.50% <100.00%> (-0.03%)	⬇️
... and 10 more

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[mcollina]

lgtm

[KhafraDev]

iterating over this uses headers' entries which should already be optimized for immutable headers?

[tsctx]

@KhafraDev
This method has little overhead. But it is unnecessary since most people use iterators. This PR is closed.