Rewrite VP8 isKeyFrame check

[xdrudis]

Description

RFC 6386 describes the least significant bit of the first byte of the header as:

A 1-bit frame type (0 for key frames, 1 for interframes).

The change is functionally a no-op, but the naming implies the wrong logic (you would assume isKeyFrame == 1 means it is a key frame, but the opposite is true).

Reference issue

Fixes #...

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 78.00%. Comparing base (5edce95) to head (6379b9e).
Report is 1 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #2999   +/-   ##
=======================================
  Coverage   78.00%   78.00%           
=======================================
  Files          89       89           
  Lines       10578    10578           
=======================================
  Hits         8251     8251           
  Misses       1837     1837           
  Partials      490      490           

Flag	Coverage Δ
go	79.57% <100.00%> (ø)
wasm	63.68% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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

[Sean-Der]

Nice catch :)

you are really looking into the details!