Skip to content

Commit

Permalink
fix(TPC) Fix H.264 simulcast.
Browse files Browse the repository at this point in the history
Regression was introduced in 07f037e. Also process encodeTime stats when H.264 is selected
  • Loading branch information
jallamsetty1 committed Dec 4, 2024
1 parent 39c4422 commit 144b0ca
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 3 deletions.
2 changes: 1 addition & 1 deletion modules/RTC/TPCUtils.js
Original file line number Diff line number Diff line change
Expand Up @@ -917,7 +917,7 @@ export class TPCUtils {
* @returns {string} the munged SDP.
*/
updateAv1DdHeaders(parsedSdp) {
if (!this.supportsDDHeaderExt) {
if (!browser.supportsDDExtHeaders()) {
return parsedSdp;
}
const mungedSdp = parsedSdp;
Expand Down
2 changes: 1 addition & 1 deletion modules/RTC/TraceablePeerConnection.js
Original file line number Diff line number Diff line change
Expand Up @@ -542,7 +542,7 @@ TraceablePeerConnection.prototype._getReceiversByEndpointIds = function(endpoint
*/
TraceablePeerConnection.prototype.isSpatialScalabilityOn = function() {
const h264SimulcastEnabled = this.tpcUtils.codecSettings[CodecMimeType.H264].scalabilityModeEnabled
&& this._supportsDDHeaderExt;
&& this.tpcUtils.supportsDDHeaderExt;

return !this.options.disableSimulcast
&& (this.codecSettings.codecList[0] !== CodecMimeType.H264 || h264SimulcastEnabled);
Expand Down
1 change: 0 additions & 1 deletion modules/statistics/RTPStatsCollector.js
Original file line number Diff line number Diff line change
Expand Up @@ -630,7 +630,6 @@ StatsCollector.prototype.processStatsReport = function() {
if (now.type === 'outbound-rtp'
&& now.active
&& track?.isVideoTrack()
&& this.peerconnection.usesCodecSelectionAPI()
&& before?.totalEncodeTime
&& before?.framesEncoded
&& now.frameHeight
Expand Down

0 comments on commit 144b0ca

Please sign in to comment.