SimulcastConsumer: Fix frozen video when syncing keyframe is discarde…

…d due to too high RTP timestamp extra offset needed (#999)

- Fixes #990.
- Rationale here: #990 (comment).
- Basically we should reset this->syncRequired` nor `this->spatialLayerToSync` when the keyframe that is supposed to switch to another spatial layer is discarded due to too high RTP timestamp extra offset needed.

CHANGELOG.md

@@ -4,6 +4,7 @@
 ### NEXT
 
 * `SimulcastConsumer::GetDesiredBitrate()`: Choose the highest bitrate among all Producer streams ([PR #992](https://github.com/versatica/mediasoup/pull/992)).
+* `SimulcastConsumer`: Fix frozen video when syncing keyframe is discarded due to too high RTP timestamp extra offset needed ([PR #999](https://github.com/versatica/mediasoup/pull/999)).
 * Update NPM deps.
 
 

worker/src/RTC/SimulcastConsumer.cpp

@@ -809,6 +809,10 @@ namespace RTC
 
 					this->keyFrameForTsOffsetRequested = true;
 
+					// Reset flags since we are discarding this key frame.
+					this->syncRequired       = false;
+					this->spatialLayerToSync = -1;
+
 					return;
 				}