wire: use quicvarint.Parse when parsing frames

[marten-seemann]

name                  old time/op    new time/op    delta
ParseStreamAndACK-16     290ns ± 6%     244ns ± 4%  -15.80%  (p=0.000 n=9+10)
ParseOtherFrames-16      217ns ± 3%     171ns ± 3%  -21.43%  (p=0.000 n=9+10)

name                  old alloc/op   new alloc/op   delta
ParseStreamAndACK-16    1.58kB ± 0%    1.58kB ± 0%     ~     (all equal)
ParseOtherFrames-16       224B ± 0%      224B ± 0%     ~     (all equal)

name                  old allocs/op  new allocs/op  delta
ParseStreamAndACK-16      2.00 ± 0%      2.00 ± 0%     ~     (all equal)
ParseOtherFrames-16       6.00 ± 0%      6.00 ± 0%     ~     (all equal)

[marten-seemann]

Good thing we enabled ClusterFuzzLite PR fuzzing. It caught a pretty serious bug that would otherwise have gone unnoticed.

[codecov]

Codecov Report

Attention: Patch coverage is 98.33333% with 4 lines in your changes are missing coverage. Please review.

Project coverage is 85.08%. Comparing base (a5adbd4) to head (e40a215).
Report is 2 commits behind head on master.

Files	Patch %	Lines
internal/wire/ack_frame.go	91.18%	3 Missing ⚠️
internal/wire/frame_parser.go	97.62%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #4484      +/-   ##
==========================================
+ Coverage   84.92%   85.08%   +0.16%     
==========================================
  Files         154      154              
  Lines       14676    14719      +43     
==========================================
+ Hits        12463    12523      +60     
+ Misses       1700     1687      -13     
+ Partials      513      509       -4     

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