videojs / http-streaming

HLS, DASH, and future HTTP streaming protocols library for video.js
https://videojs-http-streaming.netlify.app/
Other
2.45k stars 416 forks source link

fix: changeType on full codec change only #1474

Closed adrums86 closed 5 months ago

adrums86 commented 5 months ago

Description

Call changeType only when the codec base has changed.

Specific Changes proposed

Parse the codec base from the new codec and the existing codec for the given track and compare that to determine if we need to call changeType on the source buffer.

Requirements Checklist

codecov[bot] commented 5 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (ceb780a) 86.06% compared to head (af2d2e4) 86.06%. Report is 4 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #1474 +/- ## ======================================= Coverage 86.06% 86.06% ======================================= Files 42 42 Lines 10749 10752 +3 Branches 2474 2474 ======================================= + Hits 9251 9254 +3 Misses 1498 1498 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.